Re: [PATCH] AArch64: Fix test for vector length safety

[Tejas Belagod]

On 31/07/2025 15:20, Kyrylo Tkachov wrote:

On 31 Jul 2025, at 14:34, Tejas Belagod <tejas.bela...@arm.com> wrote:

The test was unsafe when tested on different vector lengths.  This patch
fixes it to work on all lengths.

Ok. I’ve seen this test fail on GCC 15 branch too, do we want this fix there as 
well?

Thanks. Yes, I'll apply to 15 if ok.

Tejas.

Thanks,
Kyrill

gcc/testsuite/ChangeLog

* gcc.target/aarch64/sve/acle/general/cops.c: Fix test.
---
gcc/testsuite/gcc.target/aarch64/sve/acle/general/cops.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gcc/testsuite/gcc.target/aarch64/sve/acle/general/cops.c 
b/gcc/testsuite/gcc.target/aarch64/sve/acle/general/cops.c
index f0dc9a9b21c..1201ca089f3 100644
--- a/gcc/testsuite/gcc.target/aarch64/sve/acle/general/cops.c
+++ b/gcc/testsuite/gcc.target/aarch64/sve/acle/general/cops.c
@@ -212,22 +212,22 @@
     type init4 = svld1_ ## su ## sz (cmp ## sz, mem); \
\
     type res_init1 = func_ ## type ## _init1 (); \
-    svbool_t cmp = svcmpne_ ## su ## sz (all_true, init1, res_init1); \
+    svbool_t cmp = svcmpne_ ## su ## sz (cmp ## sz, init1, res_init1); \
     if (svptest_any (all_true, cmp)) \
       __builtin_abort (); \
\
     type res_init2 = func_ ## type ## _init2 (); \
-    cmp = svcmpne_ ## su ## sz (all_true, init2, res_init2); \
+    cmp = svcmpne_ ## su ## sz (cmp ## sz, init2, res_init2); \
     if (svptest_any (all_true, cmp)) \
       __builtin_abort (); \
\
     type res_init3 = func_ ## type ## _init3 (); \
-    cmp = svcmpne_ ## su ## sz (all_true, init3, res_init3); \
+    cmp = svcmpne_ ## su ## sz (cmp ## sz, init3, res_init3); \
     if (svptest_any (all_true, cmp)) \
       __builtin_abort (); \
\
     type res_init4 = func_ ## type ## _init4 (); \
-    cmp = svcmpne_ ## su ## sz (all_true, init4, res_init4); \
+    cmp = svcmpne_ ## su ## sz (cmp ## sz, init4, res_init4); \
     if (svptest_any (all_true, cmp)) \
       __builtin_abort (); \
   }
--
2.34.1