Re: [PATCH] Bug 111071: fix the subr with -1 to not due to the simplify.
Richard Sandiford writes: > "yanzhang.wang--- via Gcc-patches" writes: >> From: Yanzhang Wang >> >> gcc/testsuite/ChangeLog: >> >> * gcc.target/aarch64/sve/acle/asm/subr_s8.c: Modify subr with -1 >> to not. >> >> Signed-off-by: Yanzhang Wang >> --- >> >> 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 >> -** subrz0\.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
Re: [PATCH] Bug 111071: fix the subr with -1 to not due to the simplify.
"yanzhang.wang--- via Gcc-patches" writes: > From: Yanzhang Wang > > gcc/testsuite/ChangeLog: > > * gcc.target/aarch64/sve/acle/asm/subr_s8.c: Modify subr with -1 > to not. > > Signed-off-by: Yanzhang Wang > --- > > 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 > -** subrz0\.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! Richard
[PATCH] Bug 111071: fix the subr with -1 to not due to the simplify.
From: Yanzhang Wang gcc/testsuite/ChangeLog: * gcc.target/aarch64/sve/acle/asm/subr_s8.c: Modify subr with -1 to not. Signed-off-by: Yanzhang Wang --- 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 -** subrz0\.b, p0/m, z0\.b, \1 +** not z0.b, p0/m, z0.b ** ret */ TEST_UNIFORM_Z (subr_m1_s8_m, svint8_t, -- 2.41.0