6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-09-23 01:25:32 +00:00
Code Issues Projects Releases Wiki Activity

Fix kill flags when moving instructions using LiveIntervals::moveInstr(...).

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@150150 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Lang Hames
2012-02-09 04:45:38 +00:00
parent 394d712711
commit fb08b90bf4
1 changed files with 23 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
lib/CodeGen/LiveIntervalAnalysis.cpp
View File

@@ -854,9 +854,21 @@ static void handleMoveECs(LiveIntervals& lis, SlotIndex origIdx,
} }
} }
static void moveKillFlags(unsigned reg, SlotIndex oldIdx, SlotIndex newIdx,
LiveIntervals& lis,
const TargetRegisterInfo& tri) {
MachineInstr* oldKillMI = lis.getInstructionFromIndex(oldIdx);
MachineInstr* newKillMI = lis.getInstructionFromIndex(newIdx);
assert(oldKillMI->killsRegister(reg) && "Old 'kill' instr isn't a kill.");
assert(!newKillMI->killsRegister(reg) && "New kill instr is already a kill.");
oldKillMI->clearRegisterKills(reg, &tri);
newKillMI->addRegisterKilled(reg, &tri);
}
template <typename UseSetT> template <typename UseSetT>
static void handleMoveUses(const MachineBasicBlock *mbb, static void handleMoveUses(const MachineBasicBlock *mbb,
const MachineRegisterInfo& mri, const MachineRegisterInfo& mri,
const TargetRegisterInfo& tri,
const BitVector& reservedRegs, LiveIntervals &lis, const BitVector& reservedRegs, LiveIntervals &lis,
SlotIndex origIdx, SlotIndex miIdx, SlotIndex origIdx, SlotIndex miIdx,
const UseSetT &uses) { const UseSetT &uses) {
@@ -887,10 +899,15 @@ static void handleMoveUses(const MachineBasicBlock *mbb,
const MachineOperand& mop = useI.getOperand(); const MachineOperand& mop = useI.getOperand();
SlotIndex instSlot = lis.getSlotIndexes()->getInstructionIndex(mopI); SlotIndex instSlot = lis.getSlotIndexes()->getInstructionIndex(mopI);
SlotIndex opSlot = instSlot.getRegSlot(mop.isEarlyClobber()); SlotIndex opSlot = instSlot.getRegSlot(mop.isEarlyClobber());
if (opSlot >= lastUseInRange && opSlot < origIdx) { if (opSlot > lastUseInRange && opSlot < origIdx)
lastUseInRange = opSlot; lastUseInRange = opSlot;
}
} }
// If we found a new instr endpoint update the kill flags.
if (lastUseInRange != miIdx.getRegSlot())
moveKillFlags(use, miIdx, lastUseInRange, lis, tri);
// Fix up the range end.
lr->end = lastUseInRange; lr->end = lastUseInRange;
} }
} else { } else {
@@ -902,6 +919,7 @@ static void handleMoveUses(const MachineBasicBlock *mbb,
} else { } else {
bool liveOut = lr->end >= lis.getSlotIndexes()->getMBBEndIdx(mbb); bool liveOut = lr->end >= lis.getSlotIndexes()->getMBBEndIdx(mbb);
if (!liveOut && miIdx.getRegSlot() > lr->end) { if (!liveOut && miIdx.getRegSlot() > lr->end) {
moveKillFlags(use, lr->end, miIdx, lis, tri);
lr->end = miIdx.getRegSlot(); lr->end = miIdx.getRegSlot();
} }
} }
@@ -923,8 +941,7 @@ void LiveIntervals::moveInstr(MachineBasicBlock::iterator insertPt,
// Move the machine instr and obtain its new index. // Move the machine instr and obtain its new index.
indexes_->removeMachineInstrFromMaps(mi); indexes_->removeMachineInstrFromMaps(mi);
mbb->remove(mi); mbb->splice(insertPt, mbb, mi);
mbb->insert(insertPt, mi);
SlotIndex miIdx = indexes_->insertMachineInstrInMaps(mi); SlotIndex miIdx = indexes_->insertMachineInstrInMaps(mi);
// Pick the direction. // Pick the direction.
@@ -961,7 +978,7 @@ void LiveIntervals::moveInstr(MachineBasicBlock::iterator insertPt,
BitVector reservedRegs(tri_->getReservedRegs(*mbb->getParent())); BitVector reservedRegs(tri_->getReservedRegs(*mbb->getParent()));
if (movingUp) { if (movingUp) {
handleMoveUses(mbb, *mri_, reservedRegs, *this, origIdx, miIdx, uses); handleMoveUses(mbb, *mri_, *tri_, reservedRegs, *this, origIdx, miIdx, uses);
handleMoveECs(*this, origIdx, miIdx, ecs); handleMoveECs(*this, origIdx, miIdx, ecs);
handleMoveDeadDefs(*this, origIdx, miIdx, deadDefs); handleMoveDeadDefs(*this, origIdx, miIdx, deadDefs);
handleMoveDefs(*this, origIdx, miIdx, defs); handleMoveDefs(*this, origIdx, miIdx, defs);
@@ -969,7 +986,7 @@ void LiveIntervals::moveInstr(MachineBasicBlock::iterator insertPt,
handleMoveDefs(*this, origIdx, miIdx, defs); handleMoveDefs(*this, origIdx, miIdx, defs);
handleMoveDeadDefs(*this, origIdx, miIdx, deadDefs); handleMoveDeadDefs(*this, origIdx, miIdx, deadDefs);
handleMoveECs(*this, origIdx, miIdx, ecs); handleMoveECs(*this, origIdx, miIdx, ecs);
handleMoveUses(mbb, *mri_, reservedRegs, *this, origIdx, miIdx, uses); handleMoveUses(mbb, *mri_, *tri_, reservedRegs, *this, origIdx, miIdx, uses);
} }
} }