]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
testsuite: aarch64: Adjust SVE ACLE tests to new generated code
authorThiago Jung Bauermann <thiago.bauermann@linaro.org>
Mon, 4 Sep 2023 18:57:10 +0000 (19:57 +0100)
committerRichard Sandiford <richard.sandiford@arm.com>
Mon, 4 Sep 2023 18:57:10 +0000 (19:57 +0100)
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.

Suggested-by: Andrew Pinski <apinski@marvell.com>
gcc/testsuite/
PR testsuite/111071
* gcc.target/aarch64/sve/acle/asm/subr_s8.c: Adjust to new code.
* gcc.target/aarch64/sve/acle/asm/subr_u8.c: Likewise.

gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_s8.c
gcc/testsuite/gcc.target/aarch64/sve/acle/asm/subr_u8.c

index b9615de6655f9ed002f0255d82832a015d64d1c9..3e521bc9ae32da68211d94002003243bfb84796e 100644 (file)
@@ -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,
index 65606b6dda03ea1f1fa350d6e7fdd4cacf7082b8..4922bdbacc47cc94dda5458442dbb916f31258c9 100644 (file)
@@ -76,8 +76,7 @@ TEST_UNIFORM_Z (subr_1_u8_m_untied, svuint8_t,
 
 /*
 ** subr_m1_u8_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_u8_m, svuint8_t,