[PATCH] D130442: [RISCV] Peephole optimization to fold merge.vvm and unmasked intrinsics.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 2 10:33:35 PDT 2022
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:2610
+ CurDAG->getMachineNode(MaskedOpc, DL, True->getVTList(), Ops);
+ ReplaceUses(N, Result);
+
----------------
This does not handle the chain output of True correctly if it has users. We would need to replace True.getValue(1) with N->getValue(1). ReplaceUses will only replace the direct users of N.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130442/new/
https://reviews.llvm.org/D130442
More information about the llvm-commits
mailing list