This patch is a followup to PR testsuite/83889, in which it was noted that

"The failures in this PR were from forcing { dg-do run } even when
vect.exp chooses options that are incompatible with the runtime.
The default vect.exp behaviour is to execute when possible, so there's
no need for a dg-do at all."

Since the original patch for this PR was committed, a bunch of new testcases have been added that also incorrectly specify "dg-do run", leading to bogus execution failures (in the case I tripped over, trying to run code built for ARM Neon on a target that doesn't support those instructions). I've applied the same trivial fix as in the original patch. I thought this was obvious enough that I went ahead and committed it.

-Sandra
2019-09-11  Sandra Loosemore  <san...@codesourcery.com>

	PR testsuite/83889

	gcc/testsuite/
	* gcc.dg/vect/pr81740-2.c: Remove explicit dg-do run.
	* gcc.dg/vect/pr88598-1.c: Likewise.
	* gcc.dg/vect/pr88598-2.c: Likewise.
	* gcc.dg/vect/pr88598-3.c: Likewise.
	* gcc.dg/vect/pr88598-4.c: Likewise.
	* gcc.dg/vect/pr88598-5.c: Likewise.
	* gcc.dg/vect/pr88598-6.c: Likewise.
	* gcc.dg/vect/pr89440.c: Likewise.
	* gcc.dg/vect/pr90018.c: Likewise.
	* gcc.dg/vect/pr91293-1.c: Likewise.
	* gcc.dg/vect/pr91293-2.c: Likewise.
	* gcc.dg/vect/pr91293-3.c: Likewise.
Index: gcc/testsuite/gcc.dg/vect/pr81740-1.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr81740-1.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr81740-1.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-require-effective-target vect_int } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr81740-2.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr81740-2.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr81740-2.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-require-effective-target vect_int } */
 /* { dg-require-effective-target vect_hw_misalign } */
 
Index: gcc/testsuite/gcc.dg/vect/pr88598-1.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-1.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-1.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr88598-2.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-2.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-2.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr88598-3.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-3.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-3.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-ffast-math -fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr88598-4.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-4.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-4.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr88598-5.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-5.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-5.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr88598-6.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr88598-6.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr88598-6.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-ffast-math -fdump-tree-optimized" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr89440.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr89440.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr89440.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-ffast-math" } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr90018.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr90018.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr90018.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-require-effective-target vect_double } */
 
 #include "tree-vect.h"
Index: gcc/testsuite/gcc.dg/vect/pr91293-1.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr91293-1.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr91293-1.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-msse4.1" { target { sse4_runtime } } } */
 
 long long a;
Index: gcc/testsuite/gcc.dg/vect/pr91293-2.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr91293-2.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr91293-2.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-msse4.1" { target { sse4_runtime } } } */
 
 long long a;
Index: gcc/testsuite/gcc.dg/vect/pr91293-3.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/pr91293-3.c	(revision 275664)
+++ gcc/testsuite/gcc.dg/vect/pr91293-3.c	(working copy)
@@ -1,4 +1,3 @@
-/* { dg-do run } */
 /* { dg-additional-options "-msse4.1" { target { sse4_runtime } } } */
 
 long long a;

Reply via email to