Oops, didn't mean to commit that.
Spills should be hoisted out of loops, but we don't want to hoist them to dominating blocks at the same loop depth. That could cause the spills to be executed more often. llvm-svn: 134782
This commit is contained in:
parent
3f60bca9e9
commit
780db902f7
|
@ -344,7 +344,7 @@ MachineInstr *InlineSpiller::traceSiblingValue(unsigned UseReg, VNInfo *UseVNI,
|
|||
// This is a valid spill location dominating UseVNI.
|
||||
// Prefer to spill at a smaller loop depth.
|
||||
unsigned Depth = Loops.getLoopDepth(MBB);
|
||||
if (Depth <= SpillDepth) {
|
||||
if (Depth < SpillDepth) {
|
||||
DEBUG(dbgs() << " spill depth " << Depth << ": " << PrintReg(Reg)
|
||||
<< ':' << VNI->id << '@' << VNI->def << '\n');
|
||||
SVI.SpillReg = Reg;
|
||||
|
|
Loading…
Reference in New Issue