On Thu, May 12, 2022 at 05:30:16PM -0500, Segher Boessenkool wrote: > On Mon, Apr 18, 2022 at 12:15:35PM -0500, Paul A. Clarke wrote: > > A few tests need not be restricted to 'lp64', so remove the restriction. > > > > A few of those need a simple change to the DejaGnu directives to suppress > > '-mcmodel' flags for '-m32'. > > Okay for trunk. Thanks!
I noticed that removing "{ target lp64 }" just left "{ dg-do compile }", which is superfluous, so I removed the whole line before committing. Since the actual commit was different than what was posted, I'm sending the final, committed patch to the list, below. PC rs6000: Remove a few needless 'lp64' contraints. A few tests need not be restricted to 'lp64', so remove the restriction. A few of those need a simple change to the DejaGnu directives to suppress '-mcmodel' flags for '-m32'. 2022-05-13 Paul A. Clarke <p...@us.ibm.com> gcc/testsuite * g++.target/powerpc/pr65240-1.C: Adjust DejaGnu directives. * g++.target/powerpc/pr65240-2.C: Likewise. * g++.target/powerpc/pr65240-3.C: Likewise. * g++.target/powerpc/pr65240-4.C: Likewise. * g++.target/powerpc/pr65242.C: Likewise. * g++.target/powerpc/pr67211.C: Likewise. * g++.target/powerpc/pr69667.C: Likewise. * g++.target/powerpc/pr71294.C: Likewise. --- gcc/testsuite/g++.target/powerpc/pr65240-1.C | 4 ++-- gcc/testsuite/g++.target/powerpc/pr65240-2.C | 4 ++-- gcc/testsuite/g++.target/powerpc/pr65240-3.C | 4 ++-- gcc/testsuite/g++.target/powerpc/pr65240-4.C | 1 - gcc/testsuite/g++.target/powerpc/pr65242.C | 1 - gcc/testsuite/g++.target/powerpc/pr67211.C | 1 - gcc/testsuite/g++.target/powerpc/pr69667.C | 1 - gcc/testsuite/g++.target/powerpc/pr71294.C | 1 - 8 files changed, 6 insertions(+), 11 deletions(-) diff --git a/gcc/testsuite/g++.target/powerpc/pr65240-1.C b/gcc/testsuite/g++.target/powerpc/pr65240-1.C index f735a1f7834a..1cf158c69097 100644 --- a/gcc/testsuite/g++.target/powerpc/pr65240-1.C +++ b/gcc/testsuite/g++.target/powerpc/pr65240-1.C @@ -1,7 +1,7 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ -/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -mcmodel=small -mno-fp-in-toc -Wno-return-type" } */ +/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -mno-fp-in-toc -Wno-return-type" } */ +/* { dg-additional-options "-mcmodel=small" { target lp64 } } */ /* target/65240, compiler got a 'insn does not satisfy its constraints' error. */ diff --git a/gcc/testsuite/g++.target/powerpc/pr65240-2.C b/gcc/testsuite/g++.target/powerpc/pr65240-2.C index e201e3a74d71..32d1c799b0db 100644 --- a/gcc/testsuite/g++.target/powerpc/pr65240-2.C +++ b/gcc/testsuite/g++.target/powerpc/pr65240-2.C @@ -1,7 +1,7 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ -/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -mcmodel=small -mfp-in-toc -Wno-return-type" } */ +/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -mfp-in-toc -Wno-return-type" } */ +/* { dg-additional-options "-mcmodel=small" { target lp64 } } */ /* target/65240, compiler got a 'insn does not satisfy its constraints' error. */ diff --git a/gcc/testsuite/g++.target/powerpc/pr65240-3.C b/gcc/testsuite/g++.target/powerpc/pr65240-3.C index 0821f68a5cf9..02567647f304 100644 --- a/gcc/testsuite/g++.target/powerpc/pr65240-3.C +++ b/gcc/testsuite/g++.target/powerpc/pr65240-3.C @@ -1,7 +1,7 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ -/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -mcmodel=medium -Wno-return-type" } */ +/* { dg-options "-mdejagnu-cpu=power8 -O3 -ffast-math -Wno-return-type" } */ +/* { dg-additional-options "-mcmodel=medium" { target lp64 } } */ /* target/65240, compiler got a 'insn does not satisfy its constraints' error. */ diff --git a/gcc/testsuite/g++.target/powerpc/pr65240-4.C b/gcc/testsuite/g++.target/powerpc/pr65240-4.C index 92d31acb20d9..3f6993aa1cde 100644 --- a/gcc/testsuite/g++.target/powerpc/pr65240-4.C +++ b/gcc/testsuite/g++.target/powerpc/pr65240-4.C @@ -1,4 +1,3 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_vsx_ok } */ /* { dg-options "-mdejagnu-cpu=power7 -O3 -ffast-math -Wno-return-type" } */ diff --git a/gcc/testsuite/g++.target/powerpc/pr65242.C b/gcc/testsuite/g++.target/powerpc/pr65242.C index b2984d1d6083..3f5c2eaa9099 100644 --- a/gcc/testsuite/g++.target/powerpc/pr65242.C +++ b/gcc/testsuite/g++.target/powerpc/pr65242.C @@ -1,4 +1,3 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ /* { dg-options "-mdejagnu-cpu=power8 -O3" } */ diff --git a/gcc/testsuite/g++.target/powerpc/pr67211.C b/gcc/testsuite/g++.target/powerpc/pr67211.C index b58c08234272..7d5dd42765fe 100644 --- a/gcc/testsuite/g++.target/powerpc/pr67211.C +++ b/gcc/testsuite/g++.target/powerpc/pr67211.C @@ -1,4 +1,3 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ /* { dg-options "-mdejagnu-cpu=power7 -mdejagnu-tune=power8 -O3 -w" } */ diff --git a/gcc/testsuite/g++.target/powerpc/pr69667.C b/gcc/testsuite/g++.target/powerpc/pr69667.C index 6a12a3363e98..da550cd14bd6 100644 --- a/gcc/testsuite/g++.target/powerpc/pr69667.C +++ b/gcc/testsuite/g++.target/powerpc/pr69667.C @@ -1,4 +1,3 @@ -/* { dg-do compile { target lp64 } } */ /* { dg-skip-if "" { *-*-darwin* } } */ /* { dg-require-effective-target powerpc_p8vector_ok } */ /* { dg-options "-mdejagnu-cpu=power8 -w -std=c++14" } */ diff --git a/gcc/testsuite/g++.target/powerpc/pr71294.C b/gcc/testsuite/g++.target/powerpc/pr71294.C index 1da1e327a285..7f12c8d5c90a 100644 --- a/gcc/testsuite/g++.target/powerpc/pr71294.C +++ b/gcc/testsuite/g++.target/powerpc/pr71294.C @@ -1,4 +1,3 @@ -// { dg-do compile { target lp64 } } // { dg-require-effective-target powerpc_p8vector_ok } */ // { dg-options "-mdejagnu-cpu=power8 -O3 -fstack-protector" } -- 2.27.0