diff options
| author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-02-04 05:41:20 +0000 |
|---|---|---|
| committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-02-04 05:41:20 +0000 |
| commit | 7e899cbb9127c02c58f6e774186a533b0d00681d (patch) | |
| tree | ee47769328cfc5e521b6044a7ddf7419b76f6ce8 /lib/CodeGen/LiveIntervalAnalysis.cpp | |
| parent | c0dc1b7f9725433325ea06e1272ba7fd411c2eb1 (diff) | |
| download | external_llvm-7e899cbb9127c02c58f6e774186a533b0d00681d.zip external_llvm-7e899cbb9127c02c58f6e774186a533b0d00681d.tar.gz external_llvm-7e899cbb9127c02c58f6e774186a533b0d00681d.tar.bz2 | |
Correctly terminate a physreg redefined by an early clobber.
I don't have a test that fails because of this, but a test case like
CodeGen/X86/2009-12-01-EarlyClobberBug.ll exposes the problem. EAX is
redefined by a tied early clobber operand on inline asm, and the live
range should look like this:
%EAX,inf = [48r,64e:0)[64e,80r:1) 0@48r 1@64e
Previously, the two values got merged:
%EAX,inf = [48r,80r:0) 0@48r
With this bug fixed, the REDEF_BY_EC VNInfo flag is no longer needed.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@149768 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/LiveIntervalAnalysis.cpp')
| -rw-r--r-- | lib/CodeGen/LiveIntervalAnalysis.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/CodeGen/LiveIntervalAnalysis.cpp b/lib/CodeGen/LiveIntervalAnalysis.cpp index e268819..d72009c 100644 --- a/lib/CodeGen/LiveIntervalAnalysis.cpp +++ b/lib/CodeGen/LiveIntervalAnalysis.cpp @@ -402,7 +402,7 @@ void LiveIntervals::handlePhysicalRegisterDef(MachineBasicBlock *MBB, if (DefIdx != -1) { if (mi->isRegTiedToUseOperand(DefIdx)) { // Two-address instruction. - end = baseIndex.getRegSlot(); + end = baseIndex.getRegSlot(mi->getOperand(DefIdx).isEarlyClobber()); } else { // Another instruction redefines the register before it is ever read. // Then the register is essentially dead at the instruction that |