Re: [PATCH,rs6000] Correct dejagnu directives in several newly added tests

[Segher Boessenkool]

Hi Kelvin,

>   * gcc.target/powerpc/swaps-p8-30.c: Exchange the order of dg-do
>   and dg-require-effective-target directives to correct testing
>   behavior.
>   * gcc.target/powerpc/swaps-p8-32.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-41.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-34.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-43.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-36.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-45.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-29.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-38.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-31.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-40.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-33.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-42.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-35.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-44.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-28.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-37.c: Likewise.
>   * gcc.target/powerpc/swaps-p8-39.c: Likewise.

In cases like this, please sort the entries :-)

(I've fixed it, nothing more to do).

Thanks for the patch,


Segher

[PATCH,rs6000] Correct dejagnu directives in several newly added tests

[Kelvin Nilsen]

This patch corrects an error in several newly added test programs that
was causing these programs to be SUPPORTED on platforms where they were
not supposed to be SUPPORTED, which was causing unexpected FAILS.

The patch has been preapproved by seg...@gcc.gnu.org.

gcc/testsuite/ChangeLog:

2017-09-29  Kelvin Nilsen  

* gcc.target/powerpc/swaps-p8-30.c: Exchange the order of dg-do
and dg-require-effective-target directives to correct testing
behavior.
* gcc.target/powerpc/swaps-p8-32.c: Likewise.
* gcc.target/powerpc/swaps-p8-41.c: Likewise.
* gcc.target/powerpc/swaps-p8-34.c: Likewise.
* gcc.target/powerpc/swaps-p8-43.c: Likewise.
* gcc.target/powerpc/swaps-p8-36.c: Likewise.
* gcc.target/powerpc/swaps-p8-45.c: Likewise.
* gcc.target/powerpc/swaps-p8-29.c: Likewise.
* gcc.target/powerpc/swaps-p8-38.c: Likewise.
* gcc.target/powerpc/swaps-p8-31.c: Likewise.
* gcc.target/powerpc/swaps-p8-40.c: Likewise.
* gcc.target/powerpc/swaps-p8-33.c: Likewise.
* gcc.target/powerpc/swaps-p8-42.c: Likewise.
* gcc.target/powerpc/swaps-p8-35.c: Likewise.
* gcc.target/powerpc/swaps-p8-44.c: Likewise.
* gcc.target/powerpc/swaps-p8-28.c: Likewise.
* gcc.target/powerpc/swaps-p8-37.c: Likewise.
* gcc.target/powerpc/swaps-p8-39.c: Likewise.


Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-30.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-30.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-30.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do compile { target { powerpc64le-*-* } } } */
 /* { dg-require-effective-target powerpc_p8vector_ok } */
-/* { dg-do compile { target { powerpc64le-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 /* { dg-final { scan-assembler-not "xxpermdi" } } */
Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-32.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-32.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-32.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-require-effective-target p8vector_hw } */
-/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 
Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-41.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-41.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-41.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-require-effective-target p8vector_hw } */
-/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 
Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-34.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-34.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-34.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-require-effective-target p8vector_hw } */
-/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 
Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-43.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-43.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-43.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-require-effective-target p8vector_hw } */
-/* { dg-do run { target { powerpc*-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 
Index: gcc/testsuite/gcc.target/powerpc/swaps-p8-36.c
===
--- gcc/testsuite/gcc.target/powerpc/swaps-p8-36.c  (revision 253294)
+++ gcc/testsuite/gcc.target/powerpc/swaps-p8-36.c  (working copy)
@@ -1,5 +1,5 @@
+/* { dg-do compile { target { powerpc64le-*-* } } } */
 /* { dg-require-effective-target powerpc_p8vector_ok } */
-/* { dg-do compile { target { powerpc64le-*-* } } } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power8" } } */
 /* { dg-options "-mcpu=power8 -O3 " } */
 /* { dg-final { scan-assembler-not