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 | b8fabe260483cfb16ea12b872c7f9fa2ffaf1433 (patch) | |
tree | eec47a17634b804aee2afec4fea6570cc4454cbb /lib/CodeGen/LiveVariables.cpp | |
parent | 0e6763cfe1afd830c1cd2adbaf99557bdabf6c54 (diff) | |
download | external_llvm-b8fabe260483cfb16ea12b872c7f9fa2ffaf1433.zip external_llvm-b8fabe260483cfb16ea12b872c7f9fa2ffaf1433.tar.gz external_llvm-b8fabe260483cfb16ea12b872c7f9fa2ffaf1433.tar.bz2 |
Fix PR4419: handle defs of partial uses.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@73816 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/LiveVariables.cpp')
-rw-r--r-- | lib/CodeGen/LiveVariables.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/lib/CodeGen/LiveVariables.cpp b/lib/CodeGen/LiveVariables.cpp index 6228821..bd84508 100644 --- a/lib/CodeGen/LiveVariables.cpp +++ b/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; |