diff mbox

[3/7,ARC,LRA] Fix tests asm constraints.

Message ID 1496324097-21221-4-git-send-email-claziss@synopsys.com
State New
Headers show

Commit Message

Claudiu Zissulescu June 1, 2017, 1:34 p.m. UTC 
LRA doesn't like the 'X' constraint as used in our tests, remove it.

gcc/testsuite
2017-01-09  Claudiu Zissulescu  <claziss@synopsys.com>

	* gcc.target/arc/mulsi3_highpart-1.c: Remove 'X' constraint.
	* gcc.target/arc/mulsi3_highpart-2.c: Likewise.
---
 gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c | 2 +-
 gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Andrew Burgess June 16, 2017, 7:37 p.m. UTC | #1 
* Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com> [2017-06-01 15:34:53 +0200]:

> LRA doesn't like the 'X' constraint as used in our tests, remove it.
> 
> gcc/testsuite
> 2017-01-09  Claudiu Zissulescu  <claziss@synopsys.com>
> 
> 	* gcc.target/arc/mulsi3_highpart-1.c: Remove 'X' constraint.
> 	* gcc.target/arc/mulsi3_highpart-2.c: Likewise.

This looks fine and could be applied straight away I think.

Thanks,
Andrew



> ---
>  gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c | 2 +-
>  gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
> index 57cb95b..5fd6c36 100644
> --- a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
> +++ b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
> @@ -7,7 +7,7 @@
>  static int
>  id (int i)
>  {
> -  asm ("": "+Xr" (i));
> +  asm ("": "+r" (i));
>    return i;
>  }
>  
> diff --git a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
> index 287d96d..6ec4bc5 100644
> --- a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
> +++ b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
> @@ -9,7 +9,7 @@
>  static int
>  id (int i)
>  {
> -  asm ("": "+Xr" (i));
> +  asm ("": "+r" (i));
>    return i;
>  }
>  
> -- 
> 1.9.1
>
Claudiu Zissulescu July 10, 2017, 1:56 p.m. UTC | #2 
> > gcc/testsuite
> > 2017-01-09  Claudiu Zissulescu  <claziss@synopsys.com>
> >
> > 	* gcc.target/arc/mulsi3_highpart-1.c: Remove 'X' constraint.
> > 	* gcc.target/arc/mulsi3_highpart-2.c: Likewise.
> 
> This looks fine and could be applied straight away I think.
> 
> Thanks,
> Andrew
> 
Committed. Thank you for your review,
Claudiu
diff mbox

Patch

diff --git a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
index 57cb95b..5fd6c36 100644
--- a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
+++ b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c
@@ -7,7 +7,7 @@ 
 static int
 id (int i)
 {
-  asm ("": "+Xr" (i));
+  asm ("": "+r" (i));
   return i;
 }
 
diff --git a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
index 287d96d..6ec4bc5 100644
--- a/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
+++ b/gcc/testsuite/gcc.target/arc/mulsi3_highpart-2.c
@@ -9,7 +9,7 @@ 
 static int
 id (int i)
 {
-  asm ("": "+Xr" (i));
+  asm ("": "+r" (i));
   return i;
 }