On 3/31/25 1:05 PM, Alexandre Oliva wrote:

[testsuite] [riscv] limit vwaddsub-1.c to rv64

The desired vw{add,sub}.wx instructions don't come up on rv32 for the
first two functions, we get v{add,sub}.vx instead.

I suppose this is an oversight, and something about the test is meant
for rv64 only, but the fact that the instruction is spelled out in the
intrinsic name and a different instruction is generated suggests
something may be wrong after all.


I could use a pointer to specs for these intrinsics and instructions to
tell for sure, but a closer look from someone more familiar with them
would definitely be welcome.

Tested on x86_64-linux-gnu native, and gcc-14 target riscv{64,32}-elf.
Ok to install?


for  gcc/testsuite/ChangeLog

        * gcc.target/riscv/rvv/base/vwaddsub-1.c: Require rv64.
I don't immediately see anything in this test or its history to indicate it's only supposed to work for rv64. So it'd be better dig a bit further into this one.

jeff

Reply via email to