[PATCH] D127281: [Greedy RegAlloc] Fix the handling of split register in last chance re-coloring.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 8 10:46:45 PDT 2022
arsenm added inline comments.
================
Comment at: llvm/lib/CodeGen/RegAllocGreedy.cpp:1907
std::tie(LI, PhysReg) = RecolorStack[I];
- Matrix->assign(*LI, PhysReg);
+ if (!LI->empty() && !MRI->reg_nodbg_empty(LI->reg()))
+ Matrix->assign(*LI, PhysReg);
----------------
The possibility for splitting during last chance recoloring confused me when I was last looking at this here. Is it a problem that the split was also not rolled back as well?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127281/new/
https://reviews.llvm.org/D127281
More information about the llvm-commits
mailing list