[PATCH] D95489: [AMDGPU] Do not reassign spilled registers
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 27 12:53:35 PST 2021
rampitec added a comment.
I have done some more debugging on it. The only way to get into that spiller HoistSpillHelper::LRE_DidCloneVirtReg() call is from LiveRangeEdit::eliminateDeadDefs if we split an LI. That lives LRM in the inconsistent state so we cannot call unassign anymore. I will change the check to VRM->getPreSplitReg(Reg), it will work, but I have absolutely no idea how to write a reliable test for it.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95489/new/
https://reviews.llvm.org/D95489
More information about the llvm-commits
mailing list