[llvm] [RISCV] Move RISCVInsertVSETVLI to after phi elimination (PR #91440)
Luke Lau via llvm-commits
llvm-commits at lists.llvm.org
Thu May 16 21:36:53 PDT 2024
lukel97 wrote:
> ProcessImplicitDefs should run after InitUndef and add back the undef flag though, otherwise RISCVInsertVSETVLI would never consider any passthrus as undef.
I confused myself here and forgot about the PseudoRVVInitUndef pseudos.
But it turns out we only insert the undef pseudos on non-tied operands, i.e. never the passthru. RISCVInsertVSETVLI conveniently only checks for undef on the passthrus, so we don't need to worry about them.
https://github.com/llvm/llvm-project/pull/91440
More information about the llvm-commits
mailing list