Hello Richard,
Richard Sandiford <richard.sandif...@arm.com> writes: > Thiago Jung Bauermann via Gcc-patches <gcc-patches@gcc.gnu.org> writes: >> Since commit e7a36e4715c7 "[PATCH] RISC-V: Support simplify (-1-x) for >> vector." these tests fail on aarch64-linux: >> >> === g++ tests === >> >> Running g++:g++.target/aarch64/sve/acle/aarch64-sve-acle-asm.exp ... >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_s8.c -std=gnu++98 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_FULL >> check-function-bodies subr_m1_s8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_s8.c -std=gnu++98 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_OVERLOADS >> check-function-bodies subr_m1_s8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_u8.c -std=gnu++98 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_FULL >> check-function-bodies subr_m1_u8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_u8.c -std=gnu++98 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_OVERLOADS >> check-function-bodies subr_m1_u8_m >> >> === gcc tests === >> >> Running gcc:gcc.target/aarch64/sve/acle/aarch64-sve-acle-asm.exp ... >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_s8.c -std=gnu90 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_FULL >> check-function-bodies subr_m1_s8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_s8.c -std=gnu90 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_OVERLOADS >> check-function-bodies subr_m1_s8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_u8.c -std=gnu90 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_FULL >> check-function-bodies subr_m1_u8_m >> FAIL: gcc.target/aarch64/sve/acle/asm/subr_u8.c -std=gnu90 -O2 >> -fno-schedule-insns -DCHECK_ASM --save-temps -DTEST_OVERLOADS >> check-function-bodies subr_m1_u8_m >> >> Andrew Pinski's analysis in PR testsuite/111071 is that the new code is >> better and the testcase should be updated. I also asked Prathamesh Kulkarni >> in private and he agreed. >> >> Here is the update. With this change, all tests in >> gcc.target/aarch64/sve/acle/aarch64-sve-acle-asm.exp pass on aarch64-linux. >> >> gcc/testsuite/ >> PR testsuite/111071 >> * gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c: Adjust to >> new code. >> * gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_u8.c: Likewise. > > Thanks, pushed to trunk. And sorry for the delay. I somehow > missed this earlier. :( No problem. Thank you for pushing it! -- Thiago