diff options
| author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2011-09-15 04:52:06 +0000 |
|---|---|---|
| committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2011-09-15 04:52:06 +0000 |
| commit | 4ea24e993f179113a9bb76ee152cc490e738c936 (patch) | |
| tree | 1673ff9bb070016265ee0be189d534b1d3cebdb1 /lib/CodeGen | |
| parent | 573a32d4870ff4af7f59fc020caf07fb6ef61e37 (diff) | |
| download | llvm-4ea24e993f179113a9bb76ee152cc490e738c936.tar.gz llvm-4ea24e993f179113a9bb76ee152cc490e738c936.tar.bz2 llvm-4ea24e993f179113a9bb76ee152cc490e738c936.tar.xz | |
It is safe to remat a value killed by phis.
The source live range is recomputed using shrinkToUses() which does
handle phis correctly. The hasPHIKill() condition was relevant in the
old days when ReMaterializeTrivialDef() tried to recompute the live
range itself.
The shrinkToUses() function will mark the original def as dead when no
more uses and phi kills remain. It is then removed by
runOnMachineFunction().
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@139781 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen')
| -rw-r--r-- | lib/CodeGen/RegisterCoalescer.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/CodeGen/RegisterCoalescer.cpp b/lib/CodeGen/RegisterCoalescer.cpp index f3b8ae33c6..674d075c4f 100644 --- a/lib/CodeGen/RegisterCoalescer.cpp +++ b/lib/CodeGen/RegisterCoalescer.cpp @@ -805,9 +805,7 @@ bool RegisterCoalescer::ReMaterializeTrivialDef(LiveInterval &SrcInt, LiveInterval::iterator SrcLR = SrcInt.FindLiveRangeContaining(CopyIdx); assert(SrcLR != SrcInt.end() && "Live range not found!"); VNInfo *ValNo = SrcLR->valno; - // If other defs can reach uses of this def, then it's not safe to perform - // the optimization. - if (ValNo->isPHIDef() || ValNo->isUnused() || ValNo->hasPHIKill()) + if (ValNo->isPHIDef() || ValNo->isUnused()) return false; MachineInstr *DefMI = LIS->getInstructionFromIndex(ValNo->def); if (!DefMI) |