[PATCH] D59626: Add MachineDCE pass after RenameIndependentSubregs

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 25 13:49:59 PDT 2019


rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: lib/CodeGen/DeadMachineInstructionElim.cpp:193-197
+  for (auto Reg : RecalcRegs) {
+    LIS->removeInterval(Reg);
+    if (!MRI->reg_empty(Reg))
+      LIS->createAndComputeVirtRegInterval(Reg);
+  }
----------------
arsenm wrote:
> Can you just use shrinkToUses?
I am afraid not. It does far less than needed. In particular it does not handle defs, and this cannot be solved by just removing a dead interval.


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

https://reviews.llvm.org/D59626





More information about the llvm-commits mailing list