diff options
author | Evan Cheng <evan.cheng@apple.com> | 2007-02-22 23:03:39 +0000 |
---|---|---|
committer | Evan Cheng <evan.cheng@apple.com> | 2007-02-22 23:03:39 +0000 |
commit | 30cac02a925c9d56613711b0e77099cb7252bc9b (patch) | |
tree | 20e9cfc5a008b268a82f51f731badf48ad1ac94d /lib/CodeGen/LiveIntervalAnalysis.cpp | |
parent | 774b864f0e7f3812e7daddf8553649e995ce040a (diff) |
Remove unnecessary isKill properties if a live range has been lengthened due to coalescing.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@34503 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/LiveIntervalAnalysis.cpp')
-rw-r--r-- | lib/CodeGen/LiveIntervalAnalysis.cpp | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/lib/CodeGen/LiveIntervalAnalysis.cpp b/lib/CodeGen/LiveIntervalAnalysis.cpp index 29c095a1aa..f03830ab1d 100644 --- a/lib/CodeGen/LiveIntervalAnalysis.cpp +++ b/lib/CodeGen/LiveIntervalAnalysis.cpp @@ -959,10 +959,22 @@ bool LiveIntervals::JoinCopy(MachineInstr *CopyMI, DOUT << "\n\t\tJoined. Result = "; DestInt.print(DOUT, mri_); DOUT << "\n"; - + // If the intervals were swapped by Join, swap them back so that the register // mapping (in the r2i map) is correct. if (Swapped) SrcInt.swap(DestInt); + + // Live range has been lengthened due to colaescing, eliminate the + // unnecessary kills at the end of the source live ranges. + LiveVariables::VarInfo& vi = lv_->getVarInfo(repSrcReg); + for (unsigned i = 0, e = vi.Kills.size(); i != e; ++i) { + MachineInstr *Kill = vi.Kills[i]; + if (Kill == CopyMI || isRemoved(Kill)) + continue; + if (DestInt.liveAt(getInstructionIndex(Kill) + InstrSlots::NUM)) + unsetRegisterKill(Kill, repSrcReg); + } + removeInterval(repSrcReg); r2rMap_[repSrcReg] = repDstReg; @@ -1471,6 +1483,17 @@ LiveIntervals::hasRegisterUse(unsigned Reg, unsigned Start, unsigned End) { return false; } +/// unsetRegisterKill - Unset IsKill property of all uses of specific register +/// of the specific instruction. +void LiveIntervals::unsetRegisterKill(MachineInstr *MI, unsigned Reg) { + for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) { + MachineOperand &MO = MI->getOperand(i); + if (MO.isReg() && MO.isUse() && MO.isKill() && MO.getReg() && + mri_->regsOverlap(rep(MO.getReg()), Reg)) + MO.unsetIsKill(); + } +} + LiveInterval LiveIntervals::createInterval(unsigned reg) { float Weight = MRegisterInfo::isPhysicalRegister(reg) ? HUGE_VALF : 0.0F; |