[0/3] rs6000: Move g++.dg powerpc tests to g++.target

Message ID 20220221211747.53681-1-pc@us.ibm.com
Headers
Series rs6000: Move g++.dg powerpc tests to g++.target |

Message

Paul A. Clarke Feb. 21, 2022, 9:17 p.m. UTC
  Some tests in g++.dg are target-specific for powerpc. Move those to
g++.target/powerpc. Update the DejaGnu directives as needed, since
the target restriction is perhaps no longer needed when residing in the
target-specific powerpc subdirectory.

Tested with Linux on Power9, full "make check".

OK for trunk?

Paul A. Clarke (3):
  rs6000: Move g++.dg/ext powerpc tests to g++.target
  rs6000: Move g++.dg powerpc PR tests to g++.target
  rs6000: Move more g++.dg powerpc tests to g++.target

 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-1.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-10.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-11.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-12.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-13.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-14.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-15.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-16.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-17.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-18.C | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-2.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-3.C  | 4 ++--
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-4.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-5.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-6.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-7.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-8.C  | 2 +-
 gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-9.C  | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-cell-1.C       | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-cell-2.C       | 4 ++--
 .../{g++.dg/ext => g++.target/powerpc}/altivec-cell-3.C       | 4 ++--
 .../{g++.dg/ext => g++.target/powerpc}/altivec-cell-4.C       | 4 ++--
 .../{g++.dg/ext => g++.target/powerpc}/altivec-cell-5.C       | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-types-1.C      | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-types-2.C      | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-types-3.C      | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/altivec-types-4.C      | 2 +-
 .../{g++.dg/debug/dwarf2 => g++.target/powerpc}/const2.C      | 0
 .../other => g++.target/powerpc}/darwin-minversion-1.C        | 0
 .../{g++.dg/eh => g++.target/powerpc}/ppc64-sighandle-cr.C    | 0
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-1.C      | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-2.C      | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-3.C      | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-4.C      | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240.h        | 0
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65242.C        | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr67211.C        | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr69667.C        | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr71294.C        | 2 +-
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr84264.C        | 2 +-
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr84279.C        | 4 ++--
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr85657.C        | 2 +-
 gcc/testsuite/{g++.dg => g++.target/powerpc}/pr93974.C        | 0
 gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/simd-4.C      | 2 +-
 gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/simd-5.C      | 0
 .../{g++.dg/other => g++.target/powerpc}/spu2vmx-1.C          | 2 +-
 gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/uncaught3.C   | 2 +-
 .../{g++.dg/ext => g++.target/powerpc}/undef-bool-1.C         | 2 +-
 48 files changed, 54 insertions(+), 54 deletions(-)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-1.C (83%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-10.C (92%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-11.C (80%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-12.C (87%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-13.C (97%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-14.C (86%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-15.C (92%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-16.C (88%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-17.C (91%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-18.C (83%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-2.C (92%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-3.C (96%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-4.C (81%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-5.C (83%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-6.C (94%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-7.C (96%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-8.C (93%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-9.C (86%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-cell-1.C (96%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-cell-2.C (94%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-cell-3.C (85%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-cell-4.C (88%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-cell-5.C (96%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-types-1.C (98%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-types-2.C (92%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-types-3.C (92%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/altivec-types-4.C (89%)
 rename gcc/testsuite/{g++.dg/debug/dwarf2 => g++.target/powerpc}/const2.C (100%)
 rename gcc/testsuite/{g++.dg/other => g++.target/powerpc}/darwin-minversion-1.C (100%)
 rename gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/ppc64-sighandle-cr.C (100%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-1.C (76%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-2.C (76%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-3.C (76%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240-4.C (75%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65240.h (100%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr65242.C (94%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr67211.C (92%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr69667.C (97%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr71294.C (96%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr84264.C (79%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr84279.C (91%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr85657.C (90%)
 rename gcc/testsuite/{g++.dg => g++.target/powerpc}/pr93974.C (100%)
 rename gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/simd-4.C (95%)
 rename gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/simd-5.C (100%)
 rename gcc/testsuite/{g++.dg/other => g++.target/powerpc}/spu2vmx-1.C (84%)
 rename gcc/testsuite/{g++.dg/eh => g++.target/powerpc}/uncaught3.C (96%)
 rename gcc/testsuite/{g++.dg/ext => g++.target/powerpc}/undef-bool-1.C (85%)
  

Comments

Segher Boessenkool Feb. 22, 2022, 6:28 p.m. UTC | #1
On Mon, Feb 21, 2022 at 03:17:44PM -0600, Paul A. Clarke wrote:
> Some tests in g++.dg are target-specific for powerpc. Move those to
> g++.target/powerpc. Update the DejaGnu directives as needed, since
> the target restriction is perhaps no longer needed when residing in the
> target-specific powerpc subdirectory.

Not "perhaps" :-)  More specifically, powerpc.exp has

# Exit immediately if this isn't a PowerPC target.
if {![istarget powerpc*-*-*] } then {
  return
}

so anything run from that driver does not have to test for powerpc
separately anymore.


Segher
  
Paul A. Clarke Feb. 22, 2022, 7:21 p.m. UTC | #2
On Tue, Feb 22, 2022 at 12:28:56PM -0600, Segher Boessenkool wrote:
> On Mon, Feb 21, 2022 at 03:17:44PM -0600, Paul A. Clarke wrote:
> > Some tests in g++.dg are target-specific for powerpc. Move those to
> > g++.target/powerpc. Update the DejaGnu directives as needed, since
> > the target restriction is perhaps no longer needed when residing in the
> > target-specific powerpc subdirectory.
> 
> Not "perhaps" :-)  More specifically, powerpc.exp has
> 
> # Exit immediately if this isn't a PowerPC target.
> if {![istarget powerpc*-*-*] } then {
>   return
> }
> 
> so anything run from that driver does not have to test for powerpc
> separately anymore.

The context for "perhaps" is for cases like:
// { dg-do compile { target powerpc*-*-darwin* } }
and
// { dg-do compile { target { powerpc*-*-linux* } } }

where the target is still needed, albeit without the "powerpc"
restriction itself.

PC