Changes since v1:

- Clarified the commit message to include where the descision is taken
  and why it's a bad idea to use "dg-do run" in a test case.
  Note: This does not only fix it for arm-none-eabi. I see the same
  kind of construct used by for example sparc.

Sorry for the confusion Richard, I hope it's more clear why this is
needed now. :)

Ok for trunk and releases/gcc-14?

--

Using "dg-do run" with a selector overrides the default selector set by
vect.exp that picks between "dg-do run" and "dg-do compile" based on the
target's support for simd operations for Arm targets.
The actual selection of default operation is performed in
check_vect_support_and_set_flags.

gcc/testsuite/ChangeLog:

        * g++.dg/vect/pr84556.cc: Change from "dg-do run" with selector
        to instead use dg-require-effective-target with the same
        selector.

Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
---
 gcc/testsuite/g++.dg/vect/pr84556.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/g++.dg/vect/pr84556.cc 
b/gcc/testsuite/g++.dg/vect/pr84556.cc
index 6b1c9cec515..c7e331628a8 100644
--- a/gcc/testsuite/g++.dg/vect/pr84556.cc
+++ b/gcc/testsuite/g++.dg/vect/pr84556.cc
@@ -1,5 +1,5 @@
 // PR c++/84556
-// { dg-do run { target c++11 } }
+// { dg-require-effective-target c++11 }
 // { dg-additional-options "-O2 -fopenmp-simd" }
 // { dg-additional-options "-mavx" { target avx_runtime } }
 
-- 
2.25.1

Reply via email to