Chris Lattner | 55d4788 | 2003-10-12 21:44:18 +0000 | [diff] [blame] | 1 | //===- LoopSimplify.cpp - Loop Canonicalization Pass ----------------------===// |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 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 | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | 154e4d5 | 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 | 650096a | 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 | 7710f2f | 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 | 650096a | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 23 | // |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 24 | // This pass also guarantees that loops will have exactly one backedge. |
| 25 | // |
Dan Gohman | 1ef784d | 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 | 650096a | 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 | 154e4d5 | 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 | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 37 | // |
| 38 | //===----------------------------------------------------------------------===// |
| 39 | |
| 40 | #include "llvm/Transforms/Scalar.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 41 | #include "llvm/ADT/DepthFirstIterator.h" |
| 42 | #include "llvm/ADT/SetOperations.h" |
| 43 | #include "llvm/ADT/SetVector.h" |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 44 | #include "llvm/ADT/SmallVector.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 45 | #include "llvm/ADT/Statistic.h" |
Chris Lattner | 514e843 | 2005-03-25 06:37:22 +0000 | [diff] [blame] | 46 | #include "llvm/Analysis/AliasAnalysis.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 47 | #include "llvm/Analysis/BasicAliasAnalysis.h" |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 48 | #include "llvm/Analysis/AssumptionCache.h" |
Benjamin Kramer | 7736085 | 2012-10-26 17:40:50 +0000 | [diff] [blame] | 49 | #include "llvm/Analysis/DependenceAnalysis.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 50 | #include "llvm/Analysis/GlobalsModRef.h" |
Duncan Sands | 6370495 | 2010-11-16 17:41:24 +0000 | [diff] [blame] | 51 | #include "llvm/Analysis/InstructionSimplify.h" |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 52 | #include "llvm/Analysis/LoopInfo.h" |
Duncan Sands | 6370495 | 2010-11-16 17:41:24 +0000 | [diff] [blame] | 53 | #include "llvm/Analysis/ScalarEvolution.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 54 | #include "llvm/Analysis/ScalarEvolutionAliasAnalysis.h" |
Chandler Carruth | 1305dc3 | 2014-03-04 11:45:46 +0000 | [diff] [blame] | 55 | #include "llvm/IR/CFG.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 56 | #include "llvm/IR/Constants.h" |
Hal Finkel | a995f92 | 2014-07-10 14:41:31 +0000 | [diff] [blame] | 57 | #include "llvm/IR/DataLayout.h" |
Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 58 | #include "llvm/IR/Dominators.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 59 | #include "llvm/IR/Function.h" |
| 60 | #include "llvm/IR/Instructions.h" |
| 61 | #include "llvm/IR/IntrinsicInst.h" |
| 62 | #include "llvm/IR/LLVMContext.h" |
Mehdi Amini | 46a4355 | 2015-03-04 18:43:29 +0000 | [diff] [blame] | 63 | #include "llvm/IR/Module.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 64 | #include "llvm/IR/Type.h" |
Dan Gohman | 39917c7 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 65 | #include "llvm/Support/Debug.h" |
Benjamin Kramer | 799003b | 2015-03-23 19:32:43 +0000 | [diff] [blame] | 66 | #include "llvm/Support/raw_ostream.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 67 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
| 68 | #include "llvm/Transforms/Utils/Local.h" |
Hal Finkel | a969df8 | 2013-05-20 20:46:30 +0000 | [diff] [blame] | 69 | #include "llvm/Transforms/Utils/LoopUtils.h" |
Chris Lattner | 7710f2f | 2003-12-10 17:20:35 +0000 | [diff] [blame] | 70 | using namespace llvm; |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 71 | |
Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 72 | #define DEBUG_TYPE "loop-simplify" |
| 73 | |
Chris Lattner | 45f966d | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 74 | STATISTIC(NumInserted, "Number of pre-header or exit blocks inserted"); |
| 75 | STATISTIC(NumNested , "Number of nested loops split out"); |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 76 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 77 | // If the block isn't already, move the new block to right after some 'outside |
| 78 | // block' block. This prevents the preheader from being placed inside the loop |
| 79 | // body, e.g. when the loop hasn't been rotated. |
| 80 | static void placeSplitBlockCarefully(BasicBlock *NewBB, |
| 81 | SmallVectorImpl<BasicBlock *> &SplitPreds, |
| 82 | Loop *L) { |
| 83 | // Check to see if NewBB is already well placed. |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 84 | Function::iterator BBI = --NewBB->getIterator(); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 85 | for (unsigned i = 0, e = SplitPreds.size(); i != e; ++i) { |
| 86 | if (&*BBI == SplitPreds[i]) |
| 87 | return; |
| 88 | } |
Devang Patel | 09f162c | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 89 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 90 | // If it isn't already after an outside block, move it after one. This is |
| 91 | // always good as it makes the uncond branch from the outside block into a |
| 92 | // fall-through. |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 93 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 94 | // Figure out *which* outside block to put this after. Prefer an outside |
| 95 | // block that neighbors a BB actually in the loop. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 96 | BasicBlock *FoundBB = nullptr; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 97 | for (unsigned i = 0, e = SplitPreds.size(); i != e; ++i) { |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 98 | Function::iterator BBI = SplitPreds[i]->getIterator(); |
| 99 | if (++BBI != NewBB->getParent()->end() && L->contains(&*BBI)) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 100 | FoundBB = SplitPreds[i]; |
| 101 | break; |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 102 | } |
| 103 | } |
Chris Lattner | d078812 | 2004-03-14 03:59:22 +0000 | [diff] [blame] | 104 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 105 | // If our heuristic for a *good* bb to place this after doesn't find |
| 106 | // anything, just pick something. It's likely better than leaving it within |
| 107 | // the loop. |
| 108 | if (!FoundBB) |
| 109 | FoundBB = SplitPreds[0]; |
| 110 | NewBB->moveAfter(FoundBB); |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 111 | } |
| 112 | |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 113 | /// InsertPreheaderForLoop - Once we discover that a loop doesn't have a |
| 114 | /// preheader, this method is called to insert one. This method has two phases: |
| 115 | /// preheader insertion and analysis updating. |
| 116 | /// |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 117 | BasicBlock *llvm::InsertPreheaderForLoop(Loop *L, DominatorTree *DT, |
| 118 | LoopInfo *LI, bool PreserveLCSSA) { |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 119 | BasicBlock *Header = L->getHeader(); |
| 120 | |
| 121 | // Compute the set of predecessors of the loop that are not in the loop. |
Chris Lattner | a5b1170 | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 122 | SmallVector<BasicBlock*, 8> OutsideBlocks; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 123 | for (pred_iterator PI = pred_begin(Header), PE = pred_end(Header); |
| 124 | PI != PE; ++PI) { |
| 125 | BasicBlock *P = *PI; |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 126 | if (!L->contains(P)) { // Coming in from outside the loop? |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 127 | // If the loop is branched to from an indirect branch, we won't |
| 128 | // be able to fully transform the loop, because it prohibits |
| 129 | // edge splitting. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 130 | if (isa<IndirectBrInst>(P->getTerminator())) return nullptr; |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 131 | |
| 132 | // Keep track of it. |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 133 | OutsideBlocks.push_back(P); |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 134 | } |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 135 | } |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 136 | |
Chris Lattner | 608cd05 | 2006-09-23 07:40:52 +0000 | [diff] [blame] | 137 | // Split out the loop pre-header. |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 138 | BasicBlock *PreheaderBB; |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 139 | PreheaderBB = SplitBlockPredecessors(Header, OutsideBlocks, ".preheader", DT, |
| 140 | LI, PreserveLCSSA); |
David Majnemer | 654e130 | 2015-07-31 17:58:14 +0000 | [diff] [blame] | 141 | if (!PreheaderBB) |
| 142 | return nullptr; |
Chris Lattner | f2d9f94 | 2003-02-27 22:48:57 +0000 | [diff] [blame] | 143 | |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 144 | DEBUG(dbgs() << "LoopSimplify: Creating pre-header " |
| 145 | << PreheaderBB->getName() << "\n"); |
Dan Gohman | 39917c7 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 146 | |
Chris Lattner | 6bd6da4 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 147 | // Make sure that NewBB is put someplace intelligent, which doesn't mess up |
| 148 | // code layout too horribly. |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 149 | placeSplitBlockCarefully(PreheaderBB, OutsideBlocks, L); |
Dan Gohman | 4d6149f | 2009-07-14 01:37:59 +0000 | [diff] [blame] | 150 | |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 151 | return PreheaderBB; |
Chris Lattner | 650096a | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 152 | } |
| 153 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 154 | /// \brief Ensure that the loop preheader dominates all exit blocks. |
| 155 | /// |
| 156 | /// This method is used to split exit blocks that have predecessors outside of |
| 157 | /// the loop. |
Chandler Carruth | b5797b6 | 2015-01-18 09:21:15 +0000 | [diff] [blame] | 158 | static BasicBlock *rewriteLoopExitBlock(Loop *L, BasicBlock *Exit, |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 159 | DominatorTree *DT, LoopInfo *LI, |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 160 | bool PreserveLCSSA) { |
Chris Lattner | a5b1170 | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 161 | SmallVector<BasicBlock*, 8> LoopBlocks; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 162 | for (pred_iterator I = pred_begin(Exit), E = pred_end(Exit); I != E; ++I) { |
| 163 | BasicBlock *P = *I; |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 164 | if (L->contains(P)) { |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 165 | // Don't do this if the loop is exited via an indirect branch. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 166 | if (isa<IndirectBrInst>(P->getTerminator())) return nullptr; |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 167 | |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 168 | LoopBlocks.push_back(P); |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 169 | } |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 170 | } |
Chris Lattner | 650096a | 2003-02-27 20:27:08 +0000 | [diff] [blame] | 171 | |
Chris Lattner | 10b2b05 | 2003-02-27 22:31:07 +0000 | [diff] [blame] | 172 | assert(!LoopBlocks.empty() && "No edges coming in from outside the loop?"); |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 173 | BasicBlock *NewExitBB = nullptr; |
Bill Wendling | c61f765 | 2011-08-19 00:09:22 +0000 | [diff] [blame] | 174 | |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 175 | NewExitBB = SplitBlockPredecessors(Exit, LoopBlocks, ".loopexit", DT, LI, |
| 176 | PreserveLCSSA); |
David Majnemer | 654e130 | 2015-07-31 17:58:14 +0000 | [diff] [blame] | 177 | if (!NewExitBB) |
| 178 | return nullptr; |
Chris Lattner | 10b2b05 | 2003-02-27 22:31:07 +0000 | [diff] [blame] | 179 | |
Chris Lattner | bf0aa92 | 2011-01-02 22:09:33 +0000 | [diff] [blame] | 180 | DEBUG(dbgs() << "LoopSimplify: Creating dedicated exit block " |
Bill Wendling | c61f765 | 2011-08-19 00:09:22 +0000 | [diff] [blame] | 181 | << NewExitBB->getName() << "\n"); |
| 182 | return NewExitBB; |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 183 | } |
| 184 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 185 | /// Add the specified block, and all of its predecessors, to the specified set, |
| 186 | /// if it's not already in there. Stop predecessor traversal when we reach |
| 187 | /// StopBlock. |
| 188 | static void addBlockAndPredsToSet(BasicBlock *InputBB, BasicBlock *StopBlock, |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 189 | std::set<BasicBlock*> &Blocks) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 190 | SmallVector<BasicBlock *, 8> Worklist; |
| 191 | Worklist.push_back(InputBB); |
Devang Patel | 83a3adc | 2007-04-20 20:04:37 +0000 | [diff] [blame] | 192 | do { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 193 | BasicBlock *BB = Worklist.pop_back_val(); |
Devang Patel | 83a3adc | 2007-04-20 20:04:37 +0000 | [diff] [blame] | 194 | if (Blocks.insert(BB).second && BB != StopBlock) |
| 195 | // If BB is not already processed and it is not a stop block then |
| 196 | // insert its predecessor in the work list |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 197 | for (pred_iterator I = pred_begin(BB), E = pred_end(BB); I != E; ++I) { |
| 198 | BasicBlock *WBB = *I; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 199 | Worklist.push_back(WBB); |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 200 | } |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 201 | } while (!Worklist.empty()); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 202 | } |
| 203 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 204 | /// \brief The first part of loop-nestification is to find a PHI node that tells |
| 205 | /// us how to partition the loops. |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 206 | static PHINode *findPHIToPartitionLoops(Loop *L, DominatorTree *DT, |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 207 | AssumptionCache *AC) { |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 208 | const DataLayout &DL = L->getHeader()->getModule()->getDataLayout(); |
Alkis Evlogimenos | 3ce42ec | 2004-09-28 02:40:37 +0000 | [diff] [blame] | 209 | for (BasicBlock::iterator I = L->getHeader()->begin(); isa<PHINode>(I); ) { |
| 210 | PHINode *PN = cast<PHINode>(I); |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 211 | ++I; |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 212 | if (Value *V = SimplifyInstruction(PN, DL, nullptr, DT, AC)) { |
Duncan Sands | 64f1c0d | 2011-01-02 13:38:21 +0000 | [diff] [blame] | 213 | // This is a degenerate PHI already, don't modify it! |
| 214 | PN->replaceAllUsesWith(V); |
Duncan Sands | 64f1c0d | 2011-01-02 13:38:21 +0000 | [diff] [blame] | 215 | PN->eraseFromParent(); |
| 216 | continue; |
| 217 | } |
Chris Lattner | e29d634 | 2004-10-17 21:22:38 +0000 | [diff] [blame] | 218 | |
| 219 | // Scan this PHI node looking for a use of the PHI node by itself. |
| 220 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) |
| 221 | if (PN->getIncomingValue(i) == PN && |
| 222 | L->contains(PN->getIncomingBlock(i))) |
| 223 | // We found something tasty to remove. |
| 224 | return PN; |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 225 | } |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 226 | return nullptr; |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 227 | } |
| 228 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 229 | /// \brief If this loop has multiple backedges, try to pull one of them out into |
| 230 | /// a nested loop. |
| 231 | /// |
| 232 | /// This is important for code that looks like |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 233 | /// this: |
| 234 | /// |
| 235 | /// Loop: |
| 236 | /// ... |
| 237 | /// br cond, Loop, Next |
| 238 | /// ... |
| 239 | /// br cond2, Loop, Out |
| 240 | /// |
| 241 | /// To identify this common case, we look at the PHI nodes in the header of the |
| 242 | /// loop. PHI nodes with unchanging values on one backedge correspond to values |
| 243 | /// that change in the "outer" loop, but not in the "inner" loop. |
| 244 | /// |
| 245 | /// If we are able to separate out a loop, return the new outer loop that was |
| 246 | /// created. |
| 247 | /// |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 248 | static Loop *separateNestedLoop(Loop *L, BasicBlock *Preheader, |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 249 | DominatorTree *DT, LoopInfo *LI, |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 250 | ScalarEvolution *SE, bool PreserveLCSSA, |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 251 | AssumptionCache *AC) { |
Andrew Trick | f771101 | 2012-03-20 21:24:52 +0000 | [diff] [blame] | 252 | // Don't try to separate loops without a preheader. |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 253 | if (!Preheader) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 254 | return nullptr; |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 255 | |
| 256 | // The header is not a landing pad; preheader insertion should ensure this. |
David Majnemer | 654e130 | 2015-07-31 17:58:14 +0000 | [diff] [blame] | 257 | BasicBlock *Header = L->getHeader(); |
David Majnemer | eb518bd | 2015-08-04 08:21:40 +0000 | [diff] [blame] | 258 | assert(!Header->isEHPad() && "Can't insert backedge to EH pad"); |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 259 | |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 260 | PHINode *PN = findPHIToPartitionLoops(L, DT, AC); |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 261 | if (!PN) return nullptr; // No known way to partition. |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 262 | |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 263 | // Pull out all predecessors that have varying values in the loop. This |
| 264 | // handles the case when a PHI node has multiple instances of itself as |
| 265 | // arguments. |
Chris Lattner | a5b1170 | 2008-04-21 01:28:02 +0000 | [diff] [blame] | 266 | SmallVector<BasicBlock*, 8> OuterLoopPreds; |
Andrew Trick | f771101 | 2012-03-20 21:24:52 +0000 | [diff] [blame] | 267 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 268 | if (PN->getIncomingValue(i) != PN || |
Andrew Trick | f771101 | 2012-03-20 21:24:52 +0000 | [diff] [blame] | 269 | !L->contains(PN->getIncomingBlock(i))) { |
| 270 | // We can't split indirectbr edges. |
| 271 | if (isa<IndirectBrInst>(PN->getIncomingBlock(i)->getTerminator())) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 272 | return nullptr; |
Chris Lattner | a6e2281 | 2004-04-13 15:21:18 +0000 | [diff] [blame] | 273 | OuterLoopPreds.push_back(PN->getIncomingBlock(i)); |
Andrew Trick | f771101 | 2012-03-20 21:24:52 +0000 | [diff] [blame] | 274 | } |
| 275 | } |
Dan Gohman | 39917c7 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 276 | DEBUG(dbgs() << "LoopSimplify: Splitting out a new outer loop\n"); |
| 277 | |
Dan Gohman | 487e250 | 2010-09-04 02:42:48 +0000 | [diff] [blame] | 278 | // If ScalarEvolution is around and knows anything about values in |
| 279 | // this loop, tell it to forget them, because we're about to |
| 280 | // substantially change it. |
| 281 | if (SE) |
| 282 | SE->forgetLoop(L); |
| 283 | |
Chandler Carruth | b5797b6 | 2015-01-18 09:21:15 +0000 | [diff] [blame] | 284 | BasicBlock *NewBB = SplitBlockPredecessors(Header, OuterLoopPreds, ".outer", |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 285 | DT, LI, PreserveLCSSA); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 286 | |
Chris Lattner | 6bd6da4 | 2006-09-23 08:19:21 +0000 | [diff] [blame] | 287 | // Make sure that NewBB is put someplace intelligent, which doesn't mess up |
| 288 | // code layout too horribly. |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 289 | placeSplitBlockCarefully(NewBB, OuterLoopPreds, L); |
Andrew Trick | 9d8c2af | 2011-08-03 18:28:21 +0000 | [diff] [blame] | 290 | |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 291 | // Create the new outer loop. |
| 292 | Loop *NewOuter = new Loop(); |
| 293 | |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 294 | // Change the parent loop to use the outer loop as its child now. |
| 295 | if (Loop *Parent = L->getParentLoop()) |
| 296 | Parent->replaceChildLoopWith(L, NewOuter); |
| 297 | else |
Chris Lattner | cffbbee | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 298 | LI->changeTopLevelLoop(L, NewOuter); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 299 | |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 300 | // L is now a subloop of our outer loop. |
| 301 | NewOuter->addChildLoop(L); |
| 302 | |
Dan Gohman | 9007107 | 2008-06-22 20:18:58 +0000 | [diff] [blame] | 303 | for (Loop::block_iterator I = L->block_begin(), E = L->block_end(); |
| 304 | I != E; ++I) |
| 305 | NewOuter->addBlockEntry(*I); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 306 | |
Dan Gohman | 3ddbc24 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 307 | // Now reset the header in L, which had been moved by |
| 308 | // SplitBlockPredecessors for the outer loop. |
| 309 | L->moveToHeader(Header); |
| 310 | |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 311 | // Determine which blocks should stay in L and which should be moved out to |
| 312 | // the Outer loop now. |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 313 | std::set<BasicBlock*> BlocksInL; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 314 | for (pred_iterator PI=pred_begin(Header), E = pred_end(Header); PI!=E; ++PI) { |
| 315 | BasicBlock *P = *PI; |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 316 | if (DT->dominates(Header, P)) |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 317 | addBlockAndPredsToSet(P, Header, BlocksInL); |
Gabor Greif | f0821f3 | 2010-07-09 14:02:13 +0000 | [diff] [blame] | 318 | } |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 319 | |
| 320 | // Scan all of the loop children of L, moving them to OuterLoop if they are |
| 321 | // not part of the inner loop. |
David Greene | 1e2a120 | 2007-06-29 02:53:16 +0000 | [diff] [blame] | 322 | const std::vector<Loop*> &SubLoops = L->getSubLoops(); |
| 323 | for (size_t I = 0; I != SubLoops.size(); ) |
| 324 | if (BlocksInL.count(SubLoops[I]->getHeader())) |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 325 | ++I; // Loop remains in L |
| 326 | else |
David Greene | 1e2a120 | 2007-06-29 02:53:16 +0000 | [diff] [blame] | 327 | NewOuter->addChildLoop(L->removeChildLoop(SubLoops.begin() + I)); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 328 | |
| 329 | // Now that we know which blocks are in L and which need to be moved to |
| 330 | // OuterLoop, move any blocks that need it. |
| 331 | for (unsigned i = 0; i != L->getBlocks().size(); ++i) { |
| 332 | BasicBlock *BB = L->getBlocks()[i]; |
| 333 | if (!BlocksInL.count(BB)) { |
| 334 | // Move this block to the parent, updating the exit blocks sets |
| 335 | L->removeBlockFromLoop(BB); |
Chris Lattner | cffbbee | 2006-02-14 22:34:08 +0000 | [diff] [blame] | 336 | if ((*LI)[BB] == L) |
| 337 | LI->changeLoopFor(BB, NewOuter); |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 338 | --i; |
| 339 | } |
| 340 | } |
| 341 | |
Chris Lattner | 8417052 | 2004-04-13 05:05:33 +0000 | [diff] [blame] | 342 | return NewOuter; |
| 343 | } |
| 344 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 345 | /// \brief This method is called when the specified loop has more than one |
| 346 | /// backedge in it. |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 347 | /// |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 348 | /// If this occurs, revector all of these backedges to target a new basic block |
| 349 | /// and have that block branch to the loop header. This ensures that loops |
| 350 | /// have exactly one backedge. |
| 351 | static BasicBlock *insertUniqueBackedgeBlock(Loop *L, BasicBlock *Preheader, |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 352 | DominatorTree *DT, LoopInfo *LI) { |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 353 | assert(L->getNumBackEdges() > 1 && "Must have > 1 backedge!"); |
| 354 | |
| 355 | // Get information about the loop |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 356 | BasicBlock *Header = L->getHeader(); |
| 357 | Function *F = Header->getParent(); |
| 358 | |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 359 | // Unique backedge insertion currently depends on having a preheader. |
| 360 | if (!Preheader) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 361 | return nullptr; |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 362 | |
David Majnemer | eb518bd | 2015-08-04 08:21:40 +0000 | [diff] [blame] | 363 | // The header is not an EH pad; preheader insertion should ensure this. |
| 364 | assert(!Header->isEHPad() && "Can't insert backedge to EH pad"); |
Eli Friedman | 16ad290 | 2011-12-15 00:50:34 +0000 | [diff] [blame] | 365 | |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 366 | // Figure out which basic blocks contain back-edges to the loop header. |
| 367 | std::vector<BasicBlock*> BackedgeBlocks; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 368 | for (pred_iterator I = pred_begin(Header), E = pred_end(Header); I != E; ++I){ |
| 369 | BasicBlock *P = *I; |
| 370 | |
Dan Gohman | aa445c0 | 2010-08-14 00:43:09 +0000 | [diff] [blame] | 371 | // Indirectbr edges cannot be split, so we must fail if we find one. |
| 372 | if (isa<IndirectBrInst>(P->getTerminator())) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 373 | return nullptr; |
Dan Gohman | aa445c0 | 2010-08-14 00:43:09 +0000 | [diff] [blame] | 374 | |
Gabor Greif | e7650c7 | 2010-07-09 16:26:41 +0000 | [diff] [blame] | 375 | if (P != Preheader) BackedgeBlocks.push_back(P); |
| 376 | } |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 377 | |
| 378 | // Create and insert the new backedge block... |
Owen Anderson | 55f1c09 | 2009-08-13 21:58:54 +0000 | [diff] [blame] | 379 | BasicBlock *BEBlock = BasicBlock::Create(Header->getContext(), |
Alexey Samsonov | b7724b9 | 2015-06-29 21:30:14 +0000 | [diff] [blame] | 380 | Header->getName() + ".backedge", F); |
Gabor Greif | e9ecc68 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 381 | BranchInst *BETerminator = BranchInst::Create(Header, BEBlock); |
Alexey Samsonov | b7724b9 | 2015-06-29 21:30:14 +0000 | [diff] [blame] | 382 | BETerminator->setDebugLoc(Header->getFirstNonPHI()->getDebugLoc()); |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 383 | |
Chris Lattner | bf0aa92 | 2011-01-02 22:09:33 +0000 | [diff] [blame] | 384 | DEBUG(dbgs() << "LoopSimplify: Inserting unique backedge block " |
| 385 | << BEBlock->getName() << "\n"); |
Dan Gohman | 39917c7 | 2010-03-01 17:55:27 +0000 | [diff] [blame] | 386 | |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 387 | // Move the new backedge block to right after the last backedge block. |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 388 | Function::iterator InsertPos = ++BackedgeBlocks.back()->getIterator(); |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 389 | F->getBasicBlockList().splice(InsertPos, F->getBasicBlockList(), BEBlock); |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 390 | |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 391 | // Now that the block has been inserted into the function, create PHI nodes in |
| 392 | // the backedge block which correspond to any PHI nodes in the header block. |
Alkis Evlogimenos | 3ce42ec | 2004-09-28 02:40:37 +0000 | [diff] [blame] | 393 | for (BasicBlock::iterator I = Header->begin(); isa<PHINode>(I); ++I) { |
| 394 | PHINode *PN = cast<PHINode>(I); |
Jay Foad | 5213134 | 2011-03-30 11:28:46 +0000 | [diff] [blame] | 395 | PHINode *NewPN = PHINode::Create(PN->getType(), BackedgeBlocks.size(), |
| 396 | PN->getName()+".be", BETerminator); |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 397 | |
| 398 | // Loop over the PHI node, moving all entries except the one for the |
| 399 | // preheader over to the new PHI node. |
| 400 | unsigned PreheaderIdx = ~0U; |
| 401 | bool HasUniqueIncomingValue = true; |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 402 | Value *UniqueValue = nullptr; |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 403 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 404 | BasicBlock *IBB = PN->getIncomingBlock(i); |
| 405 | Value *IV = PN->getIncomingValue(i); |
| 406 | if (IBB == Preheader) { |
| 407 | PreheaderIdx = i; |
| 408 | } else { |
| 409 | NewPN->addIncoming(IV, IBB); |
| 410 | if (HasUniqueIncomingValue) { |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 411 | if (!UniqueValue) |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 412 | UniqueValue = IV; |
| 413 | else if (UniqueValue != IV) |
| 414 | HasUniqueIncomingValue = false; |
| 415 | } |
| 416 | } |
| 417 | } |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 418 | |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 419 | // Delete all of the incoming values from the old PN except the preheader's |
| 420 | assert(PreheaderIdx != ~0U && "PHI has no preheader entry??"); |
| 421 | if (PreheaderIdx != 0) { |
| 422 | PN->setIncomingValue(0, PN->getIncomingValue(PreheaderIdx)); |
| 423 | PN->setIncomingBlock(0, PN->getIncomingBlock(PreheaderIdx)); |
| 424 | } |
Chris Lattner | d8e2018 | 2005-01-29 00:39:08 +0000 | [diff] [blame] | 425 | // Nuke all entries except the zero'th. |
| 426 | for (unsigned i = 0, e = PN->getNumIncomingValues()-1; i != e; ++i) |
| 427 | PN->removeIncomingValue(e-i, false); |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 428 | |
| 429 | // Finally, add the newly constructed PHI node as the entry for the BEBlock. |
| 430 | PN->addIncoming(NewPN, BEBlock); |
| 431 | |
| 432 | // As an optimization, if all incoming values in the new PhiNode (which is a |
| 433 | // subset of the incoming values of the old PHI node) have the same value, |
| 434 | // eliminate the PHI Node. |
| 435 | if (HasUniqueIncomingValue) { |
| 436 | NewPN->replaceAllUsesWith(UniqueValue); |
| 437 | BEBlock->getInstList().erase(NewPN); |
| 438 | } |
| 439 | } |
| 440 | |
| 441 | // Now that all of the PHI nodes have been inserted and adjusted, modify the |
Nick Lewycky | 4d43d3c | 2008-04-25 16:53:59 +0000 | [diff] [blame] | 442 | // backedge blocks to just to the BEBlock instead of the header. |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 443 | for (unsigned i = 0, e = BackedgeBlocks.size(); i != e; ++i) { |
| 444 | TerminatorInst *TI = BackedgeBlocks[i]->getTerminator(); |
| 445 | for (unsigned Op = 0, e = TI->getNumSuccessors(); Op != e; ++Op) |
| 446 | if (TI->getSuccessor(Op) == Header) |
| 447 | TI->setSuccessor(Op, BEBlock); |
| 448 | } |
| 449 | |
| 450 | //===--- Update all analyses which we must preserve now -----------------===// |
| 451 | |
| 452 | // Update Loop Information - we know that this block is now in the current |
| 453 | // loop and all parent loops. |
Chandler Carruth | 691addc | 2015-01-18 01:25:51 +0000 | [diff] [blame] | 454 | L->addBasicBlockToLoop(BEBlock, *LI); |
Chris Lattner | c4622a6 | 2003-10-13 00:37:13 +0000 | [diff] [blame] | 455 | |
Devang Patel | d5258a23 | 2007-06-21 17:23:45 +0000 | [diff] [blame] | 456 | // Update dominator information |
| 457 | DT->splitBlock(BEBlock); |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 458 | |
| 459 | return BEBlock; |
| 460 | } |
| 461 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 462 | /// \brief Simplify one loop and queue further loops for simplification. |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 463 | static bool simplifyOneLoop(Loop *L, SmallVectorImpl<Loop *> &Worklist, |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 464 | DominatorTree *DT, LoopInfo *LI, |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 465 | ScalarEvolution *SE, AssumptionCache *AC, |
| 466 | bool PreserveLCSSA) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 467 | bool Changed = false; |
| 468 | ReprocessLoop: |
| 469 | |
| 470 | // Check to see that no blocks (other than the header) in this loop have |
| 471 | // predecessors that are not in the loop. This is not valid for natural |
| 472 | // loops, but can occur if the blocks are unreachable. Since they are |
| 473 | // unreachable we can just shamelessly delete those CFG edges! |
| 474 | for (Loop::block_iterator BB = L->block_begin(), E = L->block_end(); |
| 475 | BB != E; ++BB) { |
| 476 | if (*BB == L->getHeader()) continue; |
| 477 | |
| 478 | SmallPtrSet<BasicBlock*, 4> BadPreds; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 479 | for (pred_iterator PI = pred_begin(*BB), |
| 480 | PE = pred_end(*BB); PI != PE; ++PI) { |
| 481 | BasicBlock *P = *PI; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 482 | if (!L->contains(P)) |
| 483 | BadPreds.insert(P); |
| 484 | } |
| 485 | |
| 486 | // Delete each unique out-of-loop (and thus dead) predecessor. |
Craig Topper | 4627679 | 2014-08-24 23:23:06 +0000 | [diff] [blame] | 487 | for (BasicBlock *P : BadPreds) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 488 | |
| 489 | DEBUG(dbgs() << "LoopSimplify: Deleting edge from dead predecessor " |
Craig Topper | 4627679 | 2014-08-24 23:23:06 +0000 | [diff] [blame] | 490 | << P->getName() << "\n"); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 491 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 492 | // Zap the dead pred's terminator and replace it with unreachable. |
Craig Topper | 4627679 | 2014-08-24 23:23:06 +0000 | [diff] [blame] | 493 | TerminatorInst *TI = P->getTerminator(); |
David Majnemer | c67668f | 2016-01-24 19:32:52 +0000 | [diff] [blame] | 494 | changeToUnreachable(TI, /*UseLLVMTrap=*/false); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 495 | Changed = true; |
| 496 | } |
| 497 | } |
| 498 | |
| 499 | // If there are exiting blocks with branches on undef, resolve the undef in |
| 500 | // the direction which will exit the loop. This will help simplify loop |
| 501 | // trip count computations. |
| 502 | SmallVector<BasicBlock*, 8> ExitingBlocks; |
| 503 | L->getExitingBlocks(ExitingBlocks); |
| 504 | for (SmallVectorImpl<BasicBlock *>::iterator I = ExitingBlocks.begin(), |
| 505 | E = ExitingBlocks.end(); I != E; ++I) |
| 506 | if (BranchInst *BI = dyn_cast<BranchInst>((*I)->getTerminator())) |
| 507 | if (BI->isConditional()) { |
| 508 | if (UndefValue *Cond = dyn_cast<UndefValue>(BI->getCondition())) { |
| 509 | |
| 510 | DEBUG(dbgs() << "LoopSimplify: Resolving \"br i1 undef\" to exit in " |
| 511 | << (*I)->getName() << "\n"); |
| 512 | |
| 513 | BI->setCondition(ConstantInt::get(Cond->getType(), |
| 514 | !L->contains(BI->getSuccessor(0)))); |
| 515 | |
| 516 | // This may make the loop analyzable, force SCEV recomputation. |
| 517 | if (SE) |
| 518 | SE->forgetLoop(L); |
| 519 | |
| 520 | Changed = true; |
| 521 | } |
| 522 | } |
| 523 | |
| 524 | // Does the loop already have a preheader? If so, don't insert one. |
| 525 | BasicBlock *Preheader = L->getLoopPreheader(); |
| 526 | if (!Preheader) { |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 527 | Preheader = InsertPreheaderForLoop(L, DT, LI, PreserveLCSSA); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 528 | if (Preheader) { |
| 529 | ++NumInserted; |
| 530 | Changed = true; |
| 531 | } |
| 532 | } |
| 533 | |
| 534 | // Next, check to make sure that all exit nodes of the loop only have |
| 535 | // predecessors that are inside of the loop. This check guarantees that the |
| 536 | // loop preheader/header will dominate the exit blocks. If the exit block has |
| 537 | // predecessors from outside of the loop, split the edge now. |
| 538 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| 539 | L->getExitBlocks(ExitBlocks); |
| 540 | |
| 541 | SmallSetVector<BasicBlock *, 8> ExitBlockSet(ExitBlocks.begin(), |
| 542 | ExitBlocks.end()); |
| 543 | for (SmallSetVector<BasicBlock *, 8>::iterator I = ExitBlockSet.begin(), |
| 544 | E = ExitBlockSet.end(); I != E; ++I) { |
| 545 | BasicBlock *ExitBlock = *I; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 546 | for (pred_iterator PI = pred_begin(ExitBlock), PE = pred_end(ExitBlock); |
| 547 | PI != PE; ++PI) |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 548 | // Must be exactly this loop: no subloops, parent loops, or non-loop preds |
| 549 | // allowed. |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 550 | if (!L->contains(*PI)) { |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 551 | if (rewriteLoopExitBlock(L, ExitBlock, DT, LI, PreserveLCSSA)) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 552 | ++NumInserted; |
| 553 | Changed = true; |
| 554 | } |
| 555 | break; |
| 556 | } |
| 557 | } |
| 558 | |
| 559 | // If the header has more than two predecessors at this point (from the |
| 560 | // preheader and from multiple backedges), we must adjust the loop. |
| 561 | BasicBlock *LoopLatch = L->getLoopLatch(); |
| 562 | if (!LoopLatch) { |
| 563 | // If this is really a nested loop, rip it out into a child loop. Don't do |
| 564 | // this for loops with a giant number of backedges, just factor them into a |
| 565 | // common backedge instead. |
| 566 | if (L->getNumBackEdges() < 8) { |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 567 | if (Loop *OuterL = |
| 568 | separateNestedLoop(L, Preheader, DT, LI, SE, PreserveLCSSA, AC)) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 569 | ++NumNested; |
| 570 | // Enqueue the outer loop as it should be processed next in our |
| 571 | // depth-first nest walk. |
| 572 | Worklist.push_back(OuterL); |
| 573 | |
| 574 | // This is a big restructuring change, reprocess the whole loop. |
| 575 | Changed = true; |
| 576 | // GCC doesn't tail recursion eliminate this. |
| 577 | // FIXME: It isn't clear we can't rely on LLVM to TRE this. |
| 578 | goto ReprocessLoop; |
| 579 | } |
| 580 | } |
| 581 | |
| 582 | // If we either couldn't, or didn't want to, identify nesting of the loops, |
| 583 | // insert a new block that all backedges target, then make it jump to the |
| 584 | // loop header. |
Chandler Carruth | 96ada25 | 2015-07-22 09:52:54 +0000 | [diff] [blame] | 585 | LoopLatch = insertUniqueBackedgeBlock(L, Preheader, DT, LI); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 586 | if (LoopLatch) { |
| 587 | ++NumInserted; |
| 588 | Changed = true; |
| 589 | } |
| 590 | } |
| 591 | |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 592 | const DataLayout &DL = L->getHeader()->getModule()->getDataLayout(); |
| 593 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 594 | // Scan over the PHI nodes in the loop header. Since they now have only two |
| 595 | // incoming values (the loop is canonicalized), we may have simplified the PHI |
| 596 | // down to 'X = phi [X, Y]', which should be replaced with 'Y'. |
| 597 | PHINode *PN; |
| 598 | for (BasicBlock::iterator I = L->getHeader()->begin(); |
| 599 | (PN = dyn_cast<PHINode>(I++)); ) |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 600 | if (Value *V = SimplifyInstruction(PN, DL, nullptr, DT, AC)) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 601 | if (SE) SE->forgetValue(PN); |
| 602 | PN->replaceAllUsesWith(V); |
| 603 | PN->eraseFromParent(); |
| 604 | } |
| 605 | |
| 606 | // If this loop has multiple exits and the exits all go to the same |
| 607 | // block, attempt to merge the exits. This helps several passes, such |
| 608 | // as LoopRotation, which do not support loops with multiple exits. |
| 609 | // SimplifyCFG also does this (and this code uses the same utility |
| 610 | // function), however this code is loop-aware, where SimplifyCFG is |
| 611 | // not. That gives it the advantage of being able to hoist |
| 612 | // loop-invariant instructions out of the way to open up more |
| 613 | // opportunities, and the disadvantage of having the responsibility |
| 614 | // to preserve dominator information. |
| 615 | bool UniqueExit = true; |
| 616 | if (!ExitBlocks.empty()) |
| 617 | for (unsigned i = 1, e = ExitBlocks.size(); i != e; ++i) |
| 618 | if (ExitBlocks[i] != ExitBlocks[0]) { |
| 619 | UniqueExit = false; |
| 620 | break; |
| 621 | } |
| 622 | if (UniqueExit) { |
| 623 | for (unsigned i = 0, e = ExitingBlocks.size(); i != e; ++i) { |
| 624 | BasicBlock *ExitingBlock = ExitingBlocks[i]; |
| 625 | if (!ExitingBlock->getSinglePredecessor()) continue; |
| 626 | BranchInst *BI = dyn_cast<BranchInst>(ExitingBlock->getTerminator()); |
| 627 | if (!BI || !BI->isConditional()) continue; |
| 628 | CmpInst *CI = dyn_cast<CmpInst>(BI->getCondition()); |
| 629 | if (!CI || CI->getParent() != ExitingBlock) continue; |
| 630 | |
| 631 | // Attempt to hoist out all instructions except for the |
| 632 | // comparison and the branch. |
| 633 | bool AllInvariant = true; |
| 634 | bool AnyInvariant = false; |
| 635 | for (BasicBlock::iterator I = ExitingBlock->begin(); &*I != BI; ) { |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 636 | Instruction *Inst = &*I++; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 637 | // Skip debug info intrinsics. |
| 638 | if (isa<DbgInfoIntrinsic>(Inst)) |
| 639 | continue; |
| 640 | if (Inst == CI) |
| 641 | continue; |
| 642 | if (!L->makeLoopInvariant(Inst, AnyInvariant, |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 643 | Preheader ? Preheader->getTerminator() |
| 644 | : nullptr)) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 645 | AllInvariant = false; |
| 646 | break; |
| 647 | } |
| 648 | } |
| 649 | if (AnyInvariant) { |
| 650 | Changed = true; |
| 651 | // The loop disposition of all SCEV expressions that depend on any |
| 652 | // hoisted values have also changed. |
| 653 | if (SE) |
| 654 | SE->forgetLoopDispositions(L); |
| 655 | } |
| 656 | if (!AllInvariant) continue; |
| 657 | |
| 658 | // The block has now been cleared of all instructions except for |
| 659 | // a comparison and a conditional branch. SimplifyCFG may be able |
| 660 | // to fold it now. |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 661 | if (!FoldBranchToCommonDest(BI)) |
| 662 | continue; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 663 | |
| 664 | // Success. The block is now dead, so remove it from the loop, |
| 665 | // update the dominator tree and delete it. |
| 666 | DEBUG(dbgs() << "LoopSimplify: Eliminating exiting block " |
| 667 | << ExitingBlock->getName() << "\n"); |
| 668 | |
| 669 | // Notify ScalarEvolution before deleting this block. Currently assume the |
| 670 | // parent loop doesn't change (spliting edges doesn't count). If blocks, |
| 671 | // CFG edges, or other values in the parent loop change, then we need call |
| 672 | // to forgetLoop() for the parent instead. |
| 673 | if (SE) |
| 674 | SE->forgetLoop(L); |
| 675 | |
| 676 | assert(pred_begin(ExitingBlock) == pred_end(ExitingBlock)); |
| 677 | Changed = true; |
| 678 | LI->removeBlock(ExitingBlock); |
| 679 | |
| 680 | DomTreeNode *Node = DT->getNode(ExitingBlock); |
| 681 | const std::vector<DomTreeNodeBase<BasicBlock> *> &Children = |
| 682 | Node->getChildren(); |
| 683 | while (!Children.empty()) { |
| 684 | DomTreeNode *Child = Children.front(); |
| 685 | DT->changeImmediateDominator(Child, Node->getIDom()); |
| 686 | } |
| 687 | DT->eraseNode(ExitingBlock); |
| 688 | |
| 689 | BI->getSuccessor(0)->removePredecessor(ExitingBlock); |
| 690 | BI->getSuccessor(1)->removePredecessor(ExitingBlock); |
| 691 | ExitingBlock->eraseFromParent(); |
| 692 | } |
| 693 | } |
| 694 | |
| 695 | return Changed; |
| 696 | } |
| 697 | |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 698 | bool llvm::simplifyLoop(Loop *L, DominatorTree *DT, LoopInfo *LI, |
| 699 | ScalarEvolution *SE, AssumptionCache *AC, |
| 700 | bool PreserveLCSSA) { |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 701 | bool Changed = false; |
| 702 | |
| 703 | // Worklist maintains our depth-first queue of loops in this nest to process. |
| 704 | SmallVector<Loop *, 4> Worklist; |
| 705 | Worklist.push_back(L); |
| 706 | |
| 707 | // Walk the worklist from front to back, pushing newly found sub loops onto |
| 708 | // the back. This will let us process loops from back to front in depth-first |
| 709 | // order. We can use this simple process because loops form a tree. |
| 710 | for (unsigned Idx = 0; Idx != Worklist.size(); ++Idx) { |
| 711 | Loop *L2 = Worklist[Idx]; |
Benjamin Kramer | 6cd780f | 2015-02-17 15:29:18 +0000 | [diff] [blame] | 712 | Worklist.append(L2->begin(), L2->end()); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 713 | } |
| 714 | |
| 715 | while (!Worklist.empty()) |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 716 | Changed |= simplifyOneLoop(Worklist.pop_back_val(), Worklist, DT, LI, SE, |
| 717 | AC, PreserveLCSSA); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 718 | |
| 719 | return Changed; |
| 720 | } |
| 721 | |
| 722 | namespace { |
| 723 | struct LoopSimplify : public FunctionPass { |
| 724 | static char ID; // Pass identification, replacement for typeid |
| 725 | LoopSimplify() : FunctionPass(ID) { |
| 726 | initializeLoopSimplifyPass(*PassRegistry::getPassRegistry()); |
| 727 | } |
| 728 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 729 | DominatorTree *DT; |
| 730 | LoopInfo *LI; |
| 731 | ScalarEvolution *SE; |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 732 | AssumptionCache *AC; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 733 | |
Craig Topper | 3e4c697 | 2014-03-05 09:10:37 +0000 | [diff] [blame] | 734 | bool runOnFunction(Function &F) override; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 735 | |
Craig Topper | 3e4c697 | 2014-03-05 09:10:37 +0000 | [diff] [blame] | 736 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 737 | AU.addRequired<AssumptionCacheTracker>(); |
Hal Finkel | 60db058 | 2014-09-07 18:57:58 +0000 | [diff] [blame] | 738 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 739 | // We need loop information to identify the loops... |
| 740 | AU.addRequired<DominatorTreeWrapperPass>(); |
| 741 | AU.addPreserved<DominatorTreeWrapperPass>(); |
| 742 | |
Chandler Carruth | 4f8f307 | 2015-01-17 14:16:18 +0000 | [diff] [blame] | 743 | AU.addRequired<LoopInfoWrapperPass>(); |
| 744 | AU.addPreserved<LoopInfoWrapperPass>(); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 745 | |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 746 | AU.addPreserved<BasicAAWrapperPass>(); |
| 747 | AU.addPreserved<AAResultsWrapperPass>(); |
| 748 | AU.addPreserved<GlobalsAAWrapperPass>(); |
Chandler Carruth | 2f1fd16 | 2015-08-17 02:08:17 +0000 | [diff] [blame] | 749 | AU.addPreserved<ScalarEvolutionWrapperPass>(); |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 750 | AU.addPreserved<SCEVAAWrapperPass>(); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 751 | AU.addPreserved<DependenceAnalysis>(); |
| 752 | AU.addPreservedID(BreakCriticalEdgesID); // No critical edges added. |
| 753 | } |
| 754 | |
| 755 | /// verifyAnalysis() - Verify LoopSimplifyForm's guarantees. |
Craig Topper | 3e4c697 | 2014-03-05 09:10:37 +0000 | [diff] [blame] | 756 | void verifyAnalysis() const override; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 757 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 758 | } |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 759 | |
| 760 | char LoopSimplify::ID = 0; |
| 761 | INITIALIZE_PASS_BEGIN(LoopSimplify, "loop-simplify", |
Mark Heffernan | 2d393ea | 2014-11-04 23:02:09 +0000 | [diff] [blame] | 762 | "Canonicalize natural loops", false, false) |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 763 | INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker) |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 764 | INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) |
Chandler Carruth | 4f8f307 | 2015-01-17 14:16:18 +0000 | [diff] [blame] | 765 | INITIALIZE_PASS_DEPENDENCY(LoopInfoWrapperPass) |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 766 | INITIALIZE_PASS_DEPENDENCY(BasicAAWrapperPass) |
| 767 | INITIALIZE_PASS_DEPENDENCY(GlobalsAAWrapperPass) |
| 768 | INITIALIZE_PASS_DEPENDENCY(SCEVAAWrapperPass) |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 769 | INITIALIZE_PASS_END(LoopSimplify, "loop-simplify", |
Mark Heffernan | 2d393ea | 2014-11-04 23:02:09 +0000 | [diff] [blame] | 770 | "Canonicalize natural loops", false, false) |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 771 | |
| 772 | // Publicly exposed interface to pass... |
| 773 | char &llvm::LoopSimplifyID = LoopSimplify::ID; |
| 774 | Pass *llvm::createLoopSimplifyPass() { return new LoopSimplify(); } |
| 775 | |
Michael Zolotukhin | a93fec0 | 2014-04-29 07:35:33 +0000 | [diff] [blame] | 776 | /// runOnFunction - Run down all loops in the CFG (recursively, but we could do |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 777 | /// it in any convenient order) inserting preheaders... |
| 778 | /// |
| 779 | bool LoopSimplify::runOnFunction(Function &F) { |
| 780 | bool Changed = false; |
Chandler Carruth | 4f8f307 | 2015-01-17 14:16:18 +0000 | [diff] [blame] | 781 | LI = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo(); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 782 | DT = &getAnalysis<DominatorTreeWrapperPass>().getDomTree(); |
Chandler Carruth | 2f1fd16 | 2015-08-17 02:08:17 +0000 | [diff] [blame] | 783 | auto *SEWP = getAnalysisIfAvailable<ScalarEvolutionWrapperPass>(); |
| 784 | SE = SEWP ? &SEWP->getSE() : nullptr; |
Chandler Carruth | 66b3130 | 2015-01-04 12:03:27 +0000 | [diff] [blame] | 785 | AC = &getAnalysis<AssumptionCacheTracker>().getAssumptionCache(F); |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 786 | bool PreserveLCSSA = mustPreserveAnalysisID(LCSSAID); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 787 | |
| 788 | // Simplify each loop nest in the function. |
| 789 | for (LoopInfo::iterator I = LI->begin(), E = LI->end(); I != E; ++I) |
Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 790 | Changed |= simplifyLoop(*I, DT, LI, SE, AC, PreserveLCSSA); |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 791 | |
| 792 | return Changed; |
| 793 | } |
| 794 | |
| 795 | // FIXME: Restore this code when we re-enable verification in verifyAnalysis |
| 796 | // below. |
| 797 | #if 0 |
| 798 | static void verifyLoop(Loop *L) { |
| 799 | // Verify subloops. |
| 800 | for (Loop::iterator I = L->begin(), E = L->end(); I != E; ++I) |
| 801 | verifyLoop(*I); |
| 802 | |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 803 | // It used to be possible to just assert L->isLoopSimplifyForm(), however |
| 804 | // with the introduction of indirectbr, there are now cases where it's |
| 805 | // not possible to transform a loop as necessary. We can at least check |
| 806 | // that there is an indirectbr near any time there's trouble. |
| 807 | |
| 808 | // Indirectbr can interfere with preheader and unique backedge insertion. |
| 809 | if (!L->getLoopPreheader() || !L->getLoopLatch()) { |
| 810 | bool HasIndBrPred = false; |
| 811 | for (pred_iterator PI = pred_begin(L->getHeader()), |
| 812 | PE = pred_end(L->getHeader()); PI != PE; ++PI) |
| 813 | if (isa<IndirectBrInst>((*PI)->getTerminator())) { |
| 814 | HasIndBrPred = true; |
| 815 | break; |
| 816 | } |
| 817 | assert(HasIndBrPred && |
| 818 | "LoopSimplify has no excuse for missing loop header info!"); |
Duncan Sands | a41634e | 2011-08-12 14:54:45 +0000 | [diff] [blame] | 819 | (void)HasIndBrPred; |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 820 | } |
| 821 | |
Bill Wendling | b15d6eb | 2011-08-18 21:10:01 +0000 | [diff] [blame] | 822 | // Indirectbr can interfere with exit block canonicalization. |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 823 | if (!L->hasDedicatedExits()) { |
| 824 | bool HasIndBrExiting = false; |
| 825 | SmallVector<BasicBlock*, 8> ExitingBlocks; |
| 826 | L->getExitingBlocks(ExitingBlocks); |
Bill Wendling | 39257d6 | 2011-08-17 21:20:43 +0000 | [diff] [blame] | 827 | for (unsigned i = 0, e = ExitingBlocks.size(); i != e; ++i) { |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 828 | if (isa<IndirectBrInst>((ExitingBlocks[i])->getTerminator())) { |
| 829 | HasIndBrExiting = true; |
| 830 | break; |
| 831 | } |
Bill Wendling | 39257d6 | 2011-08-17 21:20:43 +0000 | [diff] [blame] | 832 | } |
| 833 | |
Bill Wendling | b15d6eb | 2011-08-18 21:10:01 +0000 | [diff] [blame] | 834 | assert(HasIndBrExiting && |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 835 | "LoopSimplify has no excuse for missing exit block info!"); |
Bill Wendling | b15d6eb | 2011-08-18 21:10:01 +0000 | [diff] [blame] | 836 | (void)HasIndBrExiting; |
Dan Gohman | 1ef784d | 2009-11-05 21:14:46 +0000 | [diff] [blame] | 837 | } |
Chris Lattner | 61992f6 | 2002-09-26 16:17:31 +0000 | [diff] [blame] | 838 | } |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 839 | #endif |
| 840 | |
| 841 | void LoopSimplify::verifyAnalysis() const { |
| 842 | // FIXME: This routine is being called mid-way through the loop pass manager |
| 843 | // as loop passes destroy this analysis. That's actually fine, but we have no |
| 844 | // way of expressing that here. Once all of the passes that destroy this are |
| 845 | // hoisted out of the loop pass manager we can add back verification here. |
| 846 | #if 0 |
| 847 | for (LoopInfo::iterator I = LI->begin(), E = LI->end(); I != E; ++I) |
| 848 | verifyLoop(*I); |
| 849 | #endif |
| 850 | } |