diff options
author | Edwin Lu <ewlu@rivosinc.com> | 2025-06-10 13:26:42 -0700 |
---|---|---|
committer | Edwin Lu <ewlu@rivosinc.com> | 2025-06-11 11:07:20 -0700 |
commit | 1c5e99ce8744c166d82cb96bbb2d58392b5fb8d7 (patch) | |
tree | 29182d7eb49951ccc0e56628f1741773c6822c4a /libstdc++-v3/testsuite/std | |
parent | 76cbd678d123ed93f99c4c52456bc14290f19b7f (diff) | |
download | gcc-1c5e99ce8744c166d82cb96bbb2d58392b5fb8d7.zip gcc-1c5e99ce8744c166d82cb96bbb2d58392b5fb8d7.tar.gz gcc-1c5e99ce8744c166d82cb96bbb2d58392b5fb8d7.tar.bz2 |
RISC-V: Prevent speculative vsetvl insn scheduling
The instruction scheduler appears to be speculatively hoisting vsetvl
insns outside of their basic block without checking for data
dependencies. This resulted in a situation where the following occurs
vsetvli a5,a1,e32,m1,tu,ma
vle32.v v2,0(a0)
sub a1,a1,a5 <-- a1 potentially set to 0
sh2add a0,a5,a0
vfmacc.vv v1,v2,v2
vsetvli a5,a1,e32,m1,tu,ma <-- incompatible vinfo. update vl to 0
beq a1,zero,.L12 <-- check if avl is 0
This patch would essentially delay the vsetvl update to after the branch
to prevent unnecessarily updating the vinfo at the end of a basic block.
PR/117974
gcc/ChangeLog:
* config/riscv/riscv.cc (struct riscv_tune_param): Add tune
param.
(riscv_sched_can_speculate_insn): Implement.
(TARGET_SCHED_CAN_SPECULATE_INSN): Ditto.
gcc/testsuite/ChangeLog:
* gcc.target/riscv/rvv/vsetvl/pr117974.c: New test.
Signed-off-by: Edwin Lu <ewlu@rivosinc.com>
Diffstat (limited to 'libstdc++-v3/testsuite/std')
0 files changed, 0 insertions, 0 deletions