Message ID | 263e2951-de3c-4ae7-aab0-ed3ccd313385@linux.ibm.com |
---|---|
State | New |
Headers | show |
Series | [version,2] rs6000, altivec-1-runnable.c update the, require-effective-target | expand |
Hi, on 2024/6/22 00:15, Carl Love wrote: > GCC maintainers: > > version 2, update the dg options per the feedback. Retested the patch on Power 10 with no regressions. > > This patch updates the dg options. > > The patch has been tested on Power 10 with no regression failures. > > Please let me know if this patch is acceptable for mainline. Thanks. > > Carl > > ---------------------------------------------------------------------- > rs6000, altivec-1-runnable.c update the require-effective-target > > Update the dg test directives. > > gcc/testsuite/ChangeLog:gcc/testsuite/ChangeLog: > * gcc.target/powerpc/altivec-1-runnable.c: Change the > require-effective-target for the test. > --- > gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c > index da8ebbc30ba..3f084c91798 100644 > --- a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c > +++ b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c > @@ -1,6 +1,7 @@ > -/* { dg-do compile { target powerpc*-*-* } } */ > -/* { dg-require-effective-target powerpc_altivec_ok } */ > -/* { dg-options "-maltivec" } */ > +/* { dg-do run { target vmx_hw } } */ > +/* { dg-do compile { target { ! vmx_hw } } } */ > +/* { dg-options "-O2 -maltivec" } */ > +/* { dg-require-effective-target powerpc_altivec } */ This one needs rebasing, "powerpc_altivec" has been adjusted on trunk. BR, Kewen > > #include <altivec.h> >
Kewen: On 6/23/24 19:41, Kewen.Lin wrote: > Hi, > > on 2024/6/22 00:15, Carl Love wrote: >> GCC maintainers: >> >> version 2, update the dg options per the feedback. Retested the patch on Power 10 with no regressions. >> >> This patch updates the dg options. >> >> The patch has been tested on Power 10 with no regression failures. >> >> Please let me know if this patch is acceptable for mainline. Thanks. >> >> Carl >> >> ---------------------------------------------------------------------- >> rs6000, altivec-1-runnable.c update the require-effective-target >> >> Update the dg test directives. >> >> gcc/testsuite/ChangeLog:gcc/testsuite/ChangeLog: >> * gcc.target/powerpc/altivec-1-runnable.c: Change the >> require-effective-target for the test. >> --- >> gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c | 7 ++++--- >> 1 file changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c >> index da8ebbc30ba..3f084c91798 100644 >> --- a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c >> +++ b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c >> @@ -1,6 +1,7 @@ >> -/* { dg-do compile { target powerpc*-*-* } } */ >> -/* { dg-require-effective-target powerpc_altivec_ok } */ >> -/* { dg-options "-maltivec" } */ >> +/* { dg-do run { target vmx_hw } } */ >> +/* { dg-do compile { target { ! vmx_hw } } } */ >> +/* { dg-options "-O2 -maltivec" } */ >> +/* { dg-require-effective-target powerpc_altivec } */ > > This one needs rebasing, "powerpc_altivec" has been adjusted on trunk. Yes, this seems to be out of sync. I will rebase on the current upstream tree and re-post. Carl
diff --git a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c index da8ebbc30ba..3f084c91798 100644 --- a/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c +++ b/gcc/testsuite/gcc.target/powerpc/altivec-1-runnable.c @@ -1,6 +1,7 @@ -/* { dg-do compile { target powerpc*-*-* } } */ -/* { dg-require-effective-target powerpc_altivec_ok } */ -/* { dg-options "-maltivec" } */ +/* { dg-do run { target vmx_hw } } */ +/* { dg-do compile { target { ! vmx_hw } } } */ +/* { dg-options "-O2 -maltivec" } */ +/* { dg-require-effective-target powerpc_altivec } */ #include <altivec.h>