From: Owen Anderson Date: Thu, 13 Nov 2008 21:53:14 +0000 (+0000) Subject: Don't allow the restore point to be placed after terminators. With this change, X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=711fd3db2d3170263850d7ee55987ac1fb9006f3;p=oota-llvm.git Don't allow the restore point to be placed after terminators. With this change, MultiSource/Applications is clean with the prealloc splitter. Some failures remain in SPEC. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@59267 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/PreAllocSplitting.cpp b/lib/CodeGen/PreAllocSplitting.cpp index f7d8ab01026..dfbc796cd3f 100644 --- a/lib/CodeGen/PreAllocSplitting.cpp +++ b/lib/CodeGen/PreAllocSplitting.cpp @@ -233,13 +233,13 @@ PreAllocSplitting::findRestorePoint(MachineBasicBlock *MBB, MachineInstr *MI, unsigned &RestoreIndex) { // FIXME: Allow spill to be inserted to the beginning of the mbb. Update mbb // begin index accordingly. - MachineBasicBlock::iterator Pt = MBB->end(); + MachineBasicBlock::iterator Pt = MBB->getFirstTerminator(); unsigned EndIdx = LIs->getMBBEndIdx(MBB); // Go bottom up if RefsInMBB is empty and the end of the mbb isn't beyond // the last index in the live range. if (RefsInMBB.empty() && LastIdx >= EndIdx) { - MachineBasicBlock::iterator MII = MBB->end(); + MachineBasicBlock::iterator MII = MBB->getFirstTerminator(); MachineBasicBlock::iterator EndPt = MI; --MII; do {