Happy new year!
Sorry it took so long.
On Tue, Oct 15, 2024 at 12:49:49PM +0530, jeevitha wrote:
> Hi All,
> 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 } */
This test is not typically needed at all, but yeah, it is possible some
tester decides to sabotage himself and disables VSX on new systems. The
extra check isn't really harmful anyway (it just might suggest to others
that it is required).
Okay for trunk and any backports you want. Thanks!
Segher
