[gcc r16-8316] testsuite: negated triplets need blank separators

Alexandre Oliva aoliva@gcc.gnu.org
Sat Mar 28 03:28:04 GMT 2026


https://gcc.gnu.org/g:630b53cd4ff1c3ab16c05fb84e8ce49cf05e2373

commit r16-8316-g630b53cd4ff1c3ab16c05fb84e8ce49cf05e2373
Author: Alexandre Oliva <oliva@adacore.com>
Date:   Wed Mar 25 15:05:14 2026 -0300

    testsuite: negated triplets need blank separators
    
    !a-b-c is not the same as ! a-b-c.  In !a-b-c, the '!' is part of the
    triplet that istarget attempts to match.  Fix all occurrences, except
    for gcc.dg/vect/vect-93.c, that is taken care of separately.
    
    
    for  gcc/testsuite/ChangeLog
    
            * gcc.dg/debug/btf/btf-variables-5.c: Separate ! from triplet.
            * gcc.dg/hoist-register-pressure-1.c: Likewise.
            * gcc.dg/hoist-register-pressure-2.c: Likewise.
            * gcc.dg/hoist-register-pressure-3.c: Likewise.
            * gcc.target/powerpc/safe-indirect-jump-1.c: Likewise.
            * gfortran.dg/gomp/declare-variant-14.f90: Likewise.

Diff:
---
 gcc/testsuite/gcc.dg/debug/btf/btf-variables-5.c        | 2 +-
 gcc/testsuite/gcc.dg/hoist-register-pressure-1.c        | 2 +-
 gcc/testsuite/gcc.dg/hoist-register-pressure-2.c        | 2 +-
 gcc/testsuite/gcc.dg/hoist-register-pressure-3.c        | 2 +-
 gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-1.c | 4 ++--
 gcc/testsuite/gfortran.dg/gomp/declare-variant-14.f90   | 2 +-
 6 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/gcc/testsuite/gcc.dg/debug/btf/btf-variables-5.c b/gcc/testsuite/gcc.dg/debug/btf/btf-variables-5.c
index a08130cfc072..1d8d7778d273 100644
--- a/gcc/testsuite/gcc.dg/debug/btf/btf-variables-5.c
+++ b/gcc/testsuite/gcc.dg/debug/btf/btf-variables-5.c
@@ -14,7 +14,7 @@
 /* Expect 2 array types, one of which is unsized.  For BPF target, -gprune-btf
    is the default and will remove the unsized array type.  */
 /* { dg-final { scan-assembler-times "\[\t \]0x4\[\t \]+\[^\n\]*bta_nelems" 1 } } */
-/* { dg-final { scan-assembler-times "\[\t \]0\[\t \]+\[^\n\]*bta_nelems" 1 { target { !bpf-*-* } } } } */
+/* { dg-final { scan-assembler-times "\[\t \]0\[\t \]+\[^\n\]*bta_nelems" 1 { target { ! bpf-*-* } } } } */
 /* { dg-final { scan-assembler-times "\[\t \]0\[\t \]+\[^\n\]*bta_nelems" 0 { target { bpf-*-* } } } } */
 
 extern const char FOO[];
diff --git a/gcc/testsuite/gcc.dg/hoist-register-pressure-1.c b/gcc/testsuite/gcc.dg/hoist-register-pressure-1.c
index 6e566da4dba4..516357d3c920 100644
--- a/gcc/testsuite/gcc.dg/hoist-register-pressure-1.c
+++ b/gcc/testsuite/gcc.dg/hoist-register-pressure-1.c
@@ -3,7 +3,7 @@
    be assigned without clobbering cc.  For a PLUS rtx on S/390 this
    requires a load address instruction which is fine on 64 bit but
    cannot be used on 31 bit since it does a 31 bit add only.  */
-/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { { !s390*-*-* || lp64 } && nonpic } } } } */
+/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { { { ! s390*-*-* } || lp64 } && nonpic } } } } */
 
 #define BUF 100
 long a[BUF];
diff --git a/gcc/testsuite/gcc.dg/hoist-register-pressure-2.c b/gcc/testsuite/gcc.dg/hoist-register-pressure-2.c
index 05ffd8d93f0a..2e1283233788 100644
--- a/gcc/testsuite/gcc.dg/hoist-register-pressure-2.c
+++ b/gcc/testsuite/gcc.dg/hoist-register-pressure-2.c
@@ -3,7 +3,7 @@
    be assigned without clobbering cc.  For a PLUS rtx on S/390 this
    requires a load address instruction which is fine on 64 bit but
    cannot be used on 31 bit since it does a 31 bit add only.  */
-/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { !s390*-*-* || lp64 } } } } */
+/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { { ! s390*-*-* } || lp64 } } } } */
 
 #define BUF 100
 long a[BUF];
diff --git a/gcc/testsuite/gcc.dg/hoist-register-pressure-3.c b/gcc/testsuite/gcc.dg/hoist-register-pressure-3.c
index f274fa133454..0a705edd79cf 100644
--- a/gcc/testsuite/gcc.dg/hoist-register-pressure-3.c
+++ b/gcc/testsuite/gcc.dg/hoist-register-pressure-3.c
@@ -3,7 +3,7 @@
    be assigned without clobbering cc.  For a PLUS rtx on S/390 this
    requires a load address instruction which is fine on 64 bit but
    cannot be used on 31 bit since it does a 31 bit add only.  */
-/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { !s390*-*-* || lp64 } } } } */
+/* { dg-final { scan-rtl-dump "PRE/HOIST: end of bb .* copying expression" "hoist" { target { { ! s390*-*-* } || lp64 } } } } */
 
 #define BUF 100
 long a[BUF];
diff --git a/gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-1.c b/gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-1.c
index b9ad8c1dd36a..adecb3cd6723 100644
--- a/gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-1.c
+++ b/gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-1.c
@@ -18,5 +18,5 @@ int bar ()
 /* { dg-final { scan-assembler "beqctrl-" { target { lp64 || { powerpc*-*-aix* } } } } } */
 
 /* The other ABIs do allow a sibcall.  */
-/* { dg-final { scan-assembler "beqctr-" { target { ilp32 && !powerpc*-*-aix* } } } } */
-/* { dg-final { scan-assembler {b \$} { target { ilp32 && !powerpc*-*-aix* } } } } */
+/* { dg-final { scan-assembler "beqctr-" { target { ilp32 && { ! powerpc*-*-aix* } } } } } */
+/* { dg-final { scan-assembler {b \$} { target { ilp32 && { ! powerpc*-*-aix* } } } } } */
diff --git a/gcc/testsuite/gfortran.dg/gomp/declare-variant-14.f90 b/gcc/testsuite/gfortran.dg/gomp/declare-variant-14.f90
index 71b66cb5d086..0ac172d571c7 100644
--- a/gcc/testsuite/gfortran.dg/gomp/declare-variant-14.f90
+++ b/gcc/testsuite/gfortran.dg/gomp/declare-variant-14.f90
@@ -36,7 +36,7 @@ contains
     integer :: a, b
 
     ! At gimplification time, we can't decide yet which function to call.
-    ! { dg-final { scan-tree-dump-times "f04 \\\(x" 2 "gimple" { target { !aarch64*-*-* } } } }
+    ! { dg-final { scan-tree-dump-times "f04 \\\(x" 2 "gimple" { target { ! aarch64*-*-* } } } }
     ! After simd clones are created, the original non-clone test1 shall
     ! call f03 (score 6), the sse2/avx/avx2 clones too, but avx512f clones
     ! shall call f01 with score 8.


More information about the Gcc-cvs mailing list