diff options
| author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2011-09-15 06:27:32 +0000 |
|---|---|---|
| committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2011-09-15 06:27:32 +0000 |
| commit | 9b82d50d209adf915d3c7f871dc82cb73349db80 (patch) | |
| tree | 3f229adcc6c31cf06177ef1a3b88740e25732f8b | |
| parent | 75c63087b4e1528cc608b1586014fc1ebad2d9fb (diff) | |
| download | external_llvm-9b82d50d209adf915d3c7f871dc82cb73349db80.zip external_llvm-9b82d50d209adf915d3c7f871dc82cb73349db80.tar.gz external_llvm-9b82d50d209adf915d3c7f871dc82cb73349db80.tar.bz2 | |
Revert r139782, "RemoveCopyByCommutingDef doesn't need hasPHIKill()."
It does, after all.
RemoveCopyByCommutingDef rewrites the uses of one particular value
number in A. It doesn't know how to rewrite phi uses, so there can't be
any.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@139787 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/CodeGen/RegisterCoalescer.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/lib/CodeGen/RegisterCoalescer.cpp b/lib/CodeGen/RegisterCoalescer.cpp index 629e7c3..674d075 100644 --- a/lib/CodeGen/RegisterCoalescer.cpp +++ b/lib/CodeGen/RegisterCoalescer.cpp @@ -566,16 +566,14 @@ bool RegisterCoalescer::AdjustCopiesBackFrom(const CoalescerPair &CP, } /// HasOtherReachingDefs - Return true if there are definitions of IntB -/// other than BValNo val# that can reach uses of AValno val# of IntA, or any -/// of its phis. +/// other than BValNo val# that can reach uses of AValno val# of IntA. bool RegisterCoalescer::HasOtherReachingDefs(LiveInterval &IntA, - LiveInterval &IntB, - VNInfo *AValNo, - VNInfo *BValNo) { + LiveInterval &IntB, + VNInfo *AValNo, + VNInfo *BValNo) { for (LiveInterval::iterator AI = IntA.begin(), AE = IntA.end(); AI != AE; ++AI) { - if (AI->valno != AValNo && !AI->valno->isPHIDef()) - continue; + if (AI->valno != AValNo) continue; LiveInterval::Ranges::iterator BI = std::upper_bound(IntB.ranges.begin(), IntB.ranges.end(), AI->start); if (BI != IntB.ranges.begin()) @@ -647,7 +645,9 @@ bool RegisterCoalescer::RemoveCopyByCommutingDef(const CoalescerPair &CP, VNInfo *AValNo = IntA.getVNInfoAt(CopyIdx.getUseIndex()); assert(AValNo && "COPY source not live"); - if (AValNo->isPHIDef() || AValNo->isUnused()) + // If other defs can reach uses of this def, then it's not safe to perform + // the optimization. + if (AValNo->isPHIDef() || AValNo->isUnused() || AValNo->hasPHIKill()) return false; MachineInstr *DefMI = LIS->getInstructionFromIndex(AValNo->def); if (!DefMI) |