diff options
author | Evan Cheng <evan.cheng@apple.com> | 2009-06-20 04:34:51 +0000 |
---|---|---|
committer | Evan Cheng <evan.cheng@apple.com> | 2009-06-20 04:34:51 +0000 |
commit | c6a8d0dbe9931b67a6ccdd6e5473df9e83581a99 (patch) | |
tree | 3c2ae9e115fcf1e7e116251f5fd0400d1ae8d9b4 /llvm/lib/CodeGen/LiveVariables.cpp | |
parent | 79ef65a271d9706f05cf2ff7ebecf77bb8a27859 (diff) | |
download | bcm5719-llvm-c6a8d0dbe9931b67a6ccdd6e5473df9e83581a99.tar.gz bcm5719-llvm-c6a8d0dbe9931b67a6ccdd6e5473df9e83581a99.zip |
Fix PR4419: handle defs of partial uses.
llvm-svn: 73816
Diffstat (limited to 'llvm/lib/CodeGen/LiveVariables.cpp')
-rw-r--r-- | llvm/lib/CodeGen/LiveVariables.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/llvm/lib/CodeGen/LiveVariables.cpp b/llvm/lib/CodeGen/LiveVariables.cpp index 62288216912..bd845085bbf 100644 --- a/llvm/lib/CodeGen/LiveVariables.cpp +++ b/llvm/lib/CodeGen/LiveVariables.cpp @@ -359,12 +359,11 @@ bool LiveVariables::HandlePhysRegKill(unsigned Reg, MachineInstr *MI) { // That is, unless we are currently processing the last reference itself. LastRefOrPartRef->addRegisterDead(Reg, TRI, true); - /* Partial uses. Mark register def dead and add implicit def of - sub-registers which are used. - FIXME: LiveIntervalAnalysis can't handle this yet! - EAX<dead> = op AL<imp-def> - That is, EAX def is dead but AL def extends pass it. - Enable this after live interval analysis is fixed to improve codegen! + // Partial uses. Mark register def dead and add implicit def of + // sub-registers which are used. + // EAX<dead> = op AL<imp-def> + // That is, EAX def is dead but AL def extends pass it. + // Enable this after live interval analysis is fixed to improve codegen! else if (!PhysRegUse[Reg]) { PhysRegDef[Reg]->addRegisterDead(Reg, TRI, true); for (const unsigned *SubRegs = TRI->getSubRegisters(Reg); @@ -377,7 +376,7 @@ bool LiveVariables::HandlePhysRegKill(unsigned Reg, MachineInstr *MI) { PartUses.erase(*SS); } } - } */ + } else LastRefOrPartRef->addRegisterKilled(Reg, TRI, true); return true; |