Owen Anderson | 2480737 | 2006-05-26 21:11:53 +0000 | [diff] [blame] | 1 | //===-- LCSSA.cpp - Convert loops into loop-closed SSA form ---------------===// |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This pass transforms loops by placing phi nodes at the end of the loops for |
| 11 | // all values that are live across the loop boundary. For example, it turns |
| 12 | // the left into the right code: |
| 13 | // |
| 14 | // for (...) for (...) |
Dan Gohman | 23d9d27 | 2007-05-11 21:10:54 +0000 | [diff] [blame] | 15 | // if (c) if (c) |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 16 | // X1 = ... X1 = ... |
| 17 | // else else |
| 18 | // X2 = ... X2 = ... |
| 19 | // X3 = phi(X1, X2) X3 = phi(X1, X2) |
Dan Gohman | c702a9e | 2008-06-03 00:57:21 +0000 | [diff] [blame] | 20 | // ... = X3 + 4 X4 = phi(X3) |
| 21 | // ... = X4 + 4 |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 22 | // |
| 23 | // This is still valid LLVM; the extra phi nodes are purely redundant, and will |
| 24 | // be trivially eliminated by InstCombine. The major benefit of this |
| 25 | // transformation is that it makes many other loop optimizations, such as |
| 26 | // LoopUnswitching, simpler. |
| 27 | // |
| 28 | //===----------------------------------------------------------------------===// |
| 29 | |
Chris Lattner | d216e8b | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 30 | #define DEBUG_TYPE "lcssa" |
Owen Anderson | 2480737 | 2006-05-26 21:11:53 +0000 | [diff] [blame] | 31 | #include "llvm/Transforms/Scalar.h" |
Owen Anderson | e4e1ecd | 2006-07-09 08:14:06 +0000 | [diff] [blame] | 32 | #include "llvm/Constants.h" |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 33 | #include "llvm/Pass.h" |
| 34 | #include "llvm/Function.h" |
| 35 | #include "llvm/Instructions.h" |
Owen Anderson | 2480737 | 2006-05-26 21:11:53 +0000 | [diff] [blame] | 36 | #include "llvm/Analysis/Dominators.h" |
Devang Patel | b4559a2 | 2007-07-13 23:57:11 +0000 | [diff] [blame] | 37 | #include "llvm/Analysis/LoopPass.h" |
| 38 | #include "llvm/Analysis/ScalarEvolution.h" |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 39 | #include "llvm/Transforms/Utils/SSAUpdater.h" |
| 40 | #include "llvm/ADT/Statistic.h" |
| 41 | #include "llvm/ADT/STLExtras.h" |
Owen Anderson | 68fbd73 | 2009-04-22 08:09:13 +0000 | [diff] [blame] | 42 | #include "llvm/Support/PredIteratorCache.h" |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 43 | using namespace llvm; |
| 44 | |
Chris Lattner | d216e8b | 2006-12-19 22:17:40 +0000 | [diff] [blame] | 45 | STATISTIC(NumLCSSA, "Number of live out of a loop variables"); |
| 46 | |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 47 | namespace { |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 48 | struct LCSSA : public LoopPass { |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 49 | static char ID; // Pass identification, replacement for typeid |
Owen Anderson | 081c34b | 2010-10-19 17:21:58 +0000 | [diff] [blame] | 50 | LCSSA() : LoopPass(ID) { |
| 51 | initializeLCSSAPass(*PassRegistry::getPassRegistry()); |
| 52 | } |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 53 | |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 54 | // Cached analysis information for the current function. |
Evan Cheng | cc045d5 | 2007-04-18 22:39:00 +0000 | [diff] [blame] | 55 | DominatorTree *DT; |
Owen Anderson | 92deacf | 2006-06-06 04:28:30 +0000 | [diff] [blame] | 56 | std::vector<BasicBlock*> LoopBlocks; |
Owen Anderson | 68fbd73 | 2009-04-22 08:09:13 +0000 | [diff] [blame] | 57 | PredIteratorCache PredCache; |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 58 | Loop *L; |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 59 | |
Devang Patel | b4559a2 | 2007-07-13 23:57:11 +0000 | [diff] [blame] | 60 | virtual bool runOnLoop(Loop *L, LPPassManager &LPM); |
| 61 | |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 62 | /// This transformation requires natural loop information & requires that |
Owen Anderson | a90b2c7 | 2006-05-27 00:31:37 +0000 | [diff] [blame] | 63 | /// loop preheaders be inserted into the CFG. It maintains both of these, |
| 64 | /// as well as the CFG. It also requires dominator information. |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 65 | /// |
| 66 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
Owen Anderson | 2480737 | 2006-05-26 21:11:53 +0000 | [diff] [blame] | 67 | AU.setPreservesCFG(); |
Dan Gohman | f6572d0 | 2009-11-09 18:28:24 +0000 | [diff] [blame] | 68 | |
Dan Gohman | 052f000 | 2010-07-26 18:11:16 +0000 | [diff] [blame] | 69 | AU.addRequired<DominatorTree>(); |
Dan Gohman | 052f000 | 2010-07-26 18:11:16 +0000 | [diff] [blame] | 70 | AU.addRequired<LoopInfo>(); |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 71 | AU.addPreservedID(LoopSimplifyID); |
Dan Gohman | 1e381fc | 2010-07-16 17:58:45 +0000 | [diff] [blame] | 72 | AU.addPreserved<ScalarEvolution>(); |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 73 | } |
| 74 | private: |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 75 | bool ProcessInstruction(Instruction *Inst, |
| 76 | const SmallVectorImpl<BasicBlock*> &ExitBlocks); |
Chris Lattner | 39b0c3d | 2009-10-11 01:07:15 +0000 | [diff] [blame] | 77 | |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 78 | /// verifyAnalysis() - Verify loop nest. |
| 79 | virtual void verifyAnalysis() const { |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 80 | // Check the special guarantees that LCSSA makes. |
Dan Gohman | bbf81d8 | 2010-03-10 19:38:49 +0000 | [diff] [blame] | 81 | assert(L->isLCSSAForm(*DT) && "LCSSA form not preserved!"); |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 82 | } |
| 83 | |
Owen Anderson | 92deacf | 2006-06-06 04:28:30 +0000 | [diff] [blame] | 84 | /// inLoop - returns true if the given block is within the current loop |
Chris Lattner | 39b0c3d | 2009-10-11 01:07:15 +0000 | [diff] [blame] | 85 | bool inLoop(BasicBlock *B) const { |
Owen Anderson | e9d93d5 | 2006-06-06 04:36:36 +0000 | [diff] [blame] | 86 | return std::binary_search(LoopBlocks.begin(), LoopBlocks.end(), B); |
| 87 | } |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 88 | }; |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 89 | } |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 90 | |
| 91 | char LCSSA::ID = 0; |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 92 | INITIALIZE_PASS_BEGIN(LCSSA, "lcssa", "Loop-Closed SSA Form Pass", false, false) |
| 93 | INITIALIZE_PASS_DEPENDENCY(DominatorTree) |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 94 | INITIALIZE_PASS_DEPENDENCY(LoopInfo) |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 95 | INITIALIZE_PASS_END(LCSSA, "lcssa", "Loop-Closed SSA Form Pass", false, false) |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 96 | |
Daniel Dunbar | 394f044 | 2008-10-22 23:32:42 +0000 | [diff] [blame] | 97 | Pass *llvm::createLCSSAPass() { return new LCSSA(); } |
Owen Anderson | 90c579d | 2010-08-06 18:33:48 +0000 | [diff] [blame] | 98 | char &llvm::LCSSAID = LCSSA::ID; |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 99 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 100 | |
| 101 | /// BlockDominatesAnExit - Return true if the specified block dominates at least |
| 102 | /// one of the blocks in the specified list. |
| 103 | static bool BlockDominatesAnExit(BasicBlock *BB, |
| 104 | const SmallVectorImpl<BasicBlock*> &ExitBlocks, |
| 105 | DominatorTree *DT) { |
| 106 | DomTreeNode *DomNode = DT->getNode(BB); |
| 107 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) |
| 108 | if (DT->dominates(DomNode, DT->getNode(ExitBlocks[i]))) |
| 109 | return true; |
| 110 | |
| 111 | return false; |
| 112 | } |
| 113 | |
| 114 | |
Owen Anderson | a452932 | 2006-06-08 20:02:53 +0000 | [diff] [blame] | 115 | /// runOnFunction - Process all loops in the function, inner-most out. |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 116 | bool LCSSA::runOnLoop(Loop *TheLoop, LPPassManager &LPM) { |
| 117 | L = TheLoop; |
Owen Anderson | 7be3f1e | 2006-06-13 19:37:18 +0000 | [diff] [blame] | 118 | |
Evan Cheng | cc045d5 | 2007-04-18 22:39:00 +0000 | [diff] [blame] | 119 | DT = &getAnalysis<DominatorTree>(); |
Devang Patel | 96bf524 | 2007-08-17 21:59:16 +0000 | [diff] [blame] | 120 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 121 | // Get the set of exiting blocks. |
| 122 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| 123 | L->getExitBlocks(ExitBlocks); |
| 124 | |
| 125 | if (ExitBlocks.empty()) |
| 126 | return false; |
| 127 | |
Chris Lattner | 39b0c3d | 2009-10-11 01:07:15 +0000 | [diff] [blame] | 128 | // Speed up queries by creating a sorted vector of blocks. |
Owen Anderson | 92deacf | 2006-06-06 04:28:30 +0000 | [diff] [blame] | 129 | LoopBlocks.clear(); |
| 130 | LoopBlocks.insert(LoopBlocks.end(), L->block_begin(), L->block_end()); |
Chris Lattner | 39b0c3d | 2009-10-11 01:07:15 +0000 | [diff] [blame] | 131 | array_pod_sort(LoopBlocks.begin(), LoopBlocks.end()); |
Owen Anderson | 2480737 | 2006-05-26 21:11:53 +0000 | [diff] [blame] | 132 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 133 | // Look at all the instructions in the loop, checking to see if they have uses |
| 134 | // outside the loop. If so, rewrite those uses. |
| 135 | bool MadeChange = false; |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 136 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 137 | for (Loop::block_iterator BBI = L->block_begin(), E = L->block_end(); |
| 138 | BBI != E; ++BBI) { |
| 139 | BasicBlock *BB = *BBI; |
| 140 | |
| 141 | // For large loops, avoid use-scanning by using dominance information: In |
| 142 | // particular, if a block does not dominate any of the loop exits, then none |
| 143 | // of the values defined in the block could be used outside the loop. |
| 144 | if (!BlockDominatesAnExit(BB, ExitBlocks, DT)) |
| 145 | continue; |
| 146 | |
| 147 | for (BasicBlock::iterator I = BB->begin(), E = BB->end(); |
| 148 | I != E; ++I) { |
| 149 | // Reject two common cases fast: instructions with no uses (like stores) |
| 150 | // and instructions with one use that is in the same block as this. |
| 151 | if (I->use_empty() || |
| 152 | (I->hasOneUse() && I->use_back()->getParent() == BB && |
| 153 | !isa<PHINode>(I->use_back()))) |
| 154 | continue; |
| 155 | |
| 156 | MadeChange |= ProcessInstruction(I, ExitBlocks); |
| 157 | } |
| 158 | } |
Owen Anderson | 00ea74c | 2006-05-29 01:00:00 +0000 | [diff] [blame] | 159 | |
Dan Gohman | bbf81d8 | 2010-03-10 19:38:49 +0000 | [diff] [blame] | 160 | assert(L->isLCSSAForm(*DT)); |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 161 | PredCache.clear(); |
| 162 | |
| 163 | return MadeChange; |
Owen Anderson | 11f510b | 2006-05-26 13:58:26 +0000 | [diff] [blame] | 164 | } |
| 165 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 166 | /// isExitBlock - Return true if the specified block is in the list. |
| 167 | static bool isExitBlock(BasicBlock *BB, |
| 168 | const SmallVectorImpl<BasicBlock*> &ExitBlocks) { |
| 169 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) |
| 170 | if (ExitBlocks[i] == BB) |
| 171 | return true; |
| 172 | return false; |
| 173 | } |
| 174 | |
| 175 | /// ProcessInstruction - Given an instruction in the loop, check to see if it |
| 176 | /// has any uses that are outside the current loop. If so, insert LCSSA PHI |
| 177 | /// nodes and rewrite the uses. |
| 178 | bool LCSSA::ProcessInstruction(Instruction *Inst, |
| 179 | const SmallVectorImpl<BasicBlock*> &ExitBlocks) { |
| 180 | SmallVector<Use*, 16> UsesToRewrite; |
| 181 | |
| 182 | BasicBlock *InstBB = Inst->getParent(); |
| 183 | |
| 184 | for (Value::use_iterator UI = Inst->use_begin(), E = Inst->use_end(); |
| 185 | UI != E; ++UI) { |
Gabor Greif | a29742d | 2010-07-09 14:29:14 +0000 | [diff] [blame] | 186 | User *U = *UI; |
| 187 | BasicBlock *UserBB = cast<Instruction>(U)->getParent(); |
| 188 | if (PHINode *PN = dyn_cast<PHINode>(U)) |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 189 | UserBB = PN->getIncomingBlock(UI); |
| 190 | |
| 191 | if (InstBB != UserBB && !inLoop(UserBB)) |
| 192 | UsesToRewrite.push_back(&UI.getUse()); |
| 193 | } |
Gabor Greif | a29742d | 2010-07-09 14:29:14 +0000 | [diff] [blame] | 194 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 195 | // If there are no uses outside the loop, exit with no change. |
| 196 | if (UsesToRewrite.empty()) return false; |
| 197 | |
Owen Anderson | 408a406 | 2006-05-31 20:55:06 +0000 | [diff] [blame] | 198 | ++NumLCSSA; // We are applying the transformation |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 199 | |
Dan Gohman | 6b9c959 | 2009-06-26 00:31:13 +0000 | [diff] [blame] | 200 | // Invoke instructions are special in that their result value is not available |
| 201 | // along their unwind edge. The code below tests to see whether DomBB dominates |
| 202 | // the value, so adjust DomBB to the normal destination block, which is |
| 203 | // effectively where the value is first usable. |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 204 | BasicBlock *DomBB = Inst->getParent(); |
| 205 | if (InvokeInst *Inv = dyn_cast<InvokeInst>(Inst)) |
Dan Gohman | 6b9c959 | 2009-06-26 00:31:13 +0000 | [diff] [blame] | 206 | DomBB = Inv->getNormalDest(); |
| 207 | |
| 208 | DomTreeNode *DomNode = DT->getNode(DomBB); |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 209 | |
Cameron Zwarich | 838b97e | 2011-03-15 07:41:25 +0000 | [diff] [blame] | 210 | SmallVector<PHINode*, 16> AddedPHIs; |
| 211 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 212 | SSAUpdater SSAUpdate; |
Duncan Sands | fc6e29d | 2010-09-02 08:14:03 +0000 | [diff] [blame] | 213 | SSAUpdate.Initialize(Inst->getType(), Inst->getName()); |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 214 | |
| 215 | // Insert the LCSSA phi's into all of the exit blocks dominated by the |
Dan Gohman | f6572d0 | 2009-11-09 18:28:24 +0000 | [diff] [blame] | 216 | // value, and add them to the Phi's map. |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 217 | for (SmallVectorImpl<BasicBlock*>::const_iterator BBI = ExitBlocks.begin(), |
Chris Lattner | 39b0c3d | 2009-10-11 01:07:15 +0000 | [diff] [blame] | 218 | BBE = ExitBlocks.end(); BBI != BBE; ++BBI) { |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 219 | BasicBlock *ExitBB = *BBI; |
| 220 | if (!DT->dominates(DomNode, DT->getNode(ExitBB))) continue; |
| 221 | |
| 222 | // If we already inserted something for this BB, don't reprocess it. |
| 223 | if (SSAUpdate.HasValueForBlock(ExitBB)) continue; |
| 224 | |
Jay Foad | 3ecfc86 | 2011-03-30 11:28:46 +0000 | [diff] [blame^] | 225 | PHINode *PN = PHINode::Create(Inst->getType(), |
| 226 | PredCache.GetNumPreds(ExitBB), |
| 227 | Inst->getName()+".lcssa", |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 228 | ExitBB->begin()); |
Chris Lattner | cbea67f | 2006-10-31 18:56:48 +0000 | [diff] [blame] | 229 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 230 | // Add inputs from inside the loop for this PHI. |
Dan Gohman | f6572d0 | 2009-11-09 18:28:24 +0000 | [diff] [blame] | 231 | for (BasicBlock **PI = PredCache.GetPreds(ExitBB); *PI; ++PI) { |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 232 | PN->addIncoming(Inst, *PI); |
Dan Gohman | f6572d0 | 2009-11-09 18:28:24 +0000 | [diff] [blame] | 233 | |
| 234 | // If the exit block has a predecessor not within the loop, arrange for |
Dan Gohman | eaa181c | 2009-11-09 18:59:22 +0000 | [diff] [blame] | 235 | // the incoming value use corresponding to that predecessor to be |
Dan Gohman | f6572d0 | 2009-11-09 18:28:24 +0000 | [diff] [blame] | 236 | // rewritten in terms of a different LCSSA PHI. |
| 237 | if (!inLoop(*PI)) |
| 238 | UsesToRewrite.push_back( |
| 239 | &PN->getOperandUse( |
| 240 | PN->getOperandNumForIncomingValue(PN->getNumIncomingValues()-1))); |
| 241 | } |
Cameron Zwarich | 838b97e | 2011-03-15 07:41:25 +0000 | [diff] [blame] | 242 | |
| 243 | AddedPHIs.push_back(PN); |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 244 | |
| 245 | // Remember that this phi makes the value alive in this block. |
| 246 | SSAUpdate.AddAvailableValue(ExitBB, PN); |
Owen Anderson | 3d2aa47 | 2006-06-12 07:10:16 +0000 | [diff] [blame] | 247 | } |
Owen Anderson | 408a406 | 2006-05-31 20:55:06 +0000 | [diff] [blame] | 248 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 249 | // Rewrite all uses outside the loop in terms of the new PHIs we just |
| 250 | // inserted. |
| 251 | for (unsigned i = 0, e = UsesToRewrite.size(); i != e; ++i) { |
| 252 | // If this use is in an exit block, rewrite to use the newly inserted PHI. |
| 253 | // This is required for correctness because SSAUpdate doesn't handle uses in |
| 254 | // the same block. It assumes the PHI we inserted is at the end of the |
| 255 | // block. |
| 256 | Instruction *User = cast<Instruction>(UsesToRewrite[i]->getUser()); |
| 257 | BasicBlock *UserBB = User->getParent(); |
| 258 | if (PHINode *PN = dyn_cast<PHINode>(User)) |
| 259 | UserBB = PN->getIncomingBlock(*UsesToRewrite[i]); |
| 260 | |
| 261 | if (isa<PHINode>(UserBB->begin()) && |
| 262 | isExitBlock(UserBB, ExitBlocks)) { |
| 263 | UsesToRewrite[i]->set(UserBB->begin()); |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 264 | continue; |
Owen Anderson | 2824da4 | 2006-06-01 06:05:47 +0000 | [diff] [blame] | 265 | } |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 266 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 267 | // Otherwise, do full PHI insertion. |
| 268 | SSAUpdate.RewriteUse(*UsesToRewrite[i]); |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 269 | } |
Cameron Zwarich | 838b97e | 2011-03-15 07:41:25 +0000 | [diff] [blame] | 270 | |
| 271 | // Remove PHI nodes that did not have any uses rewritten. |
| 272 | for (unsigned i = 0, e = AddedPHIs.size(); i != e; ++i) { |
Cameron Zwarich | 6e51c6a | 2011-03-15 18:42:33 +0000 | [diff] [blame] | 273 | if (AddedPHIs[i]->use_empty()) |
Cameron Zwarich | 838b97e | 2011-03-15 07:41:25 +0000 | [diff] [blame] | 274 | AddedPHIs[i]->eraseFromParent(); |
| 275 | } |
Owen Anderson | e4e1ecd | 2006-07-09 08:14:06 +0000 | [diff] [blame] | 276 | |
Chris Lattner | 8c1db67 | 2009-10-11 02:53:37 +0000 | [diff] [blame] | 277 | return true; |
Owen Anderson | 408a406 | 2006-05-31 20:55:06 +0000 | [diff] [blame] | 278 | } |
Chris Lattner | d41ae8b | 2006-08-02 00:06:09 +0000 | [diff] [blame] | 279 | |