public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] rs6000: Builtins test changes for pragma_misc9.c
@ 2021-11-18 16:18 Bill Schmidt
  2021-12-01 16:36 ` [PATCH, PING] " Bill Schmidt
  2021-12-01 22:32 ` [PATCH] " Segher Boessenkool
  0 siblings, 2 replies; 3+ messages in thread
From: Bill Schmidt @ 2021-11-18 16:18 UTC (permalink / raw)
  To: GCC Patches; +Cc: Segher Boessenkool, David Edelsohn

Hi!  This patch is broken out from the test suite patch for the new
builtins support.  This one is just a minor adjustment for the error
message wording.

Tested on powerpc64le-linux-gnu and powerpc64-linux-gnu (-m32/-m64)
with no regressions.  Is this okay for trunk?

Thanks!
Bill


2021-11-17  Bill Schmidt  <wschmidt@linux.ibm.com>

gcc/testsuite/
	* gcc.target/powerpc/pragma_misc9.c: Adjust error message.
---
 gcc/testsuite/gcc.target/powerpc/pragma_misc9.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c b/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
index e03099bd084..c1667d9f7db 100644
--- a/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
+++ b/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
@@ -20,7 +20,7 @@ vector bool int
 test2 (vector signed int a, vector signed int b)
 {
   return vec_cmpnez (a, b);
-  /* { dg-error "'__builtin_altivec_vcmpnezw' requires the '-mcpu=power9' option" "" { target *-*-* } .-1 } */
+  /* { dg-error "'__builtin_altivec_vcmpnezw' requires the '-mcpu=power9' and '-mvsx' options" "" { target *-*-* } .-1 } */
 }
 
 #pragma GCC target ("cpu=power7")
@@ -28,7 +28,7 @@ vector signed int
 test3 (vector signed int a, vector signed int b)
 {
   return vec_mergee (a, b);
-  /* { dg-error "'__builtin_altivec_vmrgew_v4si' requires the '-mpower8-vector' option" "" { target *-*-* } .-1 } */
+  /* { dg-error "'__builtin_altivec_vmrgew_v4si' requires the '-mcpu=power8' and '-mvsx' options" "" { target *-*-* } .-1 } */
 }
 
 #pragma GCC target ("cpu=power6")
-- 
2.27.0



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH, PING] rs6000: Builtins test changes for pragma_misc9.c
  2021-11-18 16:18 [PATCH] rs6000: Builtins test changes for pragma_misc9.c Bill Schmidt
@ 2021-12-01 16:36 ` Bill Schmidt
  2021-12-01 22:32 ` [PATCH] " Segher Boessenkool
  1 sibling, 0 replies; 3+ messages in thread
From: Bill Schmidt @ 2021-12-01 16:36 UTC (permalink / raw)
  To: GCC Patches; +Cc: Segher Boessenkool, David Edelsohn

Hi!  I'd like to ping this patch.

Thanks!
Bill

On 11/18/21 10:18 AM, Bill Schmidt wrote:
> Hi!  This patch is broken out from the test suite patch for the new
> builtins support.  This one is just a minor adjustment for the error
> message wording.
>
> Tested on powerpc64le-linux-gnu and powerpc64-linux-gnu (-m32/-m64)
> with no regressions.  Is this okay for trunk?
>
> Thanks!
> Bill
>
>
> 2021-11-17  Bill Schmidt  <wschmidt@linux.ibm.com>
>
> gcc/testsuite/
> 	* gcc.target/powerpc/pragma_misc9.c: Adjust error message.
> ---
>  gcc/testsuite/gcc.target/powerpc/pragma_misc9.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c b/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
> index e03099bd084..c1667d9f7db 100644
> --- a/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
> +++ b/gcc/testsuite/gcc.target/powerpc/pragma_misc9.c
> @@ -20,7 +20,7 @@ vector bool int
>  test2 (vector signed int a, vector signed int b)
>  {
>    return vec_cmpnez (a, b);
> -  /* { dg-error "'__builtin_altivec_vcmpnezw' requires the '-mcpu=power9' option" "" { target *-*-* } .-1 } */
> +  /* { dg-error "'__builtin_altivec_vcmpnezw' requires the '-mcpu=power9' and '-mvsx' options" "" { target *-*-* } .-1 } */
>  }
>  
>  #pragma GCC target ("cpu=power7")
> @@ -28,7 +28,7 @@ vector signed int
>  test3 (vector signed int a, vector signed int b)
>  {
>    return vec_mergee (a, b);
> -  /* { dg-error "'__builtin_altivec_vmrgew_v4si' requires the '-mpower8-vector' option" "" { target *-*-* } .-1 } */
> +  /* { dg-error "'__builtin_altivec_vmrgew_v4si' requires the '-mcpu=power8' and '-mvsx' options" "" { target *-*-* } .-1 } */
>  }
>  
>  #pragma GCC target ("cpu=power6")

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] rs6000: Builtins test changes for pragma_misc9.c
  2021-11-18 16:18 [PATCH] rs6000: Builtins test changes for pragma_misc9.c Bill Schmidt
  2021-12-01 16:36 ` [PATCH, PING] " Bill Schmidt
@ 2021-12-01 22:32 ` Segher Boessenkool
  1 sibling, 0 replies; 3+ messages in thread
From: Segher Boessenkool @ 2021-12-01 22:32 UTC (permalink / raw)
  To: Bill Schmidt; +Cc: GCC Patches, David Edelsohn

On Thu, Nov 18, 2021 at 10:18:06AM -0600, Bill Schmidt wrote:
> Hi!  This patch is broken out from the test suite patch for the new
> builtins support.  This one is just a minor adjustment for the error
> message wording.
> 
> Tested on powerpc64le-linux-gnu and powerpc64-linux-gnu (-m32/-m64)
> with no regressions.  Is this okay for trunk?

Okay.  Thanks!


Segher

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-12-01 22:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-18 16:18 [PATCH] rs6000: Builtins test changes for pragma_misc9.c Bill Schmidt
2021-12-01 16:36 ` [PATCH, PING] " Bill Schmidt
2021-12-01 22:32 ` [PATCH] " Segher Boessenkool

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).