[llvm] [CodeGen][MachineLICM] Use RegUnits in HoistRegionPostRA (PR #94608)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 6 06:01:11 PDT 2024
================
@@ -540,22 +556,22 @@ void MachineLICMBase::HoistRegionPostRA(MachineLoop *CurLoop,
// FIXME: That means a reload that're reused in successor block(s) will not
// be LICM'ed.
for (const auto &LI : BB->liveins()) {
- for (MCRegAliasIterator AI(LI.PhysReg, TRI, true); AI.isValid(); ++AI)
- PhysRegDefs.set(*AI);
+ for (MCRegUnitIterator RUI(LI.PhysReg, TRI); RUI.isValid(); ++RUI)
+ RUDefs.set(*RUI);
----------------
arsenm wrote:
This isn't accounting for the lanemasks in liveins, could possibly be the regression reason
https://github.com/llvm/llvm-project/pull/94608
More information about the llvm-commits
mailing list