[all-commits] [llvm/llvm-project] f3e87a: [RISCV] Add missing VL arguments to the creation o...
Craig Topper via All-commits
all-commits at lists.llvm.org
Mon Oct 3 12:21:23 PDT 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: f3e87a63e50bac4c7e21fbe229dc5aefae32555c
https://github.com/llvm/llvm-project/commit/f3e87a63e50bac4c7e21fbe229dc5aefae32555c
Author: Craig Topper <craig.topper at sifive.com>
Date: 2022-10-03 (Mon, 03 Oct 2022)
Changed paths:
M llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
M llvm/test/CodeGen/RISCV/rvv/stepvector.ll
Log Message:
-----------
[RISCV] Add missing VL arguments to the creation of RISCVISD::VMV_V_X_VL nodes.
VMV_V_X_VL nodes should always have a passthru, a splat, and a VL.
We were sometimes missing the VL.
This went unnoticed because these cases were all selected into the
following node to form a .vx or .vi instruction. The ComplexPattern
that does this, doesn't check the VL operand. I've added an assert
to the ComplexPattern to catch if the operand is missing.
@qcolombet spotted some of these in D134703.
More information about the All-commits
mailing list