Ping! please review.
Thanks & Regards Jeevitha On 15/10/24 12:49 pm, jeevitha wrote: > Hi All, > > Removed powerpc*-*-* from the target test as it is always true. Simplified > options by removing -mpower9-misc and -mvsx, which are enabled by default with > -mdejagnu-cpu=power9. The has_arch_pwr9 check is also true with > -mdejagnu-cpu=power9, so it has been removed. > > 2024-10-15 Jeevitha Palanisamy <jeevi...@linux.ibm.com> > > gcc/testsuite/ > > * gcc.target/powerpc/amo1.c: Removed powerpc*-*-* from the target and > simplified dg-options. > * gcc.target/powerpc/amo2.c: Simplified dg-options and added powerpc_vsx > target check. > > > diff --git a/gcc/testsuite/gcc.target/powerpc/amo1.c > b/gcc/testsuite/gcc.target/powerpc/amo1.c > index c5af373b4e9..9a981cd4219 100644 > --- a/gcc/testsuite/gcc.target/powerpc/amo1.c > +++ b/gcc/testsuite/gcc.target/powerpc/amo1.c > @@ -1,6 +1,5 @@ > -/* { dg-do compile { target { powerpc*-*-* && lp64 } } } */ > -/* { dg-options "-mvsx -mpower9-misc -O2" } */ > -/* { dg-additional-options "-mdejagnu-cpu=power9" { target { ! has_arch_pwr9 > } } } */ > +/* { dg-do compile { target { lp64 } } } */ > +/* { dg-options "-mdejagnu-cpu=power9 -O2" } */ > /* { dg-require-effective-target powerpc_vsx } */ > > /* Verify P9 atomic memory operations. */ > diff --git a/gcc/testsuite/gcc.target/powerpc/amo2.c > b/gcc/testsuite/gcc.target/powerpc/amo2.c > index 592f0fb3f92..9e4ff0ce064 100644 > --- a/gcc/testsuite/gcc.target/powerpc/amo2.c > +++ b/gcc/testsuite/gcc.target/powerpc/amo2.c > @@ -1,6 +1,6 @@ > /* { dg-do run { target { powerpc*-*-linux* && { lp64 && p9vector_hw } } } } > */ > -/* { dg-options "-O2 -mvsx -mpower9-misc" } */ > -/* { dg-additional-options "-mdejagnu-cpu=power9" { target { ! has_arch_pwr9 > } } } */ > +/* { dg-options "-mdejagnu-cpu=power9 -O2" } */ > +/* { dg-require-effective-target powerpc_vsx } */ > > #include <amo.h> > #include <stdint.h> > > >