Richard Sandiford <richard.sandif...@arm.com> writes: > "yanzhang.wang--- via Gcc-patches" <gcc-patches@gcc.gnu.org> writes: >> From: Yanzhang Wang <yanzhang.w...@intel.com> >> >> gcc/testsuite/ChangeLog: >> >> * gcc.target/aarch64/sve/acle/asm/subr_s8.c: Modify subr with -1 >> to not. >> >> Signed-off-by: Yanzhang Wang <yanzhang.w...@intel.com> >> --- >> >> Tested on my local arm environment and passed. Thanks Andrew Pinski's comment >> the code is the same with that. >> >> gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c >> b/gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c >> index b9615de6655..1cf6916a5e0 100644 >> --- a/gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c >> +++ b/gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c >> @@ -76,8 +76,7 @@ TEST_UNIFORM_Z (subr_1_s8_m_untied, svint8_t, >> >> /* >> ** subr_m1_s8_m: >> -** mov (z[0-9]+\.b), #-1 >> -** subr z0\.b, p0/m, z0\.b, \1 >> +** not z0.b, p0/m, z0.b >> ** ret >> */ >> TEST_UNIFORM_Z (subr_m1_s8_m, svint8_t, > > I think we need this for subr_u8.c too. OK with that change, > and thanks for the fix!
Actually, never mind. I just saw a patch from Thiago Jung Bauerman for the same issue, which is now in trunk. Sorry for the confusion, and thanks again for posting the fix. Richard