On Wed, Jul 12, 2023 at 4:05 AM Jeff Law <jeffreya...@gmail.com> wrote: > > > > On 7/10/23 22:44, Christoph Muellner wrote: > > From: Christoph Müllner <christoph.muell...@vrull.eu> > > > > Recently, two identical XTheadCondMov tests have been added, which both > > fail. > > Let's fix that by changing the following: > > * Merge both files into one (no need for separate tests for rv32 and rv64) > > * Drop unrelated attribute check test (we already test for `th.mveqz` > > and `th.mvnez` instructions, so there is little additional value) > > * Fix the pattern to allow matching > > > > gcc/testsuite/ChangeLog: > > > > * gcc.target/riscv/xtheadcondmov-indirect-rv32.c: Moved to... > > * gcc.target/riscv/xtheadcondmov-indirect.c: ...here. > > * gcc.target/riscv/xtheadcondmov-indirect-rv64.c: Removed. > I thought this stuff got fixed recently. Certainly happy to see the > files merged though. Here's what I got from the July 4 run:
I have the following with a GCC master from today (a454325bea77a0dd79415480d48233a7c296bc0a): FAIL: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 scan-assembler .attribute arch, "rv32i2p1_m2p0_a2p1_f2p2_d2p2_c2p0_zicsr2p0_zifencei2p0_xtheadcondmov1p0" FAIL: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 scan-assembler .attribute arch, "rv64i2p1_m2p0_a2p1_f2p2_d2p2_c2p0_zicsr2p0_zifencei2p0_xtheadcondmov1p0" With this patch the fails are gone. BR Christoph > > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O0 > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O1 > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 (test for > > excess errors) > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConEmv_imm_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConEmv_imm_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConEmv_reg_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConEmv_reg_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConNmv_imm_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConNmv_imm_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConNmv_reg_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 > > check-function-bodies ConNmv_reg_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 scan-assembler > > .attribute arch, > > "rv32i2p1_m2p0_a2p1_f2p2_d2p2_c2p0_zicsr2p0_zifencei2p0_xtheadcondmov1p0" > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O3 -g > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -Os > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 -flto > > -fno-use-linker-plugin -flto-partition=none > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv32.c -O2 -flto > > -fuse-linker-plugin -fno-fat-lto-objects > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O0 > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O1 > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 (test for > > excess errors) > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConEmv_imm_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConEmv_imm_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConEmv_reg_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConEmv_reg_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConNmv_imm_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConNmv_imm_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConNmv_reg_imm_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 > > check-function-bodies ConNmv_reg_reg_reg > > PASS: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 scan-assembler > > .attribute arch, > > "rv64i2p1_m2p0_a2p1_f2p2_d2p2_c2p0_zicsr2p0_zifencei2p0_xtheadcondmov1p0" > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O3 -g > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -Os > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 -flto > > -fno-use-linker-plugin -flto-partition=none > > UNSUPPORTED: gcc.target/riscv/xtheadcondmov-indirect-rv64.c -O2 -flto > > -fuse-linker-plugin -fno-fat-lto-objects > > > jeff