replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Change to return a value by reference. 

llvm-svn: 83873
This commit is contained in:
Bob Wilson 2009-10-12 19:01:12 +00:00
parent c7a3cf4066
commit 3af34312d4
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
llvm/lib/Target/ARM/ARMConstantIslandPass.cpp
View File

@ -164,7 +164,7 @@ namespace {
bool DecrementOldEntry(unsigned CPI, MachineInstr* CPEMI); bool DecrementOldEntry(unsigned CPI, MachineInstr* CPEMI);
int LookForExistingCPEntry(CPUser& U, unsigned UserOffset); int LookForExistingCPEntry(CPUser& U, unsigned UserOffset);
bool LookForWater(CPUser&U, unsigned UserOffset, bool LookForWater(CPUser&U, unsigned UserOffset,
MachineBasicBlock** NewMBB); MachineBasicBlock *&NewMBB);
MachineBasicBlock* AcceptWater(MachineBasicBlock *WaterBB, MachineBasicBlock* AcceptWater(MachineBasicBlock *WaterBB,
water_iterator IP); water_iterator IP);
void CreateNewWater(unsigned CPUserIndex, unsigned UserOffset, void CreateNewWater(unsigned CPUserIndex, unsigned UserOffset,
@ -947,13 +947,13 @@ MachineBasicBlock* ARMConstantIslands::AcceptWater(MachineBasicBlock *WaterBB,
/// LookForWater - look for an existing entry in the WaterList in which /// LookForWater - look for an existing entry in the WaterList in which
/// we can place the CPE referenced from U so it's within range of U's MI. /// we can place the CPE referenced from U so it's within range of U's MI.
/// Returns true if found, false if not. If it returns true, *NewMBB /// Returns true if found, false if not. If it returns true, NewMBB
/// is set to the WaterList entry. /// is set to the WaterList entry.
/// For ARM, we prefer the water that's farthest away. For Thumb, prefer /// For ARM, we prefer the water that's farthest away. For Thumb, prefer
/// water that will not introduce padding to water that will; within each /// water that will not introduce padding to water that will; within each
/// group, prefer the water that's farthest away. /// group, prefer the water that's farthest away.
bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset, bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset,
MachineBasicBlock** NewMBB) { MachineBasicBlock *&NewMBB) {
water_iterator IPThatWouldPad; water_iterator IPThatWouldPad;
MachineBasicBlock* WaterBBThatWouldPad = NULL; MachineBasicBlock* WaterBBThatWouldPad = NULL;
if (!WaterList.empty()) { if (!WaterList.empty()) {
@ -971,7 +971,7 @@ bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset,
IPThatWouldPad = IP; IPThatWouldPad = IP;
} }
} else { } else {
*NewMBB = AcceptWater(WaterBB, IP); NewMBB = AcceptWater(WaterBB, IP);
return true; return true;
} }
} }
@ -980,7 +980,7 @@ bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset,
} }
} }
if (isThumb && WaterBBThatWouldPad) { if (isThumb && WaterBBThatWouldPad) {
*NewMBB = AcceptWater(WaterBBThatWouldPad, IPThatWouldPad); NewMBB = AcceptWater(WaterBBThatWouldPad, IPThatWouldPad);
return true; return true;
} }
return false; return false;
@ -1114,7 +1114,7 @@ bool ARMConstantIslands::HandleConstantPoolUser(MachineFunction &MF,
// away that will work. Forward references only for now (although later // away that will work. Forward references only for now (although later
// we might find some that are backwards). // we might find some that are backwards).
if (!LookForWater(U, UserOffset, &NewMBB)) { if (!LookForWater(U, UserOffset, NewMBB)) {
// No water found. // No water found.
DEBUG(errs() << "No water found\n"); DEBUG(errs() << "No water found\n");
CreateNewWater(CPUserIndex, UserOffset, &NewMBB); CreateNewWater(CPUserIndex, UserOffset, &NewMBB);