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

For now, avoid commuting def MI for copy MI's whose source is not killed. That simply trade a live interval for another and because only the non-two-address operands can be folded into loads, may end up pessimising code.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47262 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Evan Cheng
2008-02-18 08:40:53 +00:00
parent 8244243a31
commit 96cfff0906
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lib/CodeGen/SimpleRegisterCoalescing.cpp
View File

@@ -247,6 +247,13 @@ bool SimpleRegisterCoalescing::RemoveCopyByCommutingDef(LiveInterval &IntA,
unsigned CopyIdx = li_->getDefIndex(li_->getInstructionIndex(CopyMI)); unsigned CopyIdx = li_->getDefIndex(li_->getInstructionIndex(CopyMI));
// FIXME: For now, only eliminate the copy by commuting its def is the source
// does not live pass the move. Coalescing those copies may end up may simply
// end up swapping a live interval for another. That and because usually only
// the non-two address operand can be folded can end up pessimizing the code.
if (CopyMI->findRegisterUseOperandIdx(IntA.reg, true) != -1)
return false;
// BValNo is a value number in B that is defined by a copy from A. 'B3' in // BValNo is a value number in B that is defined by a copy from A. 'B3' in
// the example above. // the example above.
LiveInterval::iterator BLR = IntB.FindLiveRangeContaining(CopyIdx); LiveInterval::iterator BLR = IntB.FindLiveRangeContaining(CopyIdx);