Sometimes a MI can define a register as well as defining a super-register at the

same time. Do not mark the "smaller" def as dead.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@41871 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Evan Cheng 2007-09-11 22:34:47 +00:00
parent 01537324ba
commit 6d6d352ed9
2 changed files with 16 additions and 6 deletions

View File

@ -249,9 +249,6 @@ bool LiveVariables::addRegisterDead(unsigned IncomingReg, MachineInstr *MI,
} }
void LiveVariables::HandlePhysRegUse(unsigned Reg, MachineInstr *MI) { void LiveVariables::HandlePhysRegUse(unsigned Reg, MachineInstr *MI) {
// There is a now a proper use, forget about the last partial use.
PhysRegPartUse[Reg] = NULL;
// Turn previous partial def's into read/mod/write. // Turn previous partial def's into read/mod/write.
for (unsigned i = 0, e = PhysRegPartDef[Reg].size(); i != e; ++i) { for (unsigned i = 0, e = PhysRegPartDef[Reg].size(); i != e; ++i) {
MachineInstr *Def = PhysRegPartDef[Reg][i]; MachineInstr *Def = PhysRegPartDef[Reg][i];
@ -266,12 +263,15 @@ void LiveVariables::HandlePhysRegUse(unsigned Reg, MachineInstr *MI) {
// A: EAX = ... // A: EAX = ...
// B: = AX // B: = AX
// Add implicit def to A. // Add implicit def to A.
if (PhysRegInfo[Reg] && !PhysRegUsed[Reg]) { if (PhysRegInfo[Reg] && PhysRegInfo[Reg] != PhysRegPartUse[Reg] &&
!PhysRegUsed[Reg]) {
MachineInstr *Def = PhysRegInfo[Reg]; MachineInstr *Def = PhysRegInfo[Reg];
if (!Def->findRegisterDefOperand(Reg)) if (!Def->findRegisterDefOperand(Reg))
Def->addRegOperand(Reg, true/*IsDef*/,true/*IsImp*/); Def->addRegOperand(Reg, true/*IsDef*/,true/*IsImp*/);
} }
// There is a now a proper use, forget about the last partial use.
PhysRegPartUse[Reg] = NULL;
PhysRegInfo[Reg] = MI; PhysRegInfo[Reg] = MI;
PhysRegUsed[Reg] = true; PhysRegUsed[Reg] = true;
@ -373,7 +373,8 @@ void LiveVariables::HandlePhysRegDef(unsigned Reg, MachineInstr *MI) {
} else if (PhysRegPartUse[SubReg]) } else if (PhysRegPartUse[SubReg])
// Add implicit use / kill to last use of a sub-register. // Add implicit use / kill to last use of a sub-register.
addRegisterKilled(SubReg, PhysRegPartUse[SubReg], true); addRegisterKilled(SubReg, PhysRegPartUse[SubReg], true);
else else if (LastRef != MI)
// This must be a def of the subreg on the same MI.
addRegisterDead(SubReg, LastRef); addRegisterDead(SubReg, LastRef);
} }
} }
@ -381,7 +382,7 @@ void LiveVariables::HandlePhysRegDef(unsigned Reg, MachineInstr *MI) {
if (MI) { if (MI) {
for (const unsigned *SuperRegs = RegInfo->getSuperRegisters(Reg); for (const unsigned *SuperRegs = RegInfo->getSuperRegisters(Reg);
unsigned SuperReg = *SuperRegs; ++SuperRegs) { unsigned SuperReg = *SuperRegs; ++SuperRegs) {
if (PhysRegInfo[SuperReg]) { if (PhysRegInfo[SuperReg] && PhysRegInfo[SuperReg] != MI) {
// The larger register is previously defined. Now a smaller part is // The larger register is previously defined. Now a smaller part is
// being re-defined. Treat it as read/mod/write. // being re-defined. Treat it as read/mod/write.
// EAX = // EAX =

View File

@ -0,0 +1,9 @@
; RUN: llvm-as < %s | llc -march=ppc64
%struct.TCMalloc_SpinLock = type { i32 }
define void @_ZN17TCMalloc_SpinLock4LockEv(%struct.TCMalloc_SpinLock* %this) {
entry:
%tmp3 = call i32 asm sideeffect "1: lwarx $0, 0, $1\0A\09stwcx. $2, 0, $1\0A\09bne- 1b\0A\09isync", "=&r,=*r,r,1,~{dirflag},~{fpsr},~{flags},~{memory}"( i32** null, i32 1, i32* null ) ; <i32> [#uses=0]
unreachable
}