| Message ID | 2ba31ed8-7403-5f23-dbbb-c8d4924b1872@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | rs6000: Builtins test changes for pragma_misc9.c | expand |
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")
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
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")