[PATCH] D144535: [RISCV] Enable machine copy propagation for copy-like instructions

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetMachine.cpp:344
+
+  if (TM->getOptLevel() >= CodeGenOpt::Default && EnableRISCVCopyPropagation)
+    addPass(createMachineCopyPropagationPass(true));
----------------
pcwang-thead wrote:
> craig.topper wrote:
> > Can this be before the two Expand passes? Especially the one that says "last possible moment"?
> How about between the two Expand passes? we may produce MVs when expanding `PseudoCCMOVGPR`.
Would optimizing them break the macrofusion that `PseudoCCMOVGPR` represents?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144535/new/

https://reviews.llvm.org/D144535