Convert LiveRegUnits methods to the current convention (it's new code).

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@192619 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Andrew Trick 2013-10-14 20:45:09 +00:00
parent 655a10d96c
commit 5601abb60d
2 changed files with 27 additions and 27 deletions

View File

@ -42,13 +42,13 @@ namespace llvm {
}
/// Adds a register to the set.
void AddReg(unsigned Reg, const MCRegisterInfo &MCRI) {
void addReg(unsigned Reg, const MCRegisterInfo &MCRI) {
for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
LiveUnits.insert(*RUnits);
}
/// Removes a register from the set.
void RemoveReg(unsigned Reg, const MCRegisterInfo &MCRI) {
void removeReg(unsigned Reg, const MCRegisterInfo &MCRI) {
for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
LiveUnits.erase(*RUnits);
}
@ -57,13 +57,13 @@ namespace llvm {
/// Note that we assume the high bits of a physical super register are not
/// preserved unless the instruction has an implicit-use operand reading
/// the super-register or a register unit for the upper bits is available.
void RemoveRegsInMask(const MachineOperand &Op,
void removeRegsInMask(const MachineOperand &Op,
const MCRegisterInfo &MCRI) {
const uint32_t *Mask = Op.getRegMask();
unsigned Bit = 0;
for (unsigned R = 0; R < MCRI.getNumRegs(); ++R) {
if ((*Mask & (1u << Bit)) == 0)
RemoveReg(R, MCRI);
removeReg(R, MCRI);
++Bit;
if (Bit >= 32) {
Bit = 0;
@ -74,7 +74,7 @@ namespace llvm {
/// Returns true if register @p Reg (or one of its super register) is
/// contained in the set.
bool Contains(unsigned Reg, const MCRegisterInfo &MCRI) const {
bool contains(unsigned Reg, const MCRegisterInfo &MCRI) const {
for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits) {
if (LiveUnits.count(*RUnits))
return true;
@ -84,7 +84,7 @@ namespace llvm {
/// Simulates liveness when stepping backwards over an instruction(bundle):
/// Defs are removed from the set, uses added.
void StepBackward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
void stepBackward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
// Remove defined registers and regmask kills from the set.
for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
if (O->isReg()) {
@ -93,9 +93,9 @@ namespace llvm {
unsigned Reg = O->getReg();
if (Reg == 0)
continue;
RemoveReg(Reg, MCRI);
removeReg(Reg, MCRI);
} else if (O->isRegMask()) {
RemoveRegsInMask(*O, MCRI);
removeRegsInMask(*O, MCRI);
}
}
// Add uses to the set.
@ -105,7 +105,7 @@ namespace llvm {
unsigned Reg = O->getReg();
if (Reg == 0)
continue;
AddReg(Reg, MCRI);
addReg(Reg, MCRI);
}
}
@ -115,7 +115,7 @@ namespace llvm {
/// Uses with kill flag get removed from the set, defs added. If possible
/// use StepBackward() instead of this function because some kill flags may
/// be missing.
void StepForward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
void stepForward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
SmallVector<unsigned, 4> Defs;
// Remove killed registers from the set.
for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
@ -130,23 +130,23 @@ namespace llvm {
if (!O->isKill())
continue;
assert(O->isUse());
RemoveReg(Reg, MCRI);
removeReg(Reg, MCRI);
}
} else if (O->isRegMask()) {
RemoveRegsInMask(*O, MCRI);
removeRegsInMask(*O, MCRI);
}
}
// Add defs to the set.
for (unsigned i = 0, e = Defs.size(); i != e; ++i) {
AddReg(Defs[i], MCRI);
addReg(Defs[i], MCRI);
}
}
/// Adds all registers in the live-in list of block @p BB.
void AddLiveIns(const MachineBasicBlock &BB, const MCRegisterInfo &MCRI) {
void addLiveIns(const MachineBasicBlock &BB, const MCRegisterInfo &MCRI) {
for (MachineBasicBlock::livein_iterator L = BB.livein_begin(),
LE = BB.livein_end(); L != LE; ++L) {
AddReg(*L, MCRI);
addReg(*L, MCRI);
}
}
};

View File

@ -976,15 +976,15 @@ static void UpdatePredRedefs(MachineInstr *MI, LiveRegUnits &Redefs,
unsigned Reg = Ops->getReg();
if (Reg == 0)
continue;
Redefs.RemoveReg(Reg, *TRI);
Redefs.removeReg(Reg, *TRI);
}
for (MIBundleOperands Ops(MI); Ops.isValid(); ++Ops) {
if (!Ops->isReg() || !Ops->isDef())
continue;
unsigned Reg = Ops->getReg();
if (Reg == 0 || Redefs.Contains(Reg, *TRI))
if (Reg == 0 || Redefs.contains(Reg, *TRI))
continue;
Redefs.AddReg(Reg, *TRI);
Redefs.addReg(Reg, *TRI);
MachineOperand &Op = *Ops;
MachineInstr *MI = Op.getParent();
@ -1001,7 +1001,7 @@ static void RemoveKills(MachineInstr &MI, const LiveRegUnits &DontKill,
for (MIBundleOperands O(&MI); O.isValid(); ++O) {
if (!O->isReg() || !O->isKill())
continue;
if (DontKill.Contains(O->getReg(), MCRI))
if (DontKill.contains(O->getReg(), MCRI))
O->setIsKill(false);
}
}
@ -1049,13 +1049,13 @@ bool IfConverter::IfConvertSimple(BBInfo &BBI, IfcvtKind Kind) {
// Initialize liveins to the first BB. These are potentiall redefined by
// predicated instructions.
LiveRegUnits Redefs;
Redefs.AddLiveIns(*(CvtBBI->BB), *TRI);
Redefs.AddLiveIns(*(NextBBI->BB), *TRI);
Redefs.addLiveIns(*(CvtBBI->BB), *TRI);
Redefs.addLiveIns(*(NextBBI->BB), *TRI);
// Compute a set of registers which must not be killed by instructions in
// BB1: This is everything live-in to BB2.
LiveRegUnits DontKill;
DontKill.AddLiveIns(*(NextBBI->BB), *TRI);
DontKill.addLiveIns(*(NextBBI->BB), *TRI);
if (CvtBBI->BB->pred_size() > 1) {
BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB);
@ -1154,8 +1154,8 @@ bool IfConverter::IfConvertTriangle(BBInfo &BBI, IfcvtKind Kind) {
// Initialize liveins to the first BB. These are potentially redefined by
// predicated instructions.
LiveRegUnits Redefs;
Redefs.AddLiveIns(*(CvtBBI->BB), *TRI);
Redefs.AddLiveIns(*(NextBBI->BB), *TRI);
Redefs.addLiveIns(*(CvtBBI->BB), *TRI);
Redefs.addLiveIns(*(NextBBI->BB), *TRI);
bool HasEarlyExit = CvtBBI->FalseBB != NULL;
if (CvtBBI->BB->pred_size() > 1) {
@ -1282,7 +1282,7 @@ bool IfConverter::IfConvertDiamond(BBInfo &BBI, IfcvtKind Kind,
// Initialize liveins to the first BB. These are potentially redefined by
// predicated instructions.
LiveRegUnits Redefs;
Redefs.AddLiveIns(*(BBI1->BB), *TRI);
Redefs.addLiveIns(*(BBI1->BB), *TRI);
// Remove the duplicated instructions at the beginnings of both paths.
MachineBasicBlock::iterator DI1 = BBI1->BB->begin();
@ -1315,12 +1315,12 @@ bool IfConverter::IfConvertDiamond(BBInfo &BBI, IfcvtKind Kind,
LiveRegUnits DontKill;
for (MachineBasicBlock::reverse_iterator I = BBI2->BB->rbegin(),
E = MachineBasicBlock::reverse_iterator(DI2); I != E; ++I) {
DontKill.StepBackward(*I, *TRI);
DontKill.stepBackward(*I, *TRI);
}
for (MachineBasicBlock::const_iterator I = BBI1->BB->begin(), E = DI1; I != E;
++I) {
Redefs.StepForward(*I, *TRI);
Redefs.stepForward(*I, *TRI);
}
BBI.BB->splice(BBI.BB->end(), BBI1->BB, BBI1->BB->begin(), DI1);
BBI2->BB->erase(BBI2->BB->begin(), DI2);