Overnight regression testing revealed a portability problem with several 
recently installed tests.  The tests were observed to fail on a power7 test 
platform.

The tests, which are intended to execute, are compiled with -mcpu=power8.  
Thus, they require power 8 hardware.

I have regression tested this on powerpc64-linux (P7 big-endian, both -m32 and 
-m64), both 32-bit and 64-bit.  Is this ok for trunk and for various backports 
to which the original patch is to be directed?

gcc/testsuite/ChangeLog:

2019-02-01  Kelvin Nilsen  <kel...@gcc.gnu.org>

        * gcc.target/powerpc/vec-extract-slong-1.c: Require p8 execution
        hardware.
        * gcc.target/powerpc/vec-extract-schar-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-sint128-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-sshort-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-ulong-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-uchar-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-sint-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-uint128-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-ushort-1.c: Likewise.
        * gcc.target/powerpc/vec-extract-uint-1.c: Likewise.

Index: gcc/testsuite/gcc.target/powerpc/vec-extract-slong-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-slong-1.c      (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-slong-1.c      (working copy)
@@ -2,7 +2,7 @@
    signed longs remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-schar-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-schar-1.c      (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-schar-1.c      (working copy)
@@ -2,7 +2,7 @@
    signed chars remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-sint128-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-sint128-1.c    (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-sint128-1.c    (working copy)
@@ -2,7 +2,7 @@
    signed __int128s remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-sshort-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-sshort-1.c     (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-sshort-1.c     (working copy)
@@ -2,7 +2,7 @@
    signed shorts remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-ulong-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-ulong-1.c      (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-ulong-1.c      (working copy)
@@ -2,7 +2,7 @@
    unsigned longs remains unsigned.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-uchar-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-uchar-1.c      (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-uchar-1.c      (working copy)
@@ -2,7 +2,7 @@
    unsigned chars remains unsigned.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-sint-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-sint-1.c       (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-sint-1.c       (working copy)
@@ -2,7 +2,7 @@
    signed ints remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-uint128-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-uint128-1.c    (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-uint128-1.c    (working copy)
@@ -2,7 +2,7 @@
    unsigned __int128s remains unsigned.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-ushort-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-ushort-1.c     (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-ushort-1.c     (working copy)
@@ -2,7 +2,7 @@
    signed shorts remains signed.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>
Index: gcc/testsuite/gcc.target/powerpc/vec-extract-uint-1.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/vec-extract-uint-1.c       (revision 
268424)
+++ gcc/testsuite/gcc.target/powerpc/vec-extract-uint-1.c       (working copy)
@@ -2,7 +2,7 @@
    unsigned ints remains unsigned.  */
 /* { dg-do run } */
 /* { dg-options "-ansi -mcpu=power8 " } */
-/* { dg-require-effective-target powerpc_p8vector_ok } */
+/* { dg-require-effective-target p8vector_hw } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 
 #include <altivec.h>

Reply via email to