Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 1 | //===-- UnrollLoop.cpp - Loop unrolling utilities -------------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This file implements some loop unrolling utilities. It does not define any |
| 11 | // actual pass or policy, but provides a single function to perform loop |
| 12 | // unrolling. |
| 13 | // |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 14 | // The process of unrolling can produce extraneous basic blocks linked with |
| 15 | // unconditional branches. This will be corrected in the future. |
Chris Lattner | dfcfcb4 | 2011-01-11 08:00:40 +0000 | [diff] [blame] | 16 | // |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 17 | //===----------------------------------------------------------------------===// |
| 18 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 19 | #include "llvm/Transforms/Utils/UnrollLoop.h" |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 20 | #include "llvm/ADT/Statistic.h" |
Duncan Sands | 433c167 | 2010-11-23 20:26:33 +0000 | [diff] [blame] | 21 | #include "llvm/Analysis/InstructionSimplify.h" |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 22 | #include "llvm/Analysis/LoopIterator.h" |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 23 | #include "llvm/Analysis/LoopPass.h" |
Dan Gohman | a7908ae | 2010-07-26 18:02:06 +0000 | [diff] [blame] | 24 | #include "llvm/Analysis/ScalarEvolution.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 25 | #include "llvm/IR/BasicBlock.h" |
Hal Finkel | a995f92 | 2014-07-10 14:41:31 +0000 | [diff] [blame^] | 26 | #include "llvm/IR/DataLayout.h" |
Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 27 | #include "llvm/IR/Dominators.h" |
Diego Novillo | 7f8af8b | 2014-05-22 14:19:46 +0000 | [diff] [blame] | 28 | #include "llvm/IR/DiagnosticInfo.h" |
Diego Novillo | 34fc8a7 | 2014-04-29 14:27:31 +0000 | [diff] [blame] | 29 | #include "llvm/IR/LLVMContext.h" |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 30 | #include "llvm/Support/Debug.h" |
Daniel Dunbar | 0dd5e1e | 2009-07-25 00:23:56 +0000 | [diff] [blame] | 31 | #include "llvm/Support/raw_ostream.h" |
Chris Lattner | dc3f6f2 | 2008-12-03 19:44:02 +0000 | [diff] [blame] | 32 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 33 | #include "llvm/Transforms/Utils/Cloning.h" |
| 34 | #include "llvm/Transforms/Utils/Local.h" |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 35 | #include "llvm/Transforms/Utils/LoopUtils.h" |
Andrew Trick | 4d0040b | 2011-08-10 04:29:49 +0000 | [diff] [blame] | 36 | #include "llvm/Transforms/Utils/SimplifyIndVar.h" |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 37 | using namespace llvm; |
| 38 | |
Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 39 | #define DEBUG_TYPE "loop-unroll" |
| 40 | |
Chris Lattner | dc3f6f2 | 2008-12-03 19:44:02 +0000 | [diff] [blame] | 41 | // TODO: Should these be here or in LoopUnroll? |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 42 | STATISTIC(NumCompletelyUnrolled, "Number of loops completely unrolled"); |
Chris Lattner | dfcfcb4 | 2011-01-11 08:00:40 +0000 | [diff] [blame] | 43 | STATISTIC(NumUnrolled, "Number of loops unrolled (completely or otherwise)"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 44 | |
| 45 | /// RemapInstruction - Convert the instruction operands from referencing the |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 46 | /// current values into those specified by VMap. |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 47 | static inline void RemapInstruction(Instruction *I, |
Rafael Espindola | 229e38f | 2010-10-13 01:36:30 +0000 | [diff] [blame] | 48 | ValueToValueMapTy &VMap) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 49 | for (unsigned op = 0, E = I->getNumOperands(); op != E; ++op) { |
| 50 | Value *Op = I->getOperand(op); |
Rafael Espindola | 229e38f | 2010-10-13 01:36:30 +0000 | [diff] [blame] | 51 | ValueToValueMapTy::iterator It = VMap.find(Op); |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 52 | if (It != VMap.end()) |
Dan Gohman | fa8969f | 2009-10-31 14:46:50 +0000 | [diff] [blame] | 53 | I->setOperand(op, It->second); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 54 | } |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 55 | |
| 56 | if (PHINode *PN = dyn_cast<PHINode>(I)) { |
| 57 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 58 | ValueToValueMapTy::iterator It = VMap.find(PN->getIncomingBlock(i)); |
| 59 | if (It != VMap.end()) |
| 60 | PN->setIncomingBlock(i, cast<BasicBlock>(It->second)); |
| 61 | } |
| 62 | } |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 63 | } |
| 64 | |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 65 | /// FoldBlockIntoPredecessor - Folds a basic block into its predecessor if it |
| 66 | /// only has one predecessor, and that predecessor only has one successor. |
| 67 | /// The LoopInfo Analysis that is passed will be kept consistent. |
| 68 | /// Returns the new combined block. |
Andrew Trick | bf69d03 | 2011-08-03 18:32:11 +0000 | [diff] [blame] | 69 | static BasicBlock *FoldBlockIntoPredecessor(BasicBlock *BB, LoopInfo* LI, |
| 70 | LPPassManager *LPM) { |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 71 | // Merge basic blocks into their predecessor if there is only one distinct |
| 72 | // pred, and if there is only one distinct successor of the predecessor, and |
| 73 | // if there are no PHI nodes. |
| 74 | BasicBlock *OnlyPred = BB->getSinglePredecessor(); |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 75 | if (!OnlyPred) return nullptr; |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 76 | |
| 77 | if (OnlyPred->getTerminator()->getNumSuccessors() != 1) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 78 | return nullptr; |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 79 | |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 80 | DEBUG(dbgs() << "Merging: " << *BB << "into: " << *OnlyPred); |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 81 | |
| 82 | // Resolve any PHI nodes at the start of the block. They are all |
| 83 | // guaranteed to have exactly one entry if they exist, unless there are |
| 84 | // multiple duplicate (but guaranteed to be equal) entries for the |
| 85 | // incoming edges. This occurs when there are multiple edges from |
| 86 | // OnlyPred to OnlySucc. |
| 87 | FoldSingleEntryPHINodes(BB); |
| 88 | |
| 89 | // Delete the unconditional branch from the predecessor... |
| 90 | OnlyPred->getInstList().pop_back(); |
| 91 | |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 92 | // Make all PHI nodes that referred to BB now refer to Pred as their |
| 93 | // source... |
| 94 | BB->replaceAllUsesWith(OnlyPred); |
| 95 | |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 96 | // Move all definitions in the successor to the predecessor... |
| 97 | OnlyPred->getInstList().splice(OnlyPred->end(), BB->getInstList()); |
| 98 | |
NAKAMURA Takumi | f9c8339 | 2013-11-17 18:05:34 +0000 | [diff] [blame] | 99 | // OldName will be valid until erased. |
Jakub Staszak | 86a7492 | 2013-11-13 20:09:11 +0000 | [diff] [blame] | 100 | StringRef OldName = BB->getName(); |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 101 | |
| 102 | // Erase basic block from the function... |
Andrew Trick | bf69d03 | 2011-08-03 18:32:11 +0000 | [diff] [blame] | 103 | |
| 104 | // ScalarEvolution holds references to loop exit blocks. |
Andrew Trick | a6fb910 | 2012-06-05 17:51:05 +0000 | [diff] [blame] | 105 | if (LPM) { |
| 106 | if (ScalarEvolution *SE = LPM->getAnalysisIfAvailable<ScalarEvolution>()) { |
| 107 | if (Loop *L = LI->getLoopFor(BB)) |
| 108 | SE->forgetLoop(L); |
| 109 | } |
Andrew Trick | bf69d03 | 2011-08-03 18:32:11 +0000 | [diff] [blame] | 110 | } |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 111 | LI->removeBlock(BB); |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 112 | |
| 113 | // Inherit predecessor's name if it exists... |
| 114 | if (!OldName.empty() && !OnlyPred->hasName()) |
| 115 | OnlyPred->setName(OldName); |
| 116 | |
NAKAMURA Takumi | f9c8339 | 2013-11-17 18:05:34 +0000 | [diff] [blame] | 117 | BB->eraseFromParent(); |
| 118 | |
Dan Gohman | 2d02ff8 | 2009-10-31 17:33:01 +0000 | [diff] [blame] | 119 | return OnlyPred; |
| 120 | } |
| 121 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 122 | /// Unroll the given loop by Count. The loop must be in LCSSA form. Returns true |
Chris Lattner | 4a14fbc | 2011-02-18 04:25:21 +0000 | [diff] [blame] | 123 | /// if unrolling was successful, or false if the loop was unmodified. Unrolling |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 124 | /// can only fail when the loop's latch block is not terminated by a conditional |
| 125 | /// branch instruction. However, if the trip count (and multiple) are not known, |
| 126 | /// loop unrolling will mostly produce more code that is no faster. |
| 127 | /// |
Andrew Trick | 990f771 | 2011-07-25 22:17:47 +0000 | [diff] [blame] | 128 | /// TripCount is generally defined as the number of times the loop header |
| 129 | /// executes. UnrollLoop relaxes the definition to permit early exits: here |
| 130 | /// TripCount is the iteration on which control exits LatchBlock if no early |
| 131 | /// exits were taken. Note that UnrollLoop assumes that the loop counter test |
| 132 | /// terminates LatchBlock in order to remove unnecesssary instances of the |
| 133 | /// test. In other words, control may exit the loop prior to TripCount |
| 134 | /// iterations via an early branch, but control may not exit the loop from the |
| 135 | /// LatchBlock's terminator prior to TripCount iterations. |
| 136 | /// |
| 137 | /// Similarly, TripMultiple divides the number of times that the LatchBlock may |
| 138 | /// execute without exiting the loop. |
| 139 | /// |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 140 | /// The LoopInfo Analysis that is passed will be kept consistent. |
| 141 | /// |
| 142 | /// If a LoopPassManager is passed in, and the loop is fully removed, it will be |
| 143 | /// removed from the LoopPassManager as well. LPM can also be NULL. |
Andrew Trick | 4d0040b | 2011-08-10 04:29:49 +0000 | [diff] [blame] | 144 | /// |
| 145 | /// This utility preserves LoopInfo. If DominatorTree or ScalarEvolution are |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 146 | /// available from the Pass it must also preserve those analyses. |
Andrew Trick | 1cabe54 | 2011-07-23 00:33:05 +0000 | [diff] [blame] | 147 | bool llvm::UnrollLoop(Loop *L, unsigned Count, unsigned TripCount, |
Andrew Trick | d04d1529 | 2011-12-09 06:19:40 +0000 | [diff] [blame] | 148 | bool AllowRuntime, unsigned TripMultiple, |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 149 | LoopInfo *LI, Pass *PP, LPPassManager *LPM) { |
Dan Gohman | 415c64e | 2009-11-05 19:44:06 +0000 | [diff] [blame] | 150 | BasicBlock *Preheader = L->getLoopPreheader(); |
| 151 | if (!Preheader) { |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 152 | DEBUG(dbgs() << " Can't unroll; loop preheader-insertion failed.\n"); |
Dan Gohman | 415c64e | 2009-11-05 19:44:06 +0000 | [diff] [blame] | 153 | return false; |
| 154 | } |
| 155 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 156 | BasicBlock *LatchBlock = L->getLoopLatch(); |
Dan Gohman | 415c64e | 2009-11-05 19:44:06 +0000 | [diff] [blame] | 157 | if (!LatchBlock) { |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 158 | DEBUG(dbgs() << " Can't unroll; loop exit-block-insertion failed.\n"); |
Dan Gohman | 415c64e | 2009-11-05 19:44:06 +0000 | [diff] [blame] | 159 | return false; |
| 160 | } |
| 161 | |
Andrew Trick | 4442bfe | 2012-04-10 05:14:42 +0000 | [diff] [blame] | 162 | // Loops with indirectbr cannot be cloned. |
| 163 | if (!L->isSafeToClone()) { |
| 164 | DEBUG(dbgs() << " Can't unroll; Loop body cannot be cloned.\n"); |
| 165 | return false; |
| 166 | } |
| 167 | |
Dan Gohman | 415c64e | 2009-11-05 19:44:06 +0000 | [diff] [blame] | 168 | BasicBlock *Header = L->getHeader(); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 169 | BranchInst *BI = dyn_cast<BranchInst>(LatchBlock->getTerminator()); |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 170 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 171 | if (!BI || BI->isUnconditional()) { |
| 172 | // The loop-rotate pass can be helpful to avoid this in many cases. |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 173 | DEBUG(dbgs() << |
Chris Lattner | b25de3f | 2009-08-23 04:37:46 +0000 | [diff] [blame] | 174 | " Can't unroll; loop not terminated by a conditional branch.\n"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 175 | return false; |
| 176 | } |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 177 | |
Chris Lattner | 4a14fbc | 2011-02-18 04:25:21 +0000 | [diff] [blame] | 178 | if (Header->hasAddressTaken()) { |
| 179 | // The loop-rotate pass can be helpful to avoid this in many cases. |
| 180 | DEBUG(dbgs() << |
| 181 | " Won't unroll loop: address of header block is taken.\n"); |
| 182 | return false; |
| 183 | } |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 184 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 185 | if (TripCount != 0) |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 186 | DEBUG(dbgs() << " Trip Count = " << TripCount << "\n"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 187 | if (TripMultiple != 1) |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 188 | DEBUG(dbgs() << " Trip Multiple = " << TripMultiple << "\n"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 189 | |
| 190 | // Effectively "DCE" unrolled iterations that are beyond the tripcount |
| 191 | // and will never be executed. |
| 192 | if (TripCount != 0 && Count > TripCount) |
| 193 | Count = TripCount; |
| 194 | |
Andrew Trick | ca3417e | 2011-12-16 02:03:48 +0000 | [diff] [blame] | 195 | // Don't enter the unroll code if there is nothing to do. This way we don't |
| 196 | // need to support "partial unrolling by 1". |
| 197 | if (TripCount == 0 && Count < 2) |
| 198 | return false; |
| 199 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 200 | assert(Count > 0); |
| 201 | assert(TripMultiple > 0); |
| 202 | assert(TripCount == 0 || TripCount % TripMultiple == 0); |
| 203 | |
| 204 | // Are we eliminating the loop control altogether? |
| 205 | bool CompletelyUnroll = Count == TripCount; |
| 206 | |
Andrew Trick | d04d1529 | 2011-12-09 06:19:40 +0000 | [diff] [blame] | 207 | // We assume a run-time trip count if the compiler cannot |
| 208 | // figure out the loop trip count and the unroll-runtime |
| 209 | // flag is specified. |
| 210 | bool RuntimeTripCount = (TripCount == 0 && Count > 0 && AllowRuntime); |
| 211 | |
| 212 | if (RuntimeTripCount && !UnrollRuntimeLoopProlog(L, Count, LI, LPM)) |
| 213 | return false; |
| 214 | |
| 215 | // Notify ScalarEvolution that the loop will be substantially changed, |
| 216 | // if not outright eliminated. |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 217 | if (PP) { |
| 218 | ScalarEvolution *SE = PP->getAnalysisIfAvailable<ScalarEvolution>(); |
Andrew Trick | d29cd73 | 2012-05-08 02:52:09 +0000 | [diff] [blame] | 219 | if (SE) |
| 220 | SE->forgetLoop(L); |
| 221 | } |
Andrew Trick | d04d1529 | 2011-12-09 06:19:40 +0000 | [diff] [blame] | 222 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 223 | // If we know the trip count, we know the multiple... |
| 224 | unsigned BreakoutTrip = 0; |
| 225 | if (TripCount != 0) { |
| 226 | BreakoutTrip = TripCount % Count; |
| 227 | TripMultiple = 0; |
| 228 | } else { |
| 229 | // Figure out what multiple to use. |
| 230 | BreakoutTrip = TripMultiple = |
| 231 | (unsigned)GreatestCommonDivisor64(Count, TripMultiple); |
| 232 | } |
| 233 | |
Diego Novillo | 34fc8a7 | 2014-04-29 14:27:31 +0000 | [diff] [blame] | 234 | // Report the unrolling decision. |
| 235 | DebugLoc LoopLoc = L->getStartLoc(); |
| 236 | Function *F = Header->getParent(); |
| 237 | LLVMContext &Ctx = F->getContext(); |
| 238 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 239 | if (CompletelyUnroll) { |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 240 | DEBUG(dbgs() << "COMPLETELY UNROLLING loop %" << Header->getName() |
Daniel Dunbar | 0dd5e1e | 2009-07-25 00:23:56 +0000 | [diff] [blame] | 241 | << " with trip count " << TripCount << "!\n"); |
Diego Novillo | 7f8af8b | 2014-05-22 14:19:46 +0000 | [diff] [blame] | 242 | emitOptimizationRemark(Ctx, DEBUG_TYPE, *F, LoopLoc, |
| 243 | Twine("completely unrolled loop with ") + |
| 244 | Twine(TripCount) + " iterations"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 245 | } else { |
Benjamin Kramer | cccdadc | 2014-07-08 14:55:06 +0000 | [diff] [blame] | 246 | auto EmitDiag = [&](const Twine &T) { |
| 247 | emitOptimizationRemark(Ctx, DEBUG_TYPE, *F, LoopLoc, |
| 248 | "unrolled loop by a factor of " + Twine(Count) + |
| 249 | T); |
| 250 | }; |
| 251 | |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 252 | DEBUG(dbgs() << "UNROLLING loop %" << Header->getName() |
Daniel Dunbar | 0dd5e1e | 2009-07-25 00:23:56 +0000 | [diff] [blame] | 253 | << " by " << Count); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 254 | if (TripMultiple == 0 || BreakoutTrip != TripMultiple) { |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 255 | DEBUG(dbgs() << " with a breakout at trip " << BreakoutTrip); |
Benjamin Kramer | cccdadc | 2014-07-08 14:55:06 +0000 | [diff] [blame] | 256 | EmitDiag(" with a breakout at trip " + Twine(BreakoutTrip)); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 257 | } else if (TripMultiple != 1) { |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 258 | DEBUG(dbgs() << " with " << TripMultiple << " trips per branch"); |
Benjamin Kramer | cccdadc | 2014-07-08 14:55:06 +0000 | [diff] [blame] | 259 | EmitDiag(" with " + Twine(TripMultiple) + " trips per branch"); |
Andrew Trick | d04d1529 | 2011-12-09 06:19:40 +0000 | [diff] [blame] | 260 | } else if (RuntimeTripCount) { |
| 261 | DEBUG(dbgs() << " with run-time trip count"); |
Benjamin Kramer | cccdadc | 2014-07-08 14:55:06 +0000 | [diff] [blame] | 262 | EmitDiag(" with run-time trip count"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 263 | } |
David Greene | 627f40a | 2010-01-05 01:26:41 +0000 | [diff] [blame] | 264 | DEBUG(dbgs() << "!\n"); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 265 | } |
| 266 | |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 267 | bool ContinueOnTrue = L->contains(BI->getSuccessor(0)); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 268 | BasicBlock *LoopExit = BI->getSuccessor(ContinueOnTrue); |
| 269 | |
| 270 | // For the first iteration of the loop, we should use the precloned values for |
| 271 | // PHI nodes. Insert associations now. |
Devang Patel | 2176643 | 2010-04-20 22:24:18 +0000 | [diff] [blame] | 272 | ValueToValueMapTy LastValueMap; |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 273 | std::vector<PHINode*> OrigPHINode; |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 274 | for (BasicBlock::iterator I = Header->begin(); isa<PHINode>(I); ++I) { |
Andrew Trick | 5e0ee1c | 2011-08-09 03:11:29 +0000 | [diff] [blame] | 275 | OrigPHINode.push_back(cast<PHINode>(I)); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 276 | } |
| 277 | |
| 278 | std::vector<BasicBlock*> Headers; |
| 279 | std::vector<BasicBlock*> Latches; |
| 280 | Headers.push_back(Header); |
| 281 | Latches.push_back(LatchBlock); |
| 282 | |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 283 | // The current on-the-fly SSA update requires blocks to be processed in |
| 284 | // reverse postorder so that LastValueMap contains the correct value at each |
| 285 | // exit. |
| 286 | LoopBlocksDFS DFS(L); |
Andrew Trick | 78b40c3 | 2011-08-10 01:59:05 +0000 | [diff] [blame] | 287 | DFS.perform(LI); |
| 288 | |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 289 | // Stash the DFS iterators before adding blocks to the loop. |
| 290 | LoopBlocksDFS::RPOIterator BlockBegin = DFS.beginRPO(); |
| 291 | LoopBlocksDFS::RPOIterator BlockEnd = DFS.endRPO(); |
| 292 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 293 | for (unsigned It = 1; It != Count; ++It) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 294 | std::vector<BasicBlock*> NewBlocks; |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 295 | |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 296 | for (LoopBlocksDFS::RPOIterator BB = BlockBegin; BB != BlockEnd; ++BB) { |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 297 | ValueToValueMapTy VMap; |
| 298 | BasicBlock *New = CloneBasicBlock(*BB, VMap, "." + Twine(It)); |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 299 | Header->getParent()->getBasicBlockList().push_back(New); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 300 | |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 301 | // Loop over all of the PHI nodes in the block, changing them to use the |
| 302 | // incoming values from the previous block. |
| 303 | if (*BB == Header) |
| 304 | for (unsigned i = 0, e = OrigPHINode.size(); i != e; ++i) { |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 305 | PHINode *NewPHI = cast<PHINode>(VMap[OrigPHINode[i]]); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 306 | Value *InVal = NewPHI->getIncomingValueForBlock(LatchBlock); |
| 307 | if (Instruction *InValI = dyn_cast<Instruction>(InVal)) |
Dan Gohman | 18fa568 | 2009-12-18 01:24:09 +0000 | [diff] [blame] | 308 | if (It > 1 && L->contains(InValI)) |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 309 | InVal = LastValueMap[InValI]; |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 310 | VMap[OrigPHINode[i]] = InVal; |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 311 | New->getInstList().erase(NewPHI); |
| 312 | } |
| 313 | |
| 314 | // Update our running map of newest clones |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 315 | LastValueMap[*BB] = New; |
Devang Patel | b8f11de | 2010-06-23 23:55:51 +0000 | [diff] [blame] | 316 | for (ValueToValueMapTy::iterator VI = VMap.begin(), VE = VMap.end(); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 317 | VI != VE; ++VI) |
| 318 | LastValueMap[VI->first] = VI->second; |
| 319 | |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 320 | L->addBasicBlockToLoop(New, LI->getBase()); |
| 321 | |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 322 | // Add phi entries for newly created values to all exit blocks. |
| 323 | for (succ_iterator SI = succ_begin(*BB), SE = succ_end(*BB); |
| 324 | SI != SE; ++SI) { |
| 325 | if (L->contains(*SI)) |
| 326 | continue; |
| 327 | for (BasicBlock::iterator BBI = (*SI)->begin(); |
| 328 | PHINode *phi = dyn_cast<PHINode>(BBI); ++BBI) { |
| 329 | Value *Incoming = phi->getIncomingValueForBlock(*BB); |
| 330 | ValueToValueMapTy::iterator It = LastValueMap.find(Incoming); |
| 331 | if (It != LastValueMap.end()) |
| 332 | Incoming = It->second; |
| 333 | phi->addIncoming(Incoming, New); |
| 334 | } |
| 335 | } |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 336 | // Keep track of new headers and latches as we create them, so that |
| 337 | // we can insert the proper branches later. |
| 338 | if (*BB == Header) |
| 339 | Headers.push_back(New); |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 340 | if (*BB == LatchBlock) |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 341 | Latches.push_back(New); |
| 342 | |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 343 | NewBlocks.push_back(New); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 344 | } |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 345 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 346 | // Remap all instructions in the most recent iteration |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 347 | for (unsigned i = 0; i < NewBlocks.size(); ++i) |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 348 | for (BasicBlock::iterator I = NewBlocks[i]->begin(), |
| 349 | E = NewBlocks[i]->end(); I != E; ++I) |
Rafael Espindola | 229e38f | 2010-10-13 01:36:30 +0000 | [diff] [blame] | 350 | ::RemapInstruction(I, LastValueMap); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 351 | } |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 352 | |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 353 | // Loop over the PHI nodes in the original block, setting incoming values. |
| 354 | for (unsigned i = 0, e = OrigPHINode.size(); i != e; ++i) { |
| 355 | PHINode *PN = OrigPHINode[i]; |
| 356 | if (CompletelyUnroll) { |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 357 | PN->replaceAllUsesWith(PN->getIncomingValueForBlock(Preheader)); |
| 358 | Header->getInstList().erase(PN); |
| 359 | } |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 360 | else if (Count > 1) { |
| 361 | Value *InVal = PN->removeIncomingValue(LatchBlock, false); |
| 362 | // If this value was defined in the loop, take the value defined by the |
| 363 | // last iteration of the loop. |
| 364 | if (Instruction *InValI = dyn_cast<Instruction>(InVal)) { |
| 365 | if (L->contains(InValI)) |
| 366 | InVal = LastValueMap[InVal]; |
| 367 | } |
| 368 | assert(Latches.back() == LastValueMap[LatchBlock] && "bad last latch"); |
| 369 | PN->addIncoming(InVal, Latches.back()); |
| 370 | } |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 371 | } |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 372 | |
| 373 | // Now that all the basic blocks for the unrolled iterations are in place, |
| 374 | // set up the branches to connect them. |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 375 | for (unsigned i = 0, e = Latches.size(); i != e; ++i) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 376 | // The original branch was replicated in each unrolled iteration. |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 377 | BranchInst *Term = cast<BranchInst>(Latches[i]->getTerminator()); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 378 | |
| 379 | // The branch destination. |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 380 | unsigned j = (i + 1) % e; |
| 381 | BasicBlock *Dest = Headers[j]; |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 382 | bool NeedConditional = true; |
| 383 | |
Andrew Trick | d04d1529 | 2011-12-09 06:19:40 +0000 | [diff] [blame] | 384 | if (RuntimeTripCount && j != 0) { |
| 385 | NeedConditional = false; |
| 386 | } |
| 387 | |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 388 | // For a complete unroll, make the last iteration end with a branch |
| 389 | // to the exit block. |
| 390 | if (CompletelyUnroll && j == 0) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 391 | Dest = LoopExit; |
| 392 | NeedConditional = false; |
| 393 | } |
| 394 | |
| 395 | // If we know the trip count or a multiple of it, we can safely use an |
| 396 | // unconditional branch for some iterations. |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 397 | if (j != BreakoutTrip && (TripMultiple == 0 || j % TripMultiple != 0)) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 398 | NeedConditional = false; |
| 399 | } |
| 400 | |
| 401 | if (NeedConditional) { |
| 402 | // Update the conditional branch's successor for the following |
| 403 | // iteration. |
| 404 | Term->setSuccessor(!ContinueOnTrue, Dest); |
| 405 | } else { |
Andrew Trick | b72bbe2 | 2011-08-10 00:28:10 +0000 | [diff] [blame] | 406 | // Remove phi operands at this loop exit |
| 407 | if (Dest != LoopExit) { |
| 408 | BasicBlock *BB = Latches[i]; |
| 409 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); |
| 410 | SI != SE; ++SI) { |
| 411 | if (*SI == Headers[i]) |
| 412 | continue; |
| 413 | for (BasicBlock::iterator BBI = (*SI)->begin(); |
| 414 | PHINode *Phi = dyn_cast<PHINode>(BBI); ++BBI) { |
| 415 | Phi->removeIncomingValue(BB, false); |
| 416 | } |
| 417 | } |
| 418 | } |
Jay Foad | 89afb43 | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 419 | // Replace the conditional branch with an unconditional one. |
| 420 | BranchInst::Create(Dest, Term); |
| 421 | Term->eraseFromParent(); |
Jay Foad | a97a2c9 | 2011-06-21 10:33:19 +0000 | [diff] [blame] | 422 | } |
| 423 | } |
| 424 | |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 425 | // Merge adjacent basic blocks, if possible. |
| 426 | for (unsigned i = 0, e = Latches.size(); i != e; ++i) { |
| 427 | BranchInst *Term = cast<BranchInst>(Latches[i]->getTerminator()); |
| 428 | if (Term->isUnconditional()) { |
| 429 | BasicBlock *Dest = Term->getSuccessor(0); |
Andrew Trick | bf69d03 | 2011-08-03 18:32:11 +0000 | [diff] [blame] | 430 | if (BasicBlock *Fold = FoldBlockIntoPredecessor(Dest, LI, LPM)) |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 431 | std::replace(Latches.begin(), Latches.end(), Dest, Fold); |
| 432 | } |
| 433 | } |
Andrew Trick | 279e7a6 | 2011-07-23 00:29:16 +0000 | [diff] [blame] | 434 | |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 435 | DominatorTree *DT = nullptr; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 436 | if (PP) { |
Andrew Trick | d29cd73 | 2012-05-08 02:52:09 +0000 | [diff] [blame] | 437 | // FIXME: Reconstruct dom info, because it is not preserved properly. |
| 438 | // Incrementally updating domtree after loop unrolling would be easy. |
Chandler Carruth | 7352302 | 2014-01-13 13:07:17 +0000 | [diff] [blame] | 439 | if (DominatorTreeWrapperPass *DTWP = |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 440 | PP->getAnalysisIfAvailable<DominatorTreeWrapperPass>()) { |
| 441 | DT = &DTWP->getDomTree(); |
| 442 | DT->recalculate(*L->getHeader()->getParent()); |
| 443 | } |
Andrew Trick | 4d0040b | 2011-08-10 04:29:49 +0000 | [diff] [blame] | 444 | |
Andrew Trick | d29cd73 | 2012-05-08 02:52:09 +0000 | [diff] [blame] | 445 | // Simplify any new induction variables in the partially unrolled loop. |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 446 | ScalarEvolution *SE = PP->getAnalysisIfAvailable<ScalarEvolution>(); |
Andrew Trick | d29cd73 | 2012-05-08 02:52:09 +0000 | [diff] [blame] | 447 | if (SE && !CompletelyUnroll) { |
| 448 | SmallVector<WeakVH, 16> DeadInsts; |
| 449 | simplifyLoopIVs(L, SE, LPM, DeadInsts); |
Andrew Trick | 4d0040b | 2011-08-10 04:29:49 +0000 | [diff] [blame] | 450 | |
Andrew Trick | d29cd73 | 2012-05-08 02:52:09 +0000 | [diff] [blame] | 451 | // Aggressively clean up dead instructions that simplifyLoopIVs already |
| 452 | // identified. Any remaining should be cleaned up below. |
| 453 | while (!DeadInsts.empty()) |
| 454 | if (Instruction *Inst = |
| 455 | dyn_cast_or_null<Instruction>(&*DeadInsts.pop_back_val())) |
| 456 | RecursivelyDeleteTriviallyDeadInstructions(Inst); |
| 457 | } |
Andrew Trick | 4d0040b | 2011-08-10 04:29:49 +0000 | [diff] [blame] | 458 | } |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 459 | // At this point, the code is well formed. We now do a quick sweep over the |
| 460 | // inserted code, doing constant propagation and dead code elimination as we |
| 461 | // go. |
| 462 | const std::vector<BasicBlock*> &NewLoopBlocks = L->getBlocks(); |
| 463 | for (std::vector<BasicBlock*>::const_iterator BB = NewLoopBlocks.begin(), |
| 464 | BBE = NewLoopBlocks.end(); BB != BBE; ++BB) |
| 465 | for (BasicBlock::iterator I = (*BB)->begin(), E = (*BB)->end(); I != E; ) { |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 466 | Instruction *Inst = I++; |
| 467 | |
| 468 | if (isInstructionTriviallyDead(Inst)) |
Dan Gohman | 04c8bd7 | 2008-06-24 20:44:42 +0000 | [diff] [blame] | 469 | (*BB)->getInstList().erase(Inst); |
Duncan Sands | 433c167 | 2010-11-23 20:26:33 +0000 | [diff] [blame] | 470 | else if (Value *V = SimplifyInstruction(Inst)) |
| 471 | if (LI->replacementPreservesLCSSAForm(Inst, V)) { |
| 472 | Inst->replaceAllUsesWith(V); |
| 473 | (*BB)->getInstList().erase(Inst); |
| 474 | } |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 475 | } |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 476 | |
| 477 | NumCompletelyUnrolled += CompletelyUnroll; |
| 478 | ++NumUnrolled; |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 479 | |
| 480 | Loop *OuterL = L->getParentLoop(); |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 481 | // Remove the loop from the LoopPassManager if it's completely removed. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 482 | if (CompletelyUnroll && LPM != nullptr) |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 483 | LPM->deleteLoopFromQueue(L); |
| 484 | |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 485 | // If we have a pass and a DominatorTree we should re-simplify impacted loops |
| 486 | // to ensure subsequent analyses can rely on this form. We want to simplify |
| 487 | // at least one layer outside of the loop that was unrolled so that any |
| 488 | // changes to the parent loop exposed by the unrolling are considered. |
| 489 | if (PP && DT) { |
| 490 | if (!OuterL && !CompletelyUnroll) |
| 491 | OuterL = L; |
Chandler Carruth | d84f776 | 2014-01-28 01:25:38 +0000 | [diff] [blame] | 492 | if (OuterL) { |
Hal Finkel | a995f92 | 2014-07-10 14:41:31 +0000 | [diff] [blame^] | 493 | DataLayoutPass *DLP = PP->getAnalysisIfAvailable<DataLayoutPass>(); |
| 494 | const DataLayout *DL = DLP ? &DLP->getDataLayout() : nullptr; |
Chandler Carruth | d84f776 | 2014-01-28 01:25:38 +0000 | [diff] [blame] | 495 | ScalarEvolution *SE = PP->getAnalysisIfAvailable<ScalarEvolution>(); |
Hal Finkel | a995f92 | 2014-07-10 14:41:31 +0000 | [diff] [blame^] | 496 | simplifyLoop(OuterL, DT, LI, PP, /*AliasAnalysis*/ nullptr, SE, DL); |
Dinesh Dwivedi | d266cb1 | 2014-05-29 06:47:23 +0000 | [diff] [blame] | 497 | |
| 498 | // LCSSA must be performed on the outermost affected loop. The unrolled |
| 499 | // loop's last loop latch is guaranteed to be in the outermost loop after |
| 500 | // deleteLoopFromQueue updates LoopInfo. |
| 501 | Loop *LatchLoop = LI->getLoopFor(Latches.back()); |
| 502 | if (!OuterL->contains(LatchLoop)) |
| 503 | while (OuterL->getParentLoop() != LatchLoop) |
| 504 | OuterL = OuterL->getParentLoop(); |
| 505 | |
Chandler Carruth | d84f776 | 2014-01-28 01:25:38 +0000 | [diff] [blame] | 506 | formLCSSARecursively(*OuterL, *DT, SE); |
| 507 | } |
Chandler Carruth | aa7fa5e | 2014-01-23 11:23:19 +0000 | [diff] [blame] | 508 | } |
| 509 | |
Dan Gohman | 3dc2d92 | 2008-05-14 00:24:14 +0000 | [diff] [blame] | 510 | return true; |
| 511 | } |