On 1/31/2026 1:11 AM, Alexandre Oliva wrote:
Combine was prevented for some time from combining two ashifts when
the substituted insn couldn't be removed, and r15-7801 changed the
expectations to match.

Combine was later allowed to combine such insns, and the scheduler
reorders the insns so that the expected function body doesn't match.

I've added options to disable scheduling, and I've restored the
combine expectation.

Regstrapped on x86_64-linux-gnu along with other patches in this batch;
also tested with targets riscv64-elf and riscv32-elf on the same host,
both trunk and gcc-15.  Ok to install?


for  gcc/testsuite/ChangeLog

        * gcc.target/riscv/rvv/autovec/bug-3.c: Disable scheduling,
        and restore expectation of slli combine.
OK.  Another possibility with this kind of test is to evaluate how important those insns are for what the test is meant to do and potentially just ignore them.  Given this test is in the rvv/autovec directory it's hard to believe this shift is really important for the core issue in this testcase.

Certainly OK for the trunk, but it's possible just dropping the test of those instructions may be better in the long term.

Jeff

Reply via email to