[PATCH] D78159: [MachineDCE] Make sure MachineDCE considers subregs when adding liveins.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 16:20:24 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/DeadMachineInstructionElim.cpp:129
       for (const auto &LI : (*S)->liveins())
-        LivePhysRegs.set(LI.PhysReg);
+        for (MCSubRegIterator SR(LI.PhysReg, TRI, /*IncludeSelf=*/true);
+             SR.isValid(); ++SR)
----------------
RegAliasIterator? also need the super regs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78159





More information about the llvm-commits mailing list