[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