blob: d13e4abff9dc0f05f94b238b91170a907db718a6 [file] [log] [blame]
Chris Lattnere15051d2008-05-14 20:38:44 +00001//===- SimplifyCFGPass.cpp - CFG Simplification Pass ----------------------===//
Misha Brukmanb1c93172005-04-21 23:48:37 +00002//
John Criswell482202a2003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
Chris Lattnerf3ebc3f2007-12-29 20:36:04 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Misha Brukmanb1c93172005-04-21 23:48:37 +00007//
John Criswell482202a2003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattnerfa7dad82002-05-21 20:49:37 +00009//
Chris Lattner19970e62007-12-03 19:43:18 +000010// This file implements dead code elimination and basic block merging, along
11// with a collection of other peephole control flow optimizations. For example:
Chris Lattnerfa7dad82002-05-21 20:49:37 +000012//
Gordon Henriksend5687672007-11-04 16:15:04 +000013// * Removes basic blocks with no predecessors.
14// * Merges a basic block into its predecessor if there is only one and the
Chris Lattnerfa7dad82002-05-21 20:49:37 +000015// predecessor only has one successor.
Gordon Henriksend5687672007-11-04 16:15:04 +000016// * Eliminates PHI nodes for basic blocks with a single predecessor.
17// * Eliminates a basic block that only contains an unconditional branch.
Chris Lattner19970e62007-12-03 19:43:18 +000018// * Changes invoke instructions to nounwind functions to be calls.
19// * Change things like "if (x) if (y)" into "if (x&y)".
20// * etc..
Chris Lattnerfa7dad82002-05-21 20:49:37 +000021//
22//===----------------------------------------------------------------------===//
23
Chris Lattner79a42ac2006-12-19 21:40:18 +000024#define DEBUG_TYPE "simplifycfg"
Chris Lattnerfa7dad82002-05-21 20:49:37 +000025#include "llvm/Transforms/Scalar.h"
26#include "llvm/Transforms/Utils/Local.h"
Chris Lattnera67dd322004-10-18 03:00:50 +000027#include "llvm/Constants.h"
28#include "llvm/Instructions.h"
Bill Wendling55e69d12010-03-14 10:40:55 +000029#include "llvm/IntrinsicInst.h"
Chris Lattnerfa7dad82002-05-21 20:49:37 +000030#include "llvm/Module.h"
Devang Patelba3fa6c2008-09-23 23:03:40 +000031#include "llvm/Attributes.h"
Chris Lattnerfa7dad82002-05-21 20:49:37 +000032#include "llvm/Support/CFG.h"
33#include "llvm/Pass.h"
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +000034#include "llvm/Target/TargetData.h"
Chris Lattner61ce4df2007-11-13 07:32:38 +000035#include "llvm/ADT/SmallVector.h"
Chris Lattnera5403a52007-03-04 04:20:48 +000036#include "llvm/ADT/SmallPtrSet.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000037#include "llvm/ADT/Statistic.h"
Chris Lattner49525f82004-01-09 06:02:20 +000038using namespace llvm;
Brian Gaeke960707c2003-11-11 22:41:34 +000039
Chris Lattner79a42ac2006-12-19 21:40:18 +000040STATISTIC(NumSimpl, "Number of blocks simplified");
Chris Lattnerbf3a0992002-10-01 22:38:41 +000041
Chris Lattner79a42ac2006-12-19 21:40:18 +000042namespace {
Chris Lattner2dd09db2009-09-02 06:11:42 +000043 struct CFGSimplifyPass : public FunctionPass {
Nick Lewyckye7da2d62007-05-06 13:37:16 +000044 static char ID; // Pass identification, replacement for typeid
Owen Anderson6c18d1a2010-10-19 17:21:58 +000045 CFGSimplifyPass() : FunctionPass(ID) {
46 initializeCFGSimplifyPassPass(*PassRegistry::getPassRegistry());
47 }
Devang Patel09f162c2007-05-01 21:15:47 +000048
Chris Lattner113f4f42002-06-25 16:13:24 +000049 virtual bool runOnFunction(Function &F);
Chris Lattnerfa7dad82002-05-21 20:49:37 +000050 };
51}
52
Dan Gohmand78c4002008-05-13 00:00:25 +000053char CFGSimplifyPass::ID = 0;
Owen Andersona57b97e2010-07-21 22:09:45 +000054INITIALIZE_PASS(CFGSimplifyPass, "simplifycfg",
Owen Andersondf7a4f22010-10-07 22:25:06 +000055 "Simplify the CFG", false, false)
Dan Gohmand78c4002008-05-13 00:00:25 +000056
Brian Gaeke960707c2003-11-11 22:41:34 +000057// Public interface to the CFGSimplification pass
Chris Lattner49525f82004-01-09 06:02:20 +000058FunctionPass *llvm::createCFGSimplificationPass() {
Chris Lattnerfa7dad82002-05-21 20:49:37 +000059 return new CFGSimplifyPass();
60}
61
Chris Lattnera77e74e2007-11-14 06:19:25 +000062/// ChangeToUnreachable - Insert an unreachable instruction before the specified
63/// instruction, making it and the rest of the code in the block dead.
Chris Lattner84d46182010-05-08 22:15:59 +000064static void ChangeToUnreachable(Instruction *I, bool UseLLVMTrap) {
Chris Lattnera77e74e2007-11-14 06:19:25 +000065 BasicBlock *BB = I->getParent();
66 // Loop over all of the successors, removing BB's entry from any PHI
67 // nodes.
68 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI)
69 (*SI)->removePredecessor(BB);
Nadav Rotem465834c2012-07-24 10:51:42 +000070
Chris Lattner84d46182010-05-08 22:15:59 +000071 // Insert a call to llvm.trap right before this. This turns the undefined
72 // behavior into a hard fail instead of falling through into random code.
73 if (UseLLVMTrap) {
74 Function *TrapFn =
75 Intrinsic::getDeclaration(BB->getParent()->getParent(), Intrinsic::trap);
Devang Patel12bf0ab2011-04-27 17:59:27 +000076 CallInst *CallTrap = CallInst::Create(TrapFn, "", I);
77 CallTrap->setDebugLoc(I->getDebugLoc());
Chris Lattner84d46182010-05-08 22:15:59 +000078 }
Owen Anderson55f1c092009-08-13 21:58:54 +000079 new UnreachableInst(I->getContext(), I);
Nadav Rotem465834c2012-07-24 10:51:42 +000080
Chris Lattnera77e74e2007-11-14 06:19:25 +000081 // All instructions after this are dead.
82 BasicBlock::iterator BBI = I, BBE = BB->end();
83 while (BBI != BBE) {
84 if (!BBI->use_empty())
Owen Andersonb292b8c2009-07-30 23:03:37 +000085 BBI->replaceAllUsesWith(UndefValue::get(BBI->getType()));
Chris Lattnera77e74e2007-11-14 06:19:25 +000086 BB->getInstList().erase(BBI++);
87 }
88}
89
Duncan Sandsa915b532007-11-22 22:24:59 +000090/// ChangeToCall - Convert the specified invoke into a normal call.
91static void ChangeToCall(InvokeInst *II) {
Gabor Greifa2fbc0a2010-03-24 13:21:49 +000092 SmallVector<Value*, 8> Args(II->op_begin(), II->op_end() - 3);
Jay Foad5bd375a2011-07-15 08:37:34 +000093 CallInst *NewCall = CallInst::Create(II->getCalledValue(), Args, "", II);
Duncan Sandsa915b532007-11-22 22:24:59 +000094 NewCall->takeName(II);
95 NewCall->setCallingConv(II->getCallingConv());
Devang Patel4c758ea2008-09-25 21:00:45 +000096 NewCall->setAttributes(II->getAttributes());
Devang Patel5127c5d2011-06-02 22:46:58 +000097 NewCall->setDebugLoc(II->getDebugLoc());
Duncan Sandsa915b532007-11-22 22:24:59 +000098 II->replaceAllUsesWith(NewCall);
99
100 // Follow the call by a branch to the normal destination.
Gabor Greife9ecc682008-04-06 20:25:17 +0000101 BranchInst::Create(II->getNormalDest(), II);
Nuno Lopesd0bcfe42012-07-02 16:14:47 +0000102
103 // Update PHI nodes in the unwind destination
104 II->getUnwindDest()->removePredecessor(II->getParent());
Nuno Lopesb97a4e82012-06-28 22:32:27 +0000105 II->eraseFromParent();
Duncan Sandsa915b532007-11-22 22:24:59 +0000106}
107
Chris Lattnera5403a52007-03-04 04:20:48 +0000108static bool MarkAliveBlocks(BasicBlock *BB,
Nick Lewycky621fe562009-11-23 03:34:29 +0000109 SmallPtrSet<BasicBlock*, 128> &Reachable) {
Nadav Rotem465834c2012-07-24 10:51:42 +0000110
Chris Lattner61ce4df2007-11-13 07:32:38 +0000111 SmallVector<BasicBlock*, 128> Worklist;
Chris Lattner108083e2007-04-05 01:27:02 +0000112 Worklist.push_back(BB);
113 bool Changed = false;
Dan Gohman28943872010-01-05 16:27:25 +0000114 do {
115 BB = Worklist.pop_back_val();
Nadav Rotem465834c2012-07-24 10:51:42 +0000116
Chris Lattner108083e2007-04-05 01:27:02 +0000117 if (!Reachable.insert(BB))
118 continue;
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000119
Chris Lattner108083e2007-04-05 01:27:02 +0000120 // Do a quick scan of the basic block, turning any obviously unreachable
121 // instructions into LLVM unreachable insts. The instruction combining pass
Duncan Sandsa915b532007-11-22 22:24:59 +0000122 // canonicalizes unreachable insts into stores to null or undef.
Chris Lattnera77e74e2007-11-14 06:19:25 +0000123 for (BasicBlock::iterator BBI = BB->begin(), E = BB->end(); BBI != E;++BBI){
124 if (CallInst *CI = dyn_cast<CallInst>(BBI)) {
Duncan Sands3353ed02007-12-18 09:59:50 +0000125 if (CI->doesNotReturn()) {
Chris Lattnera77e74e2007-11-14 06:19:25 +0000126 // If we found a call to a no-return function, insert an unreachable
127 // instruction after it. Make sure there isn't *already* one there
128 // though.
129 ++BBI;
130 if (!isa<UnreachableInst>(BBI)) {
Chris Lattner84d46182010-05-08 22:15:59 +0000131 // Don't insert a call to llvm.trap right before the unreachable.
132 ChangeToUnreachable(BBI, false);
Chris Lattnera77e74e2007-11-14 06:19:25 +0000133 Changed = true;
Chris Lattner108083e2007-04-05 01:27:02 +0000134 }
135 break;
Chris Lattnera67dd322004-10-18 03:00:50 +0000136 }
Chris Lattnera77e74e2007-11-14 06:19:25 +0000137 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000138
Chris Lattner37536b92009-11-01 18:42:03 +0000139 // Store to undef and store to null are undefined and used to signal that
140 // they should be changed to unreachable by passes that can't modify the
141 // CFG.
Chris Lattner3dd5c5d2009-06-12 21:01:07 +0000142 if (StoreInst *SI = dyn_cast<StoreInst>(BBI)) {
Benjamin Kramerb82de422010-06-13 14:35:54 +0000143 // Don't touch volatile stores.
144 if (SI->isVolatile()) continue;
145
Chris Lattner3dd5c5d2009-06-12 21:01:07 +0000146 Value *Ptr = SI->getOperand(1);
Nadav Rotem465834c2012-07-24 10:51:42 +0000147
Dale Johannesen3be62692009-07-14 17:48:25 +0000148 if (isa<UndefValue>(Ptr) ||
149 (isa<ConstantPointerNull>(Ptr) &&
Chris Lattnera3e620c2009-08-30 20:06:40 +0000150 SI->getPointerAddressSpace() == 0)) {
Chris Lattner84d46182010-05-08 22:15:59 +0000151 ChangeToUnreachable(SI, true);
Dale Johannesen3be62692009-07-14 17:48:25 +0000152 Changed = true;
153 break;
154 }
Chris Lattner3dd5c5d2009-06-12 21:01:07 +0000155 }
Chris Lattnera77e74e2007-11-14 06:19:25 +0000156 }
Chris Lattnera67dd322004-10-18 03:00:50 +0000157
Duncan Sandsa915b532007-11-22 22:24:59 +0000158 // Turn invokes that call 'nounwind' functions into ordinary calls.
Nuno Lopes07594cb2012-06-25 17:11:47 +0000159 if (InvokeInst *II = dyn_cast<InvokeInst>(BB->getTerminator())) {
160 Value *Callee = II->getCalledValue();
161 if (isa<ConstantPointerNull>(Callee) || isa<UndefValue>(Callee)) {
162 ChangeToUnreachable(II, true);
163 Changed = true;
164 } else if (II->doesNotThrow()) {
Nuno Lopesb97a4e82012-06-28 22:32:27 +0000165 if (II->use_empty() && II->onlyReadsMemory()) {
166 // jump to the normal destination branch.
167 BranchInst::Create(II->getNormalDest(), II);
Nuno Lopes482fb192012-07-16 22:49:40 +0000168 II->getUnwindDest()->removePredecessor(II->getParent());
Nuno Lopesb97a4e82012-06-28 22:32:27 +0000169 II->eraseFromParent();
170 } else
171 ChangeToCall(II);
Duncan Sandsa915b532007-11-22 22:24:59 +0000172 Changed = true;
173 }
Nuno Lopes07594cb2012-06-25 17:11:47 +0000174 }
Duncan Sandsa915b532007-11-22 22:24:59 +0000175
Frits van Bommelad964552011-05-22 16:24:18 +0000176 Changed |= ConstantFoldTerminator(BB, true);
Chris Lattner108083e2007-04-05 01:27:02 +0000177 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI)
178 Worklist.push_back(*SI);
Dan Gohman28943872010-01-05 16:27:25 +0000179 } while (!Worklist.empty());
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000180 return Changed;
181}
182
Nadav Rotem465834c2012-07-24 10:51:42 +0000183/// RemoveUnreachableBlocksFromFn - Remove blocks that are not reachable, even
184/// if they are in a dead cycle. Return true if a change was made, false
Devang Patel3d560512008-09-08 22:14:17 +0000185/// otherwise.
186static bool RemoveUnreachableBlocksFromFn(Function &F) {
Chris Lattner61ce4df2007-11-13 07:32:38 +0000187 SmallPtrSet<BasicBlock*, 128> Reachable;
Nick Lewycky621fe562009-11-23 03:34:29 +0000188 bool Changed = MarkAliveBlocks(F.begin(), Reachable);
Nadav Rotem465834c2012-07-24 10:51:42 +0000189
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000190 // If there are unreachable blocks in the CFG...
Bill Wendling50794832008-01-26 06:51:24 +0000191 if (Reachable.size() == F.size())
Chris Lattner61ce4df2007-11-13 07:32:38 +0000192 return Changed;
Nadav Rotem465834c2012-07-24 10:51:42 +0000193
Chris Lattner61ce4df2007-11-13 07:32:38 +0000194 assert(Reachable.size() < F.size());
195 NumSimpl += F.size()-Reachable.size();
Nadav Rotem465834c2012-07-24 10:51:42 +0000196
Chris Lattner61ce4df2007-11-13 07:32:38 +0000197 // Loop over all of the basic blocks that are not reachable, dropping all of
198 // their internal references...
Chris Lattner89acd612008-02-14 07:39:01 +0000199 for (Function::iterator BB = ++F.begin(), E = F.end(); BB != E; ++BB) {
200 if (Reachable.count(BB))
201 continue;
Nadav Rotem465834c2012-07-24 10:51:42 +0000202
Chris Lattner89acd612008-02-14 07:39:01 +0000203 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI)
204 if (Reachable.count(*SI))
205 (*SI)->removePredecessor(BB);
206 BB->dropAllReferences();
207 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000208
Chris Lattner61ce4df2007-11-13 07:32:38 +0000209 for (Function::iterator I = ++F.begin(); I != F.end();)
210 if (!Reachable.count(I))
211 I = F.getBasicBlockList().erase(I);
212 else
213 ++I;
Nadav Rotem465834c2012-07-24 10:51:42 +0000214
Chris Lattner61ce4df2007-11-13 07:32:38 +0000215 return true;
216}
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000217
Chris Lattnerf21a2202009-12-22 06:07:30 +0000218/// MergeEmptyReturnBlocks - If we have more than one empty (other than phi
219/// node) return blocks, merge them together to promote recursive block merging.
220static bool MergeEmptyReturnBlocks(Function &F) {
221 bool Changed = false;
Nadav Rotem465834c2012-07-24 10:51:42 +0000222
Chris Lattnerf21a2202009-12-22 06:07:30 +0000223 BasicBlock *RetBlock = 0;
Nadav Rotem465834c2012-07-24 10:51:42 +0000224
Chris Lattnerf21a2202009-12-22 06:07:30 +0000225 // Scan all the blocks in the function, looking for empty return blocks.
226 for (Function::iterator BBI = F.begin(), E = F.end(); BBI != E; ) {
227 BasicBlock &BB = *BBI++;
Nadav Rotem465834c2012-07-24 10:51:42 +0000228
Chris Lattnerf21a2202009-12-22 06:07:30 +0000229 // Only look at return blocks.
230 ReturnInst *Ret = dyn_cast<ReturnInst>(BB.getTerminator());
231 if (Ret == 0) continue;
Nadav Rotem465834c2012-07-24 10:51:42 +0000232
Chris Lattnerf21a2202009-12-22 06:07:30 +0000233 // Only look at the block if it is empty or the only other thing in it is a
234 // single PHI node that is the operand to the return.
235 if (Ret != &BB.front()) {
236 // Check for something else in the block.
237 BasicBlock::iterator I = Ret;
238 --I;
Bill Wendling55e69d12010-03-14 10:40:55 +0000239 // Skip over debug info.
240 while (isa<DbgInfoIntrinsic>(I) && I != BB.begin())
241 --I;
242 if (!isa<DbgInfoIntrinsic>(I) &&
243 (!isa<PHINode>(I) || I != BB.begin() ||
244 Ret->getNumOperands() == 0 ||
245 Ret->getOperand(0) != I))
Chris Lattnerf21a2202009-12-22 06:07:30 +0000246 continue;
247 }
Bill Wendling55e69d12010-03-14 10:40:55 +0000248
Chris Lattnerf21a2202009-12-22 06:07:30 +0000249 // If this is the first returning block, remember it and keep going.
250 if (RetBlock == 0) {
251 RetBlock = &BB;
252 continue;
253 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000254
Chris Lattnerf21a2202009-12-22 06:07:30 +0000255 // Otherwise, we found a duplicate return block. Merge the two.
256 Changed = true;
Nadav Rotem465834c2012-07-24 10:51:42 +0000257
Chris Lattnerf21a2202009-12-22 06:07:30 +0000258 // Case when there is no input to the return or when the returned values
259 // agree is trivial. Note that they can't agree if there are phis in the
260 // blocks.
261 if (Ret->getNumOperands() == 0 ||
Nadav Rotem465834c2012-07-24 10:51:42 +0000262 Ret->getOperand(0) ==
Chris Lattnerf21a2202009-12-22 06:07:30 +0000263 cast<ReturnInst>(RetBlock->getTerminator())->getOperand(0)) {
264 BB.replaceAllUsesWith(RetBlock);
265 BB.eraseFromParent();
266 continue;
267 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000268
Chris Lattnerf21a2202009-12-22 06:07:30 +0000269 // If the canonical return block has no PHI node, create one now.
270 PHINode *RetBlockPHI = dyn_cast<PHINode>(RetBlock->begin());
271 if (RetBlockPHI == 0) {
Devang Pateld3f41e82010-03-15 19:05:46 +0000272 Value *InVal = cast<ReturnInst>(RetBlock->getTerminator())->getOperand(0);
Jay Foade0938d82011-03-30 11:19:20 +0000273 pred_iterator PB = pred_begin(RetBlock), PE = pred_end(RetBlock);
Jay Foad52131342011-03-30 11:28:46 +0000274 RetBlockPHI = PHINode::Create(Ret->getOperand(0)->getType(),
275 std::distance(PB, PE), "merge",
Chris Lattnerf21a2202009-12-22 06:07:30 +0000276 &RetBlock->front());
Nadav Rotem465834c2012-07-24 10:51:42 +0000277
Jay Foade0938d82011-03-30 11:19:20 +0000278 for (pred_iterator PI = PB; PI != PE; ++PI)
Chris Lattnerf21a2202009-12-22 06:07:30 +0000279 RetBlockPHI->addIncoming(InVal, *PI);
280 RetBlock->getTerminator()->setOperand(0, RetBlockPHI);
281 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000282
Chris Lattnerf21a2202009-12-22 06:07:30 +0000283 // Turn BB into a block that just unconditionally branches to the return
284 // block. This handles the case when the two return blocks have a common
285 // predecessor but that return different things.
286 RetBlockPHI->addIncoming(Ret->getOperand(0), &BB);
287 BB.getTerminator()->eraseFromParent();
288 BranchInst::Create(RetBlock, &BB);
289 }
Nadav Rotem465834c2012-07-24 10:51:42 +0000290
Chris Lattnerf21a2202009-12-22 06:07:30 +0000291 return Changed;
292}
293
Chris Lattner61ce4df2007-11-13 07:32:38 +0000294/// IterativeSimplifyCFG - Call SimplifyCFG on all the blocks in the function,
295/// iterating until no more changes are made.
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000296static bool IterativeSimplifyCFG(Function &F, const TargetData *TD) {
Chris Lattner61ce4df2007-11-13 07:32:38 +0000297 bool Changed = false;
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000298 bool LocalChange = true;
299 while (LocalChange) {
300 LocalChange = false;
Nadav Rotem465834c2012-07-24 10:51:42 +0000301
Dan Gohman4a63fad2010-08-14 00:29:42 +0000302 // Loop over all of the basic blocks and remove them if they are unneeded...
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000303 //
Dan Gohman4a63fad2010-08-14 00:29:42 +0000304 for (Function::iterator BBIt = F.begin(); BBIt != F.end(); ) {
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000305 if (SimplifyCFG(BBIt++, TD)) {
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000306 LocalChange = true;
307 ++NumSimpl;
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000308 }
309 }
310 Changed |= LocalChange;
311 }
Chris Lattnerfa7dad82002-05-21 20:49:37 +0000312 return Changed;
313}
Chris Lattner61ce4df2007-11-13 07:32:38 +0000314
315// It is possible that we may require multiple passes over the code to fully
316// simplify the CFG.
317//
318bool CFGSimplifyPass::runOnFunction(Function &F) {
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000319 const TargetData *TD = getAnalysisIfAvailable<TargetData>();
Devang Patel3d560512008-09-08 22:14:17 +0000320 bool EverChanged = RemoveUnreachableBlocksFromFn(F);
Chris Lattnerf21a2202009-12-22 06:07:30 +0000321 EverChanged |= MergeEmptyReturnBlocks(F);
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000322 EverChanged |= IterativeSimplifyCFG(F, TD);
323
Chris Lattner61ce4df2007-11-13 07:32:38 +0000324 // If neither pass changed anything, we're done.
325 if (!EverChanged) return false;
326
327 // IterativeSimplifyCFG can (rarely) make some loops dead. If this happens,
Devang Patel3d560512008-09-08 22:14:17 +0000328 // RemoveUnreachableBlocksFromFn is needed to nuke them, which means we should
Chris Lattner61ce4df2007-11-13 07:32:38 +0000329 // iterate between the two optimizations. We structure the code like this to
Nadav Rotem465834c2012-07-24 10:51:42 +0000330 // avoid reruning IterativeSimplifyCFG if the second pass of
Devang Patel3d560512008-09-08 22:14:17 +0000331 // RemoveUnreachableBlocksFromFn doesn't do anything.
332 if (!RemoveUnreachableBlocksFromFn(F))
Chris Lattner61ce4df2007-11-13 07:32:38 +0000333 return true;
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000334
Chris Lattner61ce4df2007-11-13 07:32:38 +0000335 do {
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000336 EverChanged = IterativeSimplifyCFG(F, TD);
Devang Patel3d560512008-09-08 22:14:17 +0000337 EverChanged |= RemoveUnreachableBlocksFromFn(F);
Chris Lattner61ce4df2007-11-13 07:32:38 +0000338 } while (EverChanged);
Jakob Stoklund Olesen916f48a2010-02-05 22:03:18 +0000339
Chris Lattner61ce4df2007-11-13 07:32:38 +0000340 return true;
341}