> 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, 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 >
