Chris Lattner | 67a9801 | 2003-10-12 21:44:18 +0000 | [diff] [blame] | 1 | //===- LoopSimplify.cpp - Loop Canonicalization Pass ----------------------===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | ee2c50c | 2003-10-12 21:43:28 +0000 | [diff] [blame] | 10 | // This pass performs several transformations to transform natural loops into a |
| 11 | // simpler form, which makes subsequent analyses and transformations simpler and |
| 12 | // more effective. |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 13 | // |
| 14 | // Loop pre-header insertion guarantees that there is a single, non-critical |
| 15 | // entry edge from outside of the loop to the loop header. This simplifies a |
| 16 | // number of analyses and transformations, such as LICM. |
| 17 | // |
| 18 | // Loop exit-block insertion guarantees that all exit blocks from the loop |
| 19 | // (blocks which are outside of the loop that have predecessors inside of the |
Chris Lattner | 66ea98e | 2003-12-10 17:20:35 +0000 | [diff] [blame] | 20 | // loop) only have predecessors from inside of the loop (and are thus dominated |
| 21 | // by the loop header). This simplifies transformations such as store-sinking |
| 22 | // that are built into LICM. |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 23 | // |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 24 | // This pass also guarantees that loops will have exactly one backedge. |
| 25 | // |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 26 | // Indirectbr instructions introduce several complications. If the loop |
| 27 | // contains or is entered by an indirectbr instruction, it may not be possible |
| 28 | // to transform the loop and make these guarantees. Client code should check |
| 29 | // that these conditions are true before relying on them. |
| 30 | // |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 31 | // Note that the simplifycfg pass will clean up blocks which are split out but |
Chris Lattner | ee2c50c | 2003-10-12 21:43:28 +0000 | [diff] [blame] | 32 | // end up being unnecessary, so usage of this pass should not pessimize |
| 33 | // generated code. |
| 34 | // |
| 35 | // This pass obviously modifies the CFG, but updates loop information and |
| 36 | // dominator information. |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 37 | // |
| 38 | //===----------------------------------------------------------------------===// |
| 39 | |
Chris Lattner | d216e8b | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 40 | #define DEBUG_TYPE "loopsimplify" |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 41 | #include "llvm/Transforms/Scalar.h" |
Chris Lattner | 3cb63dd | 2007-10-29 02:30:37 +0000 | [diff] [blame] | 42 | #include "llvm/Constants.h" |
Misha Brukman | 47b14a4 | 2004-07-29 17:30:56 +0000 | [diff] [blame] | 43 | #include "llvm/Instructions.h" |
Devang Patel | 689fac0 | 2010-03-15 21:25:29 +0000 | [diff] [blame] | 44 | #include "llvm/IntrinsicInst.h" |
Chris Lattner | 2ef703e | 2004-03-14 03:59:22 +0000 | [diff] [blame] | 45 | #include "llvm/Function.h" |
Owen Anderson | 0a205a4 | 2009-07-05 22:41:43 +0000 | [diff] [blame] | 46 | #include "llvm/LLVMContext.h" |
Chris Lattner | 2ef703e | 2004-03-14 03:59:22 +0000 | [diff] [blame] | 47 | #include "llvm/Type.h" |
Chris Lattner | cec5b88 | 2005-03-25 06:37:22 +0000 | [diff] [blame] | 48 | #include "llvm/Analysis/AliasAnalysis.h" |
Dan Gohman | ffa75cd | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 49 | #include "llvm/Analysis/ScalarEvolution.h" |
Chris Lattner | 0f98e75 | 2003-12-19 06:27:08 +0000 | [diff] [blame] | 50 | #include "llvm/Analysis/Dominators.h" |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 51 | #include "llvm/Analysis/LoopPass.h" |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 52 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 53 | #include "llvm/Transforms/Utils/Local.h" |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 54 | #include "llvm/Support/CFG.h" |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 55 | #include "llvm/Support/Debug.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 56 | #include "llvm/ADT/SetOperations.h" |
| 57 | #include "llvm/ADT/SetVector.h" |
| 58 | #include "llvm/ADT/Statistic.h" |
| 59 | #include "llvm/ADT/DepthFirstIterator.h" |
Chris Lattner | 66ea98e | 2003-12-10 17:20:35 +0000 | [diff] [blame] | 60 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 61 | |
Chris Lattner | d216e8b | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 62 | STATISTIC(NumInserted, "Number of pre-header or exit blocks inserted"); |
| 63 | STATISTIC(NumNested , "Number of nested loops split out"); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 64 | |
Chris Lattner | d216e8b | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 65 | namespace { |
Nick Lewycky | 6726b6d | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 66 | struct LoopSimplify : public LoopPass { |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 67 | static char ID; // Pass identification, replacement for typeid |
Owen Anderson | 081c34b | 2010-10-19 17:21:58 +0000 | [diff] [blame^] | 68 | LoopSimplify() : LoopPass(ID) { |
| 69 | initializeLoopSimplifyPass(*PassRegistry::getPassRegistry()); |
| 70 | } |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 71 | |
Chris Lattner | cec5b88 | 2005-03-25 06:37:22 +0000 | [diff] [blame] | 72 | // AA - If we have an alias analysis object to update, this is it, otherwise |
| 73 | // this is null. |
| 74 | AliasAnalysis *AA; |
Chris Lattner | c27e056 | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 75 | LoopInfo *LI; |
Devang Patel | 0e7f728 | 2007-06-21 17:23:45 +0000 | [diff] [blame] | 76 | DominatorTree *DT; |
Dan Gohman | ffa75cd | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 77 | ScalarEvolution *SE; |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 78 | Loop *L; |
| 79 | virtual bool runOnLoop(Loop *L, LPPassManager &LPM); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 80 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 81 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
| 82 | // We need loop information to identify the loops... |
Dan Gohman | 052f000 | 2010-07-26 18:11:16 +0000 | [diff] [blame] | 83 | AU.addRequired<DominatorTree>(); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 84 | AU.addPreserved<DominatorTree>(); |
Dan Gohman | 1e381fc | 2010-07-16 17:58:45 +0000 | [diff] [blame] | 85 | |
Dan Gohman | 052f000 | 2010-07-26 18:11:16 +0000 | [diff] [blame] | 86 | AU.addRequired<LoopInfo>(); |
Dan Gohman | 1e381fc | 2010-07-16 17:58:45 +0000 | [diff] [blame] | 87 | AU.addPreserved<LoopInfo>(); |
| 88 | |
Devang Patel | 4c37c07 | 2008-06-06 17:50:58 +0000 | [diff] [blame] | 89 | AU.addPreserved<AliasAnalysis>(); |
Dan Gohman | ffa75cd | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 90 | AU.addPreserved<ScalarEvolution>(); |
Chris Lattner | 94f4032 | 2005-08-10 02:07:32 +0000 | [diff] [blame] | 91 | AU.addPreservedID(BreakCriticalEdgesID); // No critical edges added. |
Dan Gohman | 052f000 | 2010-07-26 18:11:16 +0000 | [diff] [blame] | 92 | AU.addPreserved<DominanceFrontier>(); |
| 93 | AU.addPreservedID(LCSSAID); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 94 | } |
Devang Patel | 58e0ef1 | 2007-07-19 18:02:32 +0000 | [diff] [blame] | 95 | |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 96 | /// verifyAnalysis() - Verify LoopSimplifyForm's guarantees. |
| 97 | void verifyAnalysis() const; |
Devang Patel | 58e0ef1 | 2007-07-19 18:02:32 +0000 | [diff] [blame] | 98 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 99 | private: |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 100 | bool ProcessLoop(Loop *L, LPPassManager &LPM); |
Chris Lattner | 59fb87d | 2004-04-18 22:27:10 +0000 | [diff] [blame] | 101 | BasicBlock *RewriteLoopExitBlock(Loop *L, BasicBlock *Exit); |
Dan Gohman | 0df6e09 | 2009-07-14 01:37:59 +0000 | [diff] [blame] | 102 | BasicBlock *InsertPreheaderForLoop(Loop *L); |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 103 | Loop *SeparateNestedLoop(Loop *L, LPPassManager &LPM); |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 104 | BasicBlock *InsertUniqueBackedgeBlock(Loop *L, BasicBlock *Preheader); |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 105 | void PlaceSplitBlockCarefully(BasicBlock *NewBB, |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 106 | SmallVectorImpl<BasicBlock*> &SplitPreds, |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 107 | Loop *L); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 108 | }; |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 109 | } |
| 110 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 111 | char LoopSimplify::ID = 0; |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 112 | INITIALIZE_PASS_BEGIN(LoopSimplify, "loopsimplify", |
| 113 | "Canonicalize natural loops", true, false) |
| 114 | INITIALIZE_PASS_DEPENDENCY(DominatorTree) |
| 115 | INITIALIZE_PASS_DEPENDENCY(LoopInfo) |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 116 | INITIALIZE_PASS_END(LoopSimplify, "loopsimplify", |
Owen Anderson | ce665bd | 2010-10-07 22:25:06 +0000 | [diff] [blame] | 117 | "Canonicalize natural loops", true, false) |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 118 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 119 | // Publically exposed interface to pass... |
Owen Anderson | 90c579d | 2010-08-06 18:33:48 +0000 | [diff] [blame] | 120 | char &llvm::LoopSimplifyID = LoopSimplify::ID; |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 121 | Pass *llvm::createLoopSimplifyPass() { return new LoopSimplify(); } |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 122 | |
Dan Gohman | 34d2b90 | 2009-12-18 00:28:43 +0000 | [diff] [blame] | 123 | /// runOnLoop - Run down all loops in the CFG (recursively, but we could do |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 124 | /// it in any convenient order) inserting preheaders... |
| 125 | /// |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 126 | bool LoopSimplify::runOnLoop(Loop *l, LPPassManager &LPM) { |
| 127 | L = l; |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 128 | bool Changed = false; |
Chris Lattner | c27e056 | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 129 | LI = &getAnalysis<LoopInfo>(); |
Duncan Sands | 1465d61 | 2009-01-28 13:14:17 +0000 | [diff] [blame] | 130 | AA = getAnalysisIfAvailable<AliasAnalysis>(); |
Devang Patel | 0e7f728 | 2007-06-21 17:23:45 +0000 | [diff] [blame] | 131 | DT = &getAnalysis<DominatorTree>(); |
Dan Gohman | ffa75cd | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 132 | SE = getAnalysisIfAvailable<ScalarEvolution>(); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 133 | |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 134 | Changed |= ProcessLoop(L, LPM); |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 135 | |
| 136 | return Changed; |
| 137 | } |
| 138 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 139 | /// ProcessLoop - Walk the loop structure in depth first order, ensuring that |
| 140 | /// all loops have preheaders. |
| 141 | /// |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 142 | bool LoopSimplify::ProcessLoop(Loop *L, LPPassManager &LPM) { |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 143 | bool Changed = false; |
Chris Lattner | 3bb4657 | 2006-08-12 05:25:00 +0000 | [diff] [blame] | 144 | ReprocessLoop: |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 145 | |
Dan Gohman | 2d0a91c | 2010-03-10 02:18:48 +0000 | [diff] [blame] | 146 | // Check to see that no blocks (other than the header) in this loop have |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 147 | // predecessors that are not in the loop. This is not valid for natural |
| 148 | // loops, but can occur if the blocks are unreachable. Since they are |
| 149 | // unreachable we can just shamelessly delete those CFG edges! |
| 150 | for (Loop::block_iterator BB = L->block_begin(), E = L->block_end(); |
| 151 | BB != E; ++BB) { |
| 152 | if (*BB == L->getHeader()) continue; |
| 153 | |
Gabor Greif | 481c4c0 | 2010-07-22 17:18:03 +0000 | [diff] [blame] | 154 | SmallPtrSet<BasicBlock*, 4> BadPreds; |
| 155 | for (pred_iterator PI = pred_begin(*BB), |
| 156 | PE = pred_end(*BB); PI != PE; ++PI) { |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 157 | BasicBlock *P = *PI; |
| 158 | if (!L->contains(P)) |
| 159 | BadPreds.insert(P); |
| 160 | } |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 161 | |
| 162 | // Delete each unique out-of-loop (and thus dead) predecessor. |
Gabor Greif | 481c4c0 | 2010-07-22 17:18:03 +0000 | [diff] [blame] | 163 | for (SmallPtrSet<BasicBlock*, 4>::iterator I = BadPreds.begin(), |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 164 | E = BadPreds.end(); I != E; ++I) { |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 165 | |
| 166 | DEBUG(dbgs() << "LoopSimplify: Deleting edge from dead predecessor "; |
| 167 | WriteAsOperand(dbgs(), *I, false); |
| 168 | dbgs() << "\n"); |
| 169 | |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 170 | // Inform each successor of each dead pred. |
| 171 | for (succ_iterator SI = succ_begin(*I), SE = succ_end(*I); SI != SE; ++SI) |
| 172 | (*SI)->removePredecessor(*I); |
| 173 | // Zap the dead pred's terminator and replace it with unreachable. |
| 174 | TerminatorInst *TI = (*I)->getTerminator(); |
| 175 | TI->replaceAllUsesWith(UndefValue::get(TI->getType())); |
| 176 | (*I)->getTerminator()->eraseFromParent(); |
| 177 | new UnreachableInst((*I)->getContext(), *I); |
| 178 | Changed = true; |
| 179 | } |
| 180 | } |
Chris Lattner | 2ef703e | 2004-03-14 03:59:22 +0000 | [diff] [blame] | 181 | |
Dan Gohman | 8566963 | 2010-02-25 06:57:05 +0000 | [diff] [blame] | 182 | // If there are exiting blocks with branches on undef, resolve the undef in |
| 183 | // the direction which will exit the loop. This will help simplify loop |
| 184 | // trip count computations. |
| 185 | SmallVector<BasicBlock*, 8> ExitingBlocks; |
| 186 | L->getExitingBlocks(ExitingBlocks); |
| 187 | for (SmallVectorImpl<BasicBlock *>::iterator I = ExitingBlocks.begin(), |
| 188 | E = ExitingBlocks.end(); I != E; ++I) |
| 189 | if (BranchInst *BI = dyn_cast<BranchInst>((*I)->getTerminator())) |
| 190 | if (BI->isConditional()) { |
| 191 | if (UndefValue *Cond = dyn_cast<UndefValue>(BI->getCondition())) { |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 192 | |
| 193 | DEBUG(dbgs() << "LoopSimplify: Resolving \"br i1 undef\" to exit in "; |
| 194 | WriteAsOperand(dbgs(), *I, false); |
| 195 | dbgs() << "\n"); |
| 196 | |
Dan Gohman | 8566963 | 2010-02-25 06:57:05 +0000 | [diff] [blame] | 197 | BI->setCondition(ConstantInt::get(Cond->getType(), |
| 198 | !L->contains(BI->getSuccessor(0)))); |
| 199 | Changed = true; |
| 200 | } |
| 201 | } |
| 202 | |
Chris Lattner | fa78946 | 2006-08-12 04:51:20 +0000 | [diff] [blame] | 203 | // Does the loop already have a preheader? If so, don't insert one. |
Dan Gohman | 0df6e09 | 2009-07-14 01:37:59 +0000 | [diff] [blame] | 204 | BasicBlock *Preheader = L->getLoopPreheader(); |
| 205 | if (!Preheader) { |
| 206 | Preheader = InsertPreheaderForLoop(L); |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 207 | if (Preheader) { |
Dan Gohman | fe60104 | 2010-06-22 15:08:57 +0000 | [diff] [blame] | 208 | ++NumInserted; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 209 | Changed = true; |
| 210 | } |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 211 | } |
| 212 | |
Chris Lattner | 66ea98e | 2003-12-10 17:20:35 +0000 | [diff] [blame] | 213 | // Next, check to make sure that all exit nodes of the loop only have |
| 214 | // predecessors that are inside of the loop. This check guarantees that the |
| 215 | // loop preheader/header will dominate the exit blocks. If the exit block has |
Chris Lattner | ee628cf | 2006-02-12 01:59:10 +0000 | [diff] [blame] | 216 | // predecessors from outside of the loop, split the edge now. |
Devang Patel | b7211a2 | 2007-08-21 00:31:24 +0000 | [diff] [blame] | 217 | SmallVector<BasicBlock*, 8> ExitBlocks; |
Chris Lattner | ee628cf | 2006-02-12 01:59:10 +0000 | [diff] [blame] | 218 | L->getExitBlocks(ExitBlocks); |
Chris Lattner | c27e056 | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 219 | |
Dan Gohman | 17146ba | 2010-02-05 19:20:15 +0000 | [diff] [blame] | 220 | SmallSetVector<BasicBlock *, 8> ExitBlockSet(ExitBlocks.begin(), |
| 221 | ExitBlocks.end()); |
| 222 | for (SmallSetVector<BasicBlock *, 8>::iterator I = ExitBlockSet.begin(), |
Chris Lattner | fed22aa | 2004-07-15 08:20:22 +0000 | [diff] [blame] | 223 | E = ExitBlockSet.end(); I != E; ++I) { |
| 224 | BasicBlock *ExitBlock = *I; |
Chris Lattner | de7aee7 | 2004-07-15 05:36:31 +0000 | [diff] [blame] | 225 | for (pred_iterator PI = pred_begin(ExitBlock), PE = pred_end(ExitBlock); |
| 226 | PI != PE; ++PI) |
Chris Lattner | 8587eb3 | 2006-02-11 02:13:17 +0000 | [diff] [blame] | 227 | // Must be exactly this loop: no subloops, parent loops, or non-loop preds |
| 228 | // allowed. |
Chris Lattner | ee628cf | 2006-02-12 01:59:10 +0000 | [diff] [blame] | 229 | if (!L->contains(*PI)) { |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 230 | if (RewriteLoopExitBlock(L, ExitBlock)) { |
Dan Gohman | fe60104 | 2010-06-22 15:08:57 +0000 | [diff] [blame] | 231 | ++NumInserted; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 232 | Changed = true; |
| 233 | } |
Chris Lattner | de7aee7 | 2004-07-15 05:36:31 +0000 | [diff] [blame] | 234 | break; |
| 235 | } |
Chris Lattner | fed22aa | 2004-07-15 08:20:22 +0000 | [diff] [blame] | 236 | } |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 237 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 238 | // If the header has more than two predecessors at this point (from the |
| 239 | // preheader and from multiple backedges), we must adjust the loop. |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 240 | BasicBlock *LoopLatch = L->getLoopLatch(); |
| 241 | if (!LoopLatch) { |
Chris Lattner | 3bb4657 | 2006-08-12 05:25:00 +0000 | [diff] [blame] | 242 | // If this is really a nested loop, rip it out into a child loop. Don't do |
| 243 | // this for loops with a giant number of backedges, just factor them into a |
| 244 | // common backedge instead. |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 245 | if (L->getNumBackEdges() < 8) { |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 246 | if (SeparateNestedLoop(L, LPM)) { |
Chris Lattner | 3bb4657 | 2006-08-12 05:25:00 +0000 | [diff] [blame] | 247 | ++NumNested; |
| 248 | // This is a big restructuring change, reprocess the whole loop. |
Chris Lattner | 3bb4657 | 2006-08-12 05:25:00 +0000 | [diff] [blame] | 249 | Changed = true; |
| 250 | // GCC doesn't tail recursion eliminate this. |
| 251 | goto ReprocessLoop; |
| 252 | } |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 253 | } |
| 254 | |
Chris Lattner | 3bb4657 | 2006-08-12 05:25:00 +0000 | [diff] [blame] | 255 | // If we either couldn't, or didn't want to, identify nesting of the loops, |
| 256 | // insert a new block that all backedges target, then make it jump to the |
| 257 | // loop header. |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 258 | LoopLatch = InsertUniqueBackedgeBlock(L, Preheader); |
| 259 | if (LoopLatch) { |
Dan Gohman | fe60104 | 2010-06-22 15:08:57 +0000 | [diff] [blame] | 260 | ++NumInserted; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 261 | Changed = true; |
| 262 | } |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 263 | } |
| 264 | |
Chris Lattner | 94f4032 | 2005-08-10 02:07:32 +0000 | [diff] [blame] | 265 | // Scan over the PHI nodes in the loop header. Since they now have only two |
| 266 | // incoming values (the loop is canonicalized), we may have simplified the PHI |
| 267 | // down to 'X = phi [X, Y]', which should be replaced with 'Y'. |
| 268 | PHINode *PN; |
Chris Lattner | 94f4032 | 2005-08-10 02:07:32 +0000 | [diff] [blame] | 269 | for (BasicBlock::iterator I = L->getHeader()->begin(); |
| 270 | (PN = dyn_cast<PHINode>(I++)); ) |
Dan Gohman | bccfc24 | 2009-09-03 15:34:35 +0000 | [diff] [blame] | 271 | if (Value *V = PN->hasConstantValue(DT)) { |
Devang Patel | 4c37c07 | 2008-06-06 17:50:58 +0000 | [diff] [blame] | 272 | if (AA) AA->deleteValue(PN); |
| 273 | PN->replaceAllUsesWith(V); |
| 274 | PN->eraseFromParent(); |
| 275 | } |
Chris Lattner | 94f4032 | 2005-08-10 02:07:32 +0000 | [diff] [blame] | 276 | |
Bob Wilson | 5cd8770 | 2010-01-15 21:55:02 +0000 | [diff] [blame] | 277 | // If this loop has multiple exits and the exits all go to the same |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 278 | // block, attempt to merge the exits. This helps several passes, such |
| 279 | // as LoopRotation, which do not support loops with multiple exits. |
| 280 | // SimplifyCFG also does this (and this code uses the same utility |
| 281 | // function), however this code is loop-aware, where SimplifyCFG is |
| 282 | // not. That gives it the advantage of being able to hoist |
| 283 | // loop-invariant instructions out of the way to open up more |
| 284 | // opportunities, and the disadvantage of having the responsibility |
| 285 | // to preserve dominator information. |
Dan Gohman | b1dc915 | 2009-11-05 21:48:32 +0000 | [diff] [blame] | 286 | bool UniqueExit = true; |
| 287 | if (!ExitBlocks.empty()) |
| 288 | for (unsigned i = 1, e = ExitBlocks.size(); i != e; ++i) |
| 289 | if (ExitBlocks[i] != ExitBlocks[0]) { |
| 290 | UniqueExit = false; |
| 291 | break; |
| 292 | } |
| 293 | if (UniqueExit) { |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 294 | for (unsigned i = 0, e = ExitingBlocks.size(); i != e; ++i) { |
| 295 | BasicBlock *ExitingBlock = ExitingBlocks[i]; |
| 296 | if (!ExitingBlock->getSinglePredecessor()) continue; |
| 297 | BranchInst *BI = dyn_cast<BranchInst>(ExitingBlock->getTerminator()); |
| 298 | if (!BI || !BI->isConditional()) continue; |
| 299 | CmpInst *CI = dyn_cast<CmpInst>(BI->getCondition()); |
| 300 | if (!CI || CI->getParent() != ExitingBlock) continue; |
| 301 | |
| 302 | // Attempt to hoist out all instructions except for the |
| 303 | // comparison and the branch. |
| 304 | bool AllInvariant = true; |
Dan Gohman | 2aa93ef | 2009-06-30 01:24:43 +0000 | [diff] [blame] | 305 | for (BasicBlock::iterator I = ExitingBlock->begin(); &*I != BI; ) { |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 306 | Instruction *Inst = I++; |
Devang Patel | 689fac0 | 2010-03-15 21:25:29 +0000 | [diff] [blame] | 307 | // Skip debug info intrinsics. |
| 308 | if (isa<DbgInfoIntrinsic>(Inst)) |
| 309 | continue; |
Dan Gohman | 2aa93ef | 2009-06-30 01:24:43 +0000 | [diff] [blame] | 310 | if (Inst == CI) |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 311 | continue; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 312 | if (!L->makeLoopInvariant(Inst, Changed, |
| 313 | Preheader ? Preheader->getTerminator() : 0)) { |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 314 | AllInvariant = false; |
| 315 | break; |
| 316 | } |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 317 | } |
| 318 | if (!AllInvariant) continue; |
| 319 | |
| 320 | // The block has now been cleared of all instructions except for |
| 321 | // a comparison and a conditional branch. SimplifyCFG may be able |
| 322 | // to fold it now. |
| 323 | if (!FoldBranchToCommonDest(BI)) continue; |
| 324 | |
| 325 | // Success. The block is now dead, so remove it from the loop, |
| 326 | // update the dominator tree and dominance frontier, and delete it. |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 327 | |
| 328 | DEBUG(dbgs() << "LoopSimplify: Eliminating exiting block "; |
| 329 | WriteAsOperand(dbgs(), ExitingBlock, false); |
| 330 | dbgs() << "\n"); |
| 331 | |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 332 | assert(pred_begin(ExitingBlock) == pred_end(ExitingBlock)); |
| 333 | Changed = true; |
Dan Gohman | a1baee2 | 2009-06-27 22:32:36 +0000 | [diff] [blame] | 334 | LI->removeBlock(ExitingBlock); |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 335 | |
| 336 | DominanceFrontier *DF = getAnalysisIfAvailable<DominanceFrontier>(); |
| 337 | DomTreeNode *Node = DT->getNode(ExitingBlock); |
| 338 | const std::vector<DomTreeNodeBase<BasicBlock> *> &Children = |
| 339 | Node->getChildren(); |
Dan Gohman | 5184635 | 2009-09-30 20:54:16 +0000 | [diff] [blame] | 340 | while (!Children.empty()) { |
| 341 | DomTreeNode *Child = Children.front(); |
| 342 | DT->changeImmediateDominator(Child, Node->getIDom()); |
| 343 | if (DF) DF->changeImmediateDominator(Child->getBlock(), |
Dan Gohman | 4b35f83 | 2009-06-27 21:30:38 +0000 | [diff] [blame] | 344 | Node->getIDom()->getBlock(), |
| 345 | DT); |
| 346 | } |
| 347 | DT->eraseNode(ExitingBlock); |
| 348 | if (DF) DF->removeBlock(ExitingBlock); |
| 349 | |
| 350 | BI->getSuccessor(0)->removePredecessor(ExitingBlock); |
| 351 | BI->getSuccessor(1)->removePredecessor(ExitingBlock); |
| 352 | ExitingBlock->eraseFromParent(); |
| 353 | } |
| 354 | } |
| 355 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 356 | return Changed; |
| 357 | } |
| 358 | |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 359 | /// InsertPreheaderForLoop - Once we discover that a loop doesn't have a |
| 360 | /// preheader, this method is called to insert one. This method has two phases: |
| 361 | /// preheader insertion and analysis updating. |
| 362 | /// |
Dan Gohman | 0df6e09 | 2009-07-14 01:37:59 +0000 | [diff] [blame] | 363 | BasicBlock *LoopSimplify::InsertPreheaderForLoop(Loop *L) { |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 364 | BasicBlock *Header = L->getHeader(); |
| 365 | |
| 366 | // Compute the set of predecessors of the loop that are not in the loop. |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 367 | SmallVector<BasicBlock*, 8> OutsideBlocks; |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 368 | for (pred_iterator PI = pred_begin(Header), PE = pred_end(Header); |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 369 | PI != PE; ++PI) { |
| 370 | BasicBlock *P = *PI; |
| 371 | if (!L->contains(P)) { // Coming in from outside the loop? |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 372 | // If the loop is branched to from an indirect branch, we won't |
| 373 | // be able to fully transform the loop, because it prohibits |
| 374 | // edge splitting. |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 375 | if (isa<IndirectBrInst>(P->getTerminator())) return 0; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 376 | |
| 377 | // Keep track of it. |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 378 | OutsideBlocks.push_back(P); |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 379 | } |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 380 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 381 | |
Chris Lattner | c398457 | 2006-09-23 07:40:52 +0000 | [diff] [blame] | 382 | // Split out the loop pre-header. |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 383 | BasicBlock *NewBB = |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 384 | SplitBlockPredecessors(Header, &OutsideBlocks[0], OutsideBlocks.size(), |
| 385 | ".preheader", this); |
Chris Lattner | 9f879cf | 2003-02-27 22:48:57 +0000 | [diff] [blame] | 386 | |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 387 | DEBUG(dbgs() << "LoopSimplify: Creating pre-header "; |
| 388 | WriteAsOperand(dbgs(), NewBB, false); |
| 389 | dbgs() << "\n"); |
| 390 | |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 391 | // Make sure that NewBB is put someplace intelligent, which doesn't mess up |
| 392 | // code layout too horribly. |
| 393 | PlaceSplitBlockCarefully(NewBB, OutsideBlocks, L); |
Dan Gohman | 0df6e09 | 2009-07-14 01:37:59 +0000 | [diff] [blame] | 394 | |
| 395 | return NewBB; |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 396 | } |
| 397 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 398 | /// RewriteLoopExitBlock - Ensure that the loop preheader dominates all exit |
| 399 | /// blocks. This method is used to split exit blocks that have predecessors |
| 400 | /// outside of the loop. |
Chris Lattner | 59fb87d | 2004-04-18 22:27:10 +0000 | [diff] [blame] | 401 | BasicBlock *LoopSimplify::RewriteLoopExitBlock(Loop *L, BasicBlock *Exit) { |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 402 | SmallVector<BasicBlock*, 8> LoopBlocks; |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 403 | for (pred_iterator I = pred_begin(Exit), E = pred_end(Exit); I != E; ++I) { |
| 404 | BasicBlock *P = *I; |
| 405 | if (L->contains(P)) { |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 406 | // Don't do this if the loop is exited via an indirect branch. |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 407 | if (isa<IndirectBrInst>(P->getTerminator())) return 0; |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 408 | |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 409 | LoopBlocks.push_back(P); |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 410 | } |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 411 | } |
Chris Lattner | dbf3cd7 | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 412 | |
Chris Lattner | 7e7ad49 | 2003-02-27 22:31:07 +0000 | [diff] [blame] | 413 | assert(!LoopBlocks.empty() && "No edges coming in from outside the loop?"); |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 414 | BasicBlock *NewBB = SplitBlockPredecessors(Exit, &LoopBlocks[0], |
| 415 | LoopBlocks.size(), ".loopexit", |
| 416 | this); |
Chris Lattner | 7e7ad49 | 2003-02-27 22:31:07 +0000 | [diff] [blame] | 417 | |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 418 | DEBUG(dbgs() << "LoopSimplify: Creating dedicated exit block "; |
| 419 | WriteAsOperand(dbgs(), NewBB, false); |
| 420 | dbgs() << "\n"); |
| 421 | |
Chris Lattner | 59fb87d | 2004-04-18 22:27:10 +0000 | [diff] [blame] | 422 | return NewBB; |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 423 | } |
| 424 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 425 | /// AddBlockAndPredsToSet - Add the specified block, and all of its |
| 426 | /// predecessors, to the specified set, if it's not already in there. Stop |
| 427 | /// predecessor traversal when we reach StopBlock. |
Devang Patel | 58d7fbf | 2007-04-20 20:04:37 +0000 | [diff] [blame] | 428 | static void AddBlockAndPredsToSet(BasicBlock *InputBB, BasicBlock *StopBlock, |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 429 | std::set<BasicBlock*> &Blocks) { |
Devang Patel | 58d7fbf | 2007-04-20 20:04:37 +0000 | [diff] [blame] | 430 | std::vector<BasicBlock *> WorkList; |
| 431 | WorkList.push_back(InputBB); |
| 432 | do { |
| 433 | BasicBlock *BB = WorkList.back(); WorkList.pop_back(); |
| 434 | if (Blocks.insert(BB).second && BB != StopBlock) |
| 435 | // If BB is not already processed and it is not a stop block then |
| 436 | // insert its predecessor in the work list |
| 437 | for (pred_iterator I = pred_begin(BB), E = pred_end(BB); I != E; ++I) { |
| 438 | BasicBlock *WBB = *I; |
| 439 | WorkList.push_back(WBB); |
| 440 | } |
| 441 | } while(!WorkList.empty()); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 442 | } |
| 443 | |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 444 | /// FindPHIToPartitionLoops - The first part of loop-nestification is to find a |
| 445 | /// PHI node that tells us how to partition the loops. |
Devang Patel | dba2413 | 2007-06-08 01:50:32 +0000 | [diff] [blame] | 446 | static PHINode *FindPHIToPartitionLoops(Loop *L, DominatorTree *DT, |
Owen Anderson | ad19014 | 2007-04-09 22:54:50 +0000 | [diff] [blame] | 447 | AliasAnalysis *AA) { |
Alkis Evlogimenos | 200a360 | 2004-09-28 02:40:37 +0000 | [diff] [blame] | 448 | for (BasicBlock::iterator I = L->getHeader()->begin(); isa<PHINode>(I); ) { |
| 449 | PHINode *PN = cast<PHINode>(I); |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 450 | ++I; |
Dan Gohman | bccfc24 | 2009-09-03 15:34:35 +0000 | [diff] [blame] | 451 | if (Value *V = PN->hasConstantValue(DT)) { |
| 452 | // This is a degenerate PHI already, don't modify it! |
| 453 | PN->replaceAllUsesWith(V); |
| 454 | if (AA) AA->deleteValue(PN); |
| 455 | PN->eraseFromParent(); |
| 456 | continue; |
| 457 | } |
Chris Lattner | c30bda7 | 2004-10-17 21:22:38 +0000 | [diff] [blame] | 458 | |
| 459 | // Scan this PHI node looking for a use of the PHI node by itself. |
| 460 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) |
| 461 | if (PN->getIncomingValue(i) == PN && |
| 462 | L->contains(PN->getIncomingBlock(i))) |
| 463 | // We found something tasty to remove. |
| 464 | return PN; |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 465 | } |
| 466 | return 0; |
| 467 | } |
| 468 | |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 469 | // PlaceSplitBlockCarefully - If the block isn't already, move the new block to |
| 470 | // right after some 'outside block' block. This prevents the preheader from |
| 471 | // being placed inside the loop body, e.g. when the loop hasn't been rotated. |
| 472 | void LoopSimplify::PlaceSplitBlockCarefully(BasicBlock *NewBB, |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 473 | SmallVectorImpl<BasicBlock*> &SplitPreds, |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 474 | Loop *L) { |
| 475 | // Check to see if NewBB is already well placed. |
| 476 | Function::iterator BBI = NewBB; --BBI; |
| 477 | for (unsigned i = 0, e = SplitPreds.size(); i != e; ++i) { |
| 478 | if (&*BBI == SplitPreds[i]) |
| 479 | return; |
| 480 | } |
| 481 | |
| 482 | // If it isn't already after an outside block, move it after one. This is |
| 483 | // always good as it makes the uncond branch from the outside block into a |
| 484 | // fall-through. |
| 485 | |
| 486 | // Figure out *which* outside block to put this after. Prefer an outside |
| 487 | // block that neighbors a BB actually in the loop. |
| 488 | BasicBlock *FoundBB = 0; |
| 489 | for (unsigned i = 0, e = SplitPreds.size(); i != e; ++i) { |
| 490 | Function::iterator BBI = SplitPreds[i]; |
| 491 | if (++BBI != NewBB->getParent()->end() && |
| 492 | L->contains(BBI)) { |
| 493 | FoundBB = SplitPreds[i]; |
| 494 | break; |
| 495 | } |
| 496 | } |
| 497 | |
| 498 | // If our heuristic for a *good* bb to place this after doesn't find |
| 499 | // anything, just pick something. It's likely better than leaving it within |
| 500 | // the loop. |
| 501 | if (!FoundBB) |
| 502 | FoundBB = SplitPreds[0]; |
| 503 | NewBB->moveAfter(FoundBB); |
| 504 | } |
| 505 | |
| 506 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 507 | /// SeparateNestedLoop - If this loop has multiple backedges, try to pull one of |
| 508 | /// them out into a nested loop. This is important for code that looks like |
| 509 | /// this: |
| 510 | /// |
| 511 | /// Loop: |
| 512 | /// ... |
| 513 | /// br cond, Loop, Next |
| 514 | /// ... |
| 515 | /// br cond2, Loop, Out |
| 516 | /// |
| 517 | /// To identify this common case, we look at the PHI nodes in the header of the |
| 518 | /// loop. PHI nodes with unchanging values on one backedge correspond to values |
| 519 | /// that change in the "outer" loop, but not in the "inner" loop. |
| 520 | /// |
| 521 | /// If we are able to separate out a loop, return the new outer loop that was |
| 522 | /// created. |
| 523 | /// |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 524 | Loop *LoopSimplify::SeparateNestedLoop(Loop *L, LPPassManager &LPM) { |
Devang Patel | dba2413 | 2007-06-08 01:50:32 +0000 | [diff] [blame] | 525 | PHINode *PN = FindPHIToPartitionLoops(L, DT, AA); |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 526 | if (PN == 0) return 0; // No known way to partition. |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 527 | |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 528 | // Pull out all predecessors that have varying values in the loop. This |
| 529 | // handles the case when a PHI node has multiple instances of itself as |
| 530 | // arguments. |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 531 | SmallVector<BasicBlock*, 8> OuterLoopPreds; |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 532 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) |
| 533 | if (PN->getIncomingValue(i) != PN || |
Dan Gohman | a58a049 | 2009-11-20 20:51:18 +0000 | [diff] [blame] | 534 | !L->contains(PN->getIncomingBlock(i))) { |
| 535 | // We can't split indirectbr edges. |
| 536 | if (isa<IndirectBrInst>(PN->getIncomingBlock(i)->getTerminator())) |
| 537 | return 0; |
| 538 | |
Chris Lattner | 1f62f82 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 539 | OuterLoopPreds.push_back(PN->getIncomingBlock(i)); |
Dan Gohman | a58a049 | 2009-11-20 20:51:18 +0000 | [diff] [blame] | 540 | } |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 541 | |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 542 | DEBUG(dbgs() << "LoopSimplify: Splitting out a new outer loop\n"); |
| 543 | |
Dan Gohman | ffa75cd | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 544 | // If ScalarEvolution is around and knows anything about values in |
| 545 | // this loop, tell it to forget them, because we're about to |
| 546 | // substantially change it. |
| 547 | if (SE) |
| 548 | SE->forgetLoop(L); |
| 549 | |
Chris Lattner | 4b66242 | 2004-04-13 16:23:25 +0000 | [diff] [blame] | 550 | BasicBlock *Header = L->getHeader(); |
Chris Lattner | 54b9c3b | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 551 | BasicBlock *NewBB = SplitBlockPredecessors(Header, &OuterLoopPreds[0], |
| 552 | OuterLoopPreds.size(), |
| 553 | ".outer", this); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 554 | |
Chris Lattner | 120fce5 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 555 | // Make sure that NewBB is put someplace intelligent, which doesn't mess up |
| 556 | // code layout too horribly. |
| 557 | PlaceSplitBlockCarefully(NewBB, OuterLoopPreds, L); |
| 558 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 559 | // Create the new outer loop. |
| 560 | Loop *NewOuter = new Loop(); |
| 561 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 562 | // Change the parent loop to use the outer loop as its child now. |
| 563 | if (Loop *Parent = L->getParentLoop()) |
| 564 | Parent->replaceChildLoopWith(L, NewOuter); |
| 565 | else |
Chris Lattner | c27e056 | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 566 | LI->changeTopLevelLoop(L, NewOuter); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 567 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 568 | // L is now a subloop of our outer loop. |
| 569 | NewOuter->addChildLoop(L); |
| 570 | |
Dan Gohman | d84db11 | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 571 | // Add the new loop to the pass manager queue. |
| 572 | LPM.insertLoopIntoQueue(NewOuter); |
| 573 | |
Dan Gohman | 9b78763 | 2008-06-22 20:18:58 +0000 | [diff] [blame] | 574 | for (Loop::block_iterator I = L->block_begin(), E = L->block_end(); |
| 575 | I != E; ++I) |
| 576 | NewOuter->addBlockEntry(*I); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 577 | |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 578 | // Now reset the header in L, which had been moved by |
| 579 | // SplitBlockPredecessors for the outer loop. |
| 580 | L->moveToHeader(Header); |
| 581 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 582 | // Determine which blocks should stay in L and which should be moved out to |
| 583 | // the Outer loop now. |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 584 | std::set<BasicBlock*> BlocksInL; |
Gabor Greif | 9672414 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 585 | for (pred_iterator PI=pred_begin(Header), E = pred_end(Header); PI!=E; ++PI) { |
| 586 | BasicBlock *P = *PI; |
| 587 | if (DT->dominates(Header, P)) |
| 588 | AddBlockAndPredsToSet(P, Header, BlocksInL); |
| 589 | } |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 590 | |
| 591 | // Scan all of the loop children of L, moving them to OuterLoop if they are |
| 592 | // not part of the inner loop. |
David Greene | c08fa28 | 2007-06-29 02:53:16 +0000 | [diff] [blame] | 593 | const std::vector<Loop*> &SubLoops = L->getSubLoops(); |
| 594 | for (size_t I = 0; I != SubLoops.size(); ) |
| 595 | if (BlocksInL.count(SubLoops[I]->getHeader())) |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 596 | ++I; // Loop remains in L |
| 597 | else |
David Greene | c08fa28 | 2007-06-29 02:53:16 +0000 | [diff] [blame] | 598 | NewOuter->addChildLoop(L->removeChildLoop(SubLoops.begin() + I)); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 599 | |
| 600 | // Now that we know which blocks are in L and which need to be moved to |
| 601 | // OuterLoop, move any blocks that need it. |
| 602 | for (unsigned i = 0; i != L->getBlocks().size(); ++i) { |
| 603 | BasicBlock *BB = L->getBlocks()[i]; |
| 604 | if (!BlocksInL.count(BB)) { |
| 605 | // Move this block to the parent, updating the exit blocks sets |
| 606 | L->removeBlockFromLoop(BB); |
Chris Lattner | c27e056 | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 607 | if ((*LI)[BB] == L) |
| 608 | LI->changeLoopFor(BB, NewOuter); |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 609 | --i; |
| 610 | } |
| 611 | } |
| 612 | |
Chris Lattner | 529b28d | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 613 | return NewOuter; |
| 614 | } |
| 615 | |
| 616 | |
| 617 | |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 618 | /// InsertUniqueBackedgeBlock - This method is called when the specified loop |
| 619 | /// has more than one backedge in it. If this occurs, revector all of these |
| 620 | /// backedges to target a new basic block and have that block branch to the loop |
| 621 | /// header. This ensures that loops have exactly one backedge. |
| 622 | /// |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 623 | BasicBlock * |
| 624 | LoopSimplify::InsertUniqueBackedgeBlock(Loop *L, BasicBlock *Preheader) { |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 625 | assert(L->getNumBackEdges() > 1 && "Must have > 1 backedge!"); |
| 626 | |
| 627 | // Get information about the loop |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 628 | BasicBlock *Header = L->getHeader(); |
| 629 | Function *F = Header->getParent(); |
| 630 | |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 631 | // Unique backedge insertion currently depends on having a preheader. |
| 632 | if (!Preheader) |
| 633 | return 0; |
| 634 | |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 635 | // Figure out which basic blocks contain back-edges to the loop header. |
| 636 | std::vector<BasicBlock*> BackedgeBlocks; |
Gabor Greif | bf2eefd | 2010-07-09 16:26:41 +0000 | [diff] [blame] | 637 | for (pred_iterator I = pred_begin(Header), E = pred_end(Header); I != E; ++I){ |
| 638 | BasicBlock *P = *I; |
Dan Gohman | c2f4006 | 2010-08-14 00:43:09 +0000 | [diff] [blame] | 639 | |
| 640 | // Indirectbr edges cannot be split, so we must fail if we find one. |
| 641 | if (isa<IndirectBrInst>(P->getTerminator())) |
| 642 | return 0; |
| 643 | |
Gabor Greif | bf2eefd | 2010-07-09 16:26:41 +0000 | [diff] [blame] | 644 | if (P != Preheader) BackedgeBlocks.push_back(P); |
| 645 | } |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 646 | |
| 647 | // Create and insert the new backedge block... |
Owen Anderson | 1d0be15 | 2009-08-13 21:58:54 +0000 | [diff] [blame] | 648 | BasicBlock *BEBlock = BasicBlock::Create(Header->getContext(), |
| 649 | Header->getName()+".backedge", F); |
Gabor Greif | 051a950 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 650 | BranchInst *BETerminator = BranchInst::Create(Header, BEBlock); |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 651 | |
Dan Gohman | c5e49c6 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 652 | DEBUG(dbgs() << "LoopSimplify: Inserting unique backedge block "; |
| 653 | WriteAsOperand(dbgs(), BEBlock, false); |
| 654 | dbgs() << "\n"); |
| 655 | |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 656 | // Move the new backedge block to right after the last backedge block. |
| 657 | Function::iterator InsertPos = BackedgeBlocks.back(); ++InsertPos; |
| 658 | F->getBasicBlockList().splice(InsertPos, F->getBasicBlockList(), BEBlock); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 659 | |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 660 | // Now that the block has been inserted into the function, create PHI nodes in |
| 661 | // the backedge block which correspond to any PHI nodes in the header block. |
Alkis Evlogimenos | 200a360 | 2004-09-28 02:40:37 +0000 | [diff] [blame] | 662 | for (BasicBlock::iterator I = Header->begin(); isa<PHINode>(I); ++I) { |
| 663 | PHINode *PN = cast<PHINode>(I); |
Gabor Greif | 051a950 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 664 | PHINode *NewPN = PHINode::Create(PN->getType(), PN->getName()+".be", |
| 665 | BETerminator); |
Chris Lattner | 5551706 | 2005-01-29 00:39:08 +0000 | [diff] [blame] | 666 | NewPN->reserveOperandSpace(BackedgeBlocks.size()); |
Chris Lattner | cec5b88 | 2005-03-25 06:37:22 +0000 | [diff] [blame] | 667 | if (AA) AA->copyValue(PN, NewPN); |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 668 | |
| 669 | // Loop over the PHI node, moving all entries except the one for the |
| 670 | // preheader over to the new PHI node. |
| 671 | unsigned PreheaderIdx = ~0U; |
| 672 | bool HasUniqueIncomingValue = true; |
| 673 | Value *UniqueValue = 0; |
| 674 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 675 | BasicBlock *IBB = PN->getIncomingBlock(i); |
| 676 | Value *IV = PN->getIncomingValue(i); |
| 677 | if (IBB == Preheader) { |
| 678 | PreheaderIdx = i; |
| 679 | } else { |
| 680 | NewPN->addIncoming(IV, IBB); |
| 681 | if (HasUniqueIncomingValue) { |
| 682 | if (UniqueValue == 0) |
| 683 | UniqueValue = IV; |
| 684 | else if (UniqueValue != IV) |
| 685 | HasUniqueIncomingValue = false; |
| 686 | } |
| 687 | } |
| 688 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 689 | |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 690 | // Delete all of the incoming values from the old PN except the preheader's |
| 691 | assert(PreheaderIdx != ~0U && "PHI has no preheader entry??"); |
| 692 | if (PreheaderIdx != 0) { |
| 693 | PN->setIncomingValue(0, PN->getIncomingValue(PreheaderIdx)); |
| 694 | PN->setIncomingBlock(0, PN->getIncomingBlock(PreheaderIdx)); |
| 695 | } |
Chris Lattner | 5551706 | 2005-01-29 00:39:08 +0000 | [diff] [blame] | 696 | // Nuke all entries except the zero'th. |
| 697 | for (unsigned i = 0, e = PN->getNumIncomingValues()-1; i != e; ++i) |
| 698 | PN->removeIncomingValue(e-i, false); |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 699 | |
| 700 | // Finally, add the newly constructed PHI node as the entry for the BEBlock. |
| 701 | PN->addIncoming(NewPN, BEBlock); |
| 702 | |
| 703 | // As an optimization, if all incoming values in the new PhiNode (which is a |
| 704 | // subset of the incoming values of the old PHI node) have the same value, |
| 705 | // eliminate the PHI Node. |
| 706 | if (HasUniqueIncomingValue) { |
| 707 | NewPN->replaceAllUsesWith(UniqueValue); |
Chris Lattner | cec5b88 | 2005-03-25 06:37:22 +0000 | [diff] [blame] | 708 | if (AA) AA->deleteValue(NewPN); |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 709 | BEBlock->getInstList().erase(NewPN); |
| 710 | } |
| 711 | } |
| 712 | |
| 713 | // Now that all of the PHI nodes have been inserted and adjusted, modify the |
Nick Lewycky | 280a6e6 | 2008-04-25 16:53:59 +0000 | [diff] [blame] | 714 | // backedge blocks to just to the BEBlock instead of the header. |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 715 | for (unsigned i = 0, e = BackedgeBlocks.size(); i != e; ++i) { |
| 716 | TerminatorInst *TI = BackedgeBlocks[i]->getTerminator(); |
| 717 | for (unsigned Op = 0, e = TI->getNumSuccessors(); Op != e; ++Op) |
| 718 | if (TI->getSuccessor(Op) == Header) |
| 719 | TI->setSuccessor(Op, BEBlock); |
| 720 | } |
| 721 | |
| 722 | //===--- Update all analyses which we must preserve now -----------------===// |
| 723 | |
| 724 | // Update Loop Information - we know that this block is now in the current |
| 725 | // loop and all parent loops. |
Owen Anderson | d735ee8 | 2007-11-27 03:43:35 +0000 | [diff] [blame] | 726 | L->addBasicBlockToLoop(BEBlock, LI->getBase()); |
Chris Lattner | 2ab6a73 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 727 | |
Devang Patel | 0e7f728 | 2007-06-21 17:23:45 +0000 | [diff] [blame] | 728 | // Update dominator information |
| 729 | DT->splitBlock(BEBlock); |
Duncan Sands | 1465d61 | 2009-01-28 13:14:17 +0000 | [diff] [blame] | 730 | if (DominanceFrontier *DF = getAnalysisIfAvailable<DominanceFrontier>()) |
Devang Patel | 0e7f728 | 2007-06-21 17:23:45 +0000 | [diff] [blame] | 731 | DF->splitBlock(BEBlock); |
Dan Gohman | f4e82d1 | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 732 | |
| 733 | return BEBlock; |
| 734 | } |
| 735 | |
| 736 | void LoopSimplify::verifyAnalysis() const { |
| 737 | // It used to be possible to just assert L->isLoopSimplifyForm(), however |
| 738 | // with the introduction of indirectbr, there are now cases where it's |
| 739 | // not possible to transform a loop as necessary. We can at least check |
| 740 | // that there is an indirectbr near any time there's trouble. |
| 741 | |
| 742 | // Indirectbr can interfere with preheader and unique backedge insertion. |
| 743 | if (!L->getLoopPreheader() || !L->getLoopLatch()) { |
| 744 | bool HasIndBrPred = false; |
| 745 | for (pred_iterator PI = pred_begin(L->getHeader()), |
| 746 | PE = pred_end(L->getHeader()); PI != PE; ++PI) |
| 747 | if (isa<IndirectBrInst>((*PI)->getTerminator())) { |
| 748 | HasIndBrPred = true; |
| 749 | break; |
| 750 | } |
| 751 | assert(HasIndBrPred && |
| 752 | "LoopSimplify has no excuse for missing loop header info!"); |
| 753 | } |
| 754 | |
| 755 | // Indirectbr can interfere with exit block canonicalization. |
| 756 | if (!L->hasDedicatedExits()) { |
| 757 | bool HasIndBrExiting = false; |
| 758 | SmallVector<BasicBlock*, 8> ExitingBlocks; |
| 759 | L->getExitingBlocks(ExitingBlocks); |
| 760 | for (unsigned i = 0, e = ExitingBlocks.size(); i != e; ++i) |
| 761 | if (isa<IndirectBrInst>((ExitingBlocks[i])->getTerminator())) { |
| 762 | HasIndBrExiting = true; |
| 763 | break; |
| 764 | } |
| 765 | assert(HasIndBrExiting && |
| 766 | "LoopSimplify has no excuse for missing exit block info!"); |
| 767 | } |
Chris Lattner | 38acf9e | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 768 | } |