[PATCH] D139877: [RISCV]{InsertVSETVLI] Reverse order of block traversal in post pass [nfc]
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 12 13:16:45 PST 2022
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp:1244
if (!Used.VL && !Used.usedVTYPE()) {
- ToDelete.push_back(PrevMI);
- // fallthrough
- } else if (canMutatePriorConfig(*PrevMI, MI, Used)) {
- PrevMI->getOperand(2).setImm(MI.getOperand(2).getImm());
ToDelete.push_back(&MI);
+ // Leave NextMI unchanged
----------------
Why is NextMI relevant to whether this instruction can be removed?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139877/new/
https://reviews.llvm.org/D139877
More information about the llvm-commits
mailing list