6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-08 06:32:24 +00:00
Code Issues Projects Releases Wiki Activity

When the allocator rewrite a spill register with new virtual register, it replaces other operands of the same register. Watch out for situations where

Browse Source 
only some of the operands are sub-register uses.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@43776 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Evan Cheng 2007-11-06 21:12:10 +00:00
parent 91508a490b
commit be6781b5d0
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/CodeGen/LiveIntervalAnalysis.cpp
View File

@ -379,10 +379,19 @@ addIntervalsForSpills(const LiveInterval &li, VirtRegMap &vrm, unsigned reg) {
if (!MI->getOperand(j).isRegister())
continue;
unsigned RegJ = MI->getOperand(j).getReg();
if (RegJ != 0 && MRegisterInfo::isVirtualRegister(RegJ) &&
RegMap->isSubRegister(RegJ))
if (RegJ == 0 || MRegisterInfo::isPhysicalRegister(RegJ))
continue;
bool isSubRegJ = RegMap->isSubRegister(RegJ);
if (isSubRegJ) {
assert(!isSubReg || RegMap->getSubRegisterIndex(RegJ) == SubIdx);
RegJ = RegMap->getSuperRegister(RegJ);
if (RegJ == li.reg) {
}
// Important to check "isSubRegJ == isSubReg".
// e.g. %reg1024 = MOVSX32rr16 %reg1025. It's possible that both
// registers are coalesced to the same register but only %reg1025 is
// a sub-register use. They should not be rewritten to the same
// register.
if (RegJ == li.reg && isSubRegJ == isSubReg) {
MI->getOperand(j).setReg(NewVReg);
HasUse |= MI->getOperand(j).isUse();
HasDef |= MI->getOperand(j).isDef();