blob: 52ad1ee0704295773683dc8e70688fac384483b7 [file] [log] [blame]
Chris Lattner01d1ee32002-05-21 20:50:24 +00001//===- SimplifyCFG.cpp - Code to perform CFG simplification ---------------===//
Misha Brukmanfd939082005-04-21 23:48:37 +00002//
John Criswellb576c942003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
Chris Lattner4ee451d2007-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 Brukmanfd939082005-04-21 23:48:37 +00007//
John Criswellb576c942003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattner01d1ee32002-05-21 20:50:24 +00009//
Chris Lattnerbb190ac2002-10-08 21:36:33 +000010// Peephole optimize the CFG.
Chris Lattner01d1ee32002-05-21 20:50:24 +000011//
12//===----------------------------------------------------------------------===//
13
Chris Lattner218a8222004-06-20 01:13:18 +000014#define DEBUG_TYPE "simplifycfg"
Chris Lattner01d1ee32002-05-21 20:50:24 +000015#include "llvm/Transforms/Utils/Local.h"
Chris Lattner723c66d2004-02-11 03:36:04 +000016#include "llvm/Constants.h"
17#include "llvm/Instructions.h"
Devang Patel383d7ed2009-02-03 22:12:02 +000018#include "llvm/IntrinsicInst.h"
Chris Lattner0d560082004-02-24 05:38:11 +000019#include "llvm/Type.h"
Reid Spencerc1030572007-01-19 21:13:56 +000020#include "llvm/DerivedTypes.h"
Dale Johannesenf8bc3002009-05-13 18:25:07 +000021#include "llvm/GlobalVariable.h"
Chris Lattner01d1ee32002-05-21 20:50:24 +000022#include "llvm/Support/CFG.h"
Reid Spencer551ccae2004-09-01 22:55:40 +000023#include "llvm/Support/Debug.h"
Daniel Dunbarce63ffb2009-07-25 00:23:56 +000024#include "llvm/Support/raw_ostream.h"
Chris Lattner79066fa2007-01-30 23:46:24 +000025#include "llvm/Analysis/ConstantFolding.h"
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +000026#include "llvm/Target/TargetData.h"
Chris Lattnereaba3a12005-09-19 23:49:37 +000027#include "llvm/Transforms/Utils/BasicBlockUtils.h"
Dan Gohman2c635662009-10-30 22:39:04 +000028#include "llvm/ADT/DenseMap.h"
Chris Lattner93e985f2007-02-13 02:10:56 +000029#include "llvm/ADT/SmallVector.h"
Chris Lattnerc9951232007-04-02 01:44:59 +000030#include "llvm/ADT/SmallPtrSet.h"
Evan Cheng502a4f52008-06-12 21:15:59 +000031#include "llvm/ADT/Statistic.h"
Chris Lattner6d4d21e2010-12-13 02:00:58 +000032#include "llvm/ADT/STLExtras.h"
Chris Lattner01d1ee32002-05-21 20:50:24 +000033#include <algorithm>
Chris Lattnerd52c2612004-02-24 07:23:58 +000034#include <set>
Chris Lattner698f96f2004-10-18 04:07:22 +000035#include <map>
Chris Lattnerf7703df2004-01-09 06:12:26 +000036using namespace llvm;
Brian Gaeked0fde302003-11-11 22:41:34 +000037
Evan Cheng502a4f52008-06-12 21:15:59 +000038STATISTIC(NumSpeculations, "Number of speculative executed instructions");
39
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +000040namespace {
41class SimplifyCFGOpt {
42 const TargetData *const TD;
43
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +000044 Value *isValueEqualityComparison(TerminatorInst *TI);
45 BasicBlock *GetValueEqualityComparisonCases(TerminatorInst *TI,
46 std::vector<std::pair<ConstantInt*, BasicBlock*> > &Cases);
47 bool SimplifyEqualityComparisonWithOnlyPredecessor(TerminatorInst *TI,
48 BasicBlock *Pred);
49 bool FoldValueComparisonIntoPredecessors(TerminatorInst *TI);
50
51public:
52 explicit SimplifyCFGOpt(const TargetData *td) : TD(td) {}
53 bool run(BasicBlock *BB);
54};
55}
56
Chris Lattner2bdcb562005-08-03 00:19:45 +000057/// SafeToMergeTerminators - Return true if it is safe to merge these two
58/// terminator instructions together.
59///
60static bool SafeToMergeTerminators(TerminatorInst *SI1, TerminatorInst *SI2) {
61 if (SI1 == SI2) return false; // Can't merge with self!
62
63 // It is not safe to merge these two switch instructions if they have a common
64 // successor, and if that successor has a PHI node, and if *that* PHI node has
65 // conflicting incoming values from the two switch blocks.
66 BasicBlock *SI1BB = SI1->getParent();
67 BasicBlock *SI2BB = SI2->getParent();
Chris Lattnerc9951232007-04-02 01:44:59 +000068 SmallPtrSet<BasicBlock*, 16> SI1Succs(succ_begin(SI1BB), succ_end(SI1BB));
Chris Lattner2bdcb562005-08-03 00:19:45 +000069
70 for (succ_iterator I = succ_begin(SI2BB), E = succ_end(SI2BB); I != E; ++I)
71 if (SI1Succs.count(*I))
72 for (BasicBlock::iterator BBI = (*I)->begin();
73 isa<PHINode>(BBI); ++BBI) {
74 PHINode *PN = cast<PHINode>(BBI);
75 if (PN->getIncomingValueForBlock(SI1BB) !=
76 PN->getIncomingValueForBlock(SI2BB))
77 return false;
78 }
79
80 return true;
81}
82
83/// AddPredecessorToBlock - Update PHI nodes in Succ to indicate that there will
84/// now be entries in it from the 'NewPred' block. The values that will be
85/// flowing into the PHI nodes will be the same as those coming in from
86/// ExistPred, an existing predecessor of Succ.
87static void AddPredecessorToBlock(BasicBlock *Succ, BasicBlock *NewPred,
88 BasicBlock *ExistPred) {
89 assert(std::find(succ_begin(ExistPred), succ_end(ExistPred), Succ) !=
90 succ_end(ExistPred) && "ExistPred is not a predecessor of Succ!");
91 if (!isa<PHINode>(Succ->begin())) return; // Quick exit if nothing to do
92
Chris Lattner093a4382008-07-13 22:23:11 +000093 PHINode *PN;
94 for (BasicBlock::iterator I = Succ->begin();
95 (PN = dyn_cast<PHINode>(I)); ++I)
96 PN->addIncoming(PN->getIncomingValueForBlock(ExistPred), NewPred);
Chris Lattner2bdcb562005-08-03 00:19:45 +000097}
98
Chris Lattner7e663482005-08-03 00:11:16 +000099
Chris Lattner723c66d2004-02-11 03:36:04 +0000100/// GetIfCondition - Given a basic block (BB) with two predecessors (and
101/// presumably PHI nodes in it), check to see if the merge at this block is due
102/// to an "if condition". If so, return the boolean condition that determines
103/// which entry into BB will be taken. Also, return by references the block
104/// that will be entered from if the condition is true, and the block that will
105/// be entered if the condition is false.
Misha Brukmanfd939082005-04-21 23:48:37 +0000106///
Chris Lattner723c66d2004-02-11 03:36:04 +0000107///
108static Value *GetIfCondition(BasicBlock *BB,
109 BasicBlock *&IfTrue, BasicBlock *&IfFalse) {
110 assert(std::distance(pred_begin(BB), pred_end(BB)) == 2 &&
111 "Function can only handle blocks with 2 predecessors!");
112 BasicBlock *Pred1 = *pred_begin(BB);
113 BasicBlock *Pred2 = *++pred_begin(BB);
114
115 // We can only handle branches. Other control flow will be lowered to
116 // branches if possible anyway.
117 if (!isa<BranchInst>(Pred1->getTerminator()) ||
118 !isa<BranchInst>(Pred2->getTerminator()))
119 return 0;
120 BranchInst *Pred1Br = cast<BranchInst>(Pred1->getTerminator());
121 BranchInst *Pred2Br = cast<BranchInst>(Pred2->getTerminator());
122
123 // Eliminate code duplication by ensuring that Pred1Br is conditional if
124 // either are.
125 if (Pred2Br->isConditional()) {
126 // If both branches are conditional, we don't have an "if statement". In
127 // reality, we could transform this case, but since the condition will be
128 // required anyway, we stand no chance of eliminating it, so the xform is
129 // probably not profitable.
130 if (Pred1Br->isConditional())
131 return 0;
132
133 std::swap(Pred1, Pred2);
134 std::swap(Pred1Br, Pred2Br);
135 }
136
137 if (Pred1Br->isConditional()) {
138 // If we found a conditional branch predecessor, make sure that it branches
139 // to BB and Pred2Br. If it doesn't, this isn't an "if statement".
140 if (Pred1Br->getSuccessor(0) == BB &&
141 Pred1Br->getSuccessor(1) == Pred2) {
142 IfTrue = Pred1;
143 IfFalse = Pred2;
144 } else if (Pred1Br->getSuccessor(0) == Pred2 &&
145 Pred1Br->getSuccessor(1) == BB) {
146 IfTrue = Pred2;
147 IfFalse = Pred1;
148 } else {
149 // We know that one arm of the conditional goes to BB, so the other must
150 // go somewhere unrelated, and this must not be an "if statement".
151 return 0;
152 }
153
154 // The only thing we have to watch out for here is to make sure that Pred2
155 // doesn't have incoming edges from other blocks. If it does, the condition
156 // doesn't dominate BB.
157 if (++pred_begin(Pred2) != pred_end(Pred2))
158 return 0;
159
160 return Pred1Br->getCondition();
161 }
162
163 // Ok, if we got here, both predecessors end with an unconditional branch to
164 // BB. Don't panic! If both blocks only have a single (identical)
165 // predecessor, and THAT is a conditional branch, then we're all ok!
166 if (pred_begin(Pred1) == pred_end(Pred1) ||
167 ++pred_begin(Pred1) != pred_end(Pred1) ||
168 pred_begin(Pred2) == pred_end(Pred2) ||
169 ++pred_begin(Pred2) != pred_end(Pred2) ||
170 *pred_begin(Pred1) != *pred_begin(Pred2))
171 return 0;
172
173 // Otherwise, if this is a conditional branch, then we can use it!
174 BasicBlock *CommonPred = *pred_begin(Pred1);
175 if (BranchInst *BI = dyn_cast<BranchInst>(CommonPred->getTerminator())) {
176 assert(BI->isConditional() && "Two successors but not conditional?");
177 if (BI->getSuccessor(0) == Pred1) {
178 IfTrue = Pred1;
179 IfFalse = Pred2;
180 } else {
181 IfTrue = Pred2;
182 IfFalse = Pred1;
183 }
184 return BI->getCondition();
185 }
186 return 0;
187}
188
Bill Wendling5049fa62009-01-19 23:43:56 +0000189/// DominatesMergePoint - If we have a merge point of an "if condition" as
190/// accepted above, return true if the specified value dominates the block. We
191/// don't handle the true generality of domination here, just a special case
192/// which works well enough for us.
193///
194/// If AggressiveInsts is non-null, and if V does not dominate BB, we check to
195/// see if V (which must be an instruction) is cheap to compute and is
196/// non-trapping. If both are true, the instruction is inserted into the set
197/// and true is returned.
Chris Lattner9c078662004-10-14 05:13:36 +0000198static bool DominatesMergePoint(Value *V, BasicBlock *BB,
199 std::set<Instruction*> *AggressiveInsts) {
Chris Lattner570751c2004-04-09 22:50:22 +0000200 Instruction *I = dyn_cast<Instruction>(V);
Chris Lattnerb74b1812006-10-20 00:42:07 +0000201 if (!I) {
202 // Non-instructions all dominate instructions, but not all constantexprs
203 // can be executed unconditionally.
204 if (ConstantExpr *C = dyn_cast<ConstantExpr>(V))
205 if (C->canTrap())
206 return false;
207 return true;
208 }
Chris Lattner570751c2004-04-09 22:50:22 +0000209 BasicBlock *PBB = I->getParent();
Chris Lattner723c66d2004-02-11 03:36:04 +0000210
Chris Lattnerda895d62005-02-27 06:18:25 +0000211 // We don't want to allow weird loops that might have the "if condition" in
Chris Lattner570751c2004-04-09 22:50:22 +0000212 // the bottom of this block.
213 if (PBB == BB) return false;
Chris Lattner723c66d2004-02-11 03:36:04 +0000214
Chris Lattner570751c2004-04-09 22:50:22 +0000215 // If this instruction is defined in a block that contains an unconditional
216 // branch to BB, then it must be in the 'conditional' part of the "if
217 // statement".
218 if (BranchInst *BI = dyn_cast<BranchInst>(PBB->getTerminator()))
219 if (BI->isUnconditional() && BI->getSuccessor(0) == BB) {
Chris Lattner9c078662004-10-14 05:13:36 +0000220 if (!AggressiveInsts) return false;
Chris Lattner570751c2004-04-09 22:50:22 +0000221 // Okay, it looks like the instruction IS in the "condition". Check to
Dan Gohman4bb31bf2010-03-30 20:04:57 +0000222 // see if it's a cheap instruction to unconditionally compute, and if it
Chris Lattner570751c2004-04-09 22:50:22 +0000223 // only uses stuff defined outside of the condition. If so, hoist it out.
Eli Friedman0b79a772009-07-17 04:28:42 +0000224 if (!I->isSafeToSpeculativelyExecute())
225 return false;
226
Chris Lattner570751c2004-04-09 22:50:22 +0000227 switch (I->getOpcode()) {
228 default: return false; // Cannot hoist this out safely.
Dale Johannesen3a56d142009-03-06 21:08:33 +0000229 case Instruction::Load: {
Eli Friedman0b79a772009-07-17 04:28:42 +0000230 // We have to check to make sure there are no instructions before the
231 // load in its basic block, as we are going to hoist the loop out to
232 // its predecessor.
Dale Johannesen3a56d142009-03-06 21:08:33 +0000233 BasicBlock::iterator IP = PBB->begin();
234 while (isa<DbgInfoIntrinsic>(IP))
235 IP++;
236 if (IP != BasicBlock::iterator(I))
Chris Lattner570751c2004-04-09 22:50:22 +0000237 return false;
238 break;
Dale Johannesen3a56d142009-03-06 21:08:33 +0000239 }
Chris Lattner570751c2004-04-09 22:50:22 +0000240 case Instruction::Add:
241 case Instruction::Sub:
242 case Instruction::And:
243 case Instruction::Or:
244 case Instruction::Xor:
245 case Instruction::Shl:
Reid Spencer3822ff52006-11-08 06:47:33 +0000246 case Instruction::LShr:
247 case Instruction::AShr:
Reid Spencere4d87aa2006-12-23 06:05:41 +0000248 case Instruction::ICmp:
Chris Lattner570751c2004-04-09 22:50:22 +0000249 break; // These are all cheap and non-trapping instructions.
250 }
Misha Brukmanfd939082005-04-21 23:48:37 +0000251
Chris Lattner570751c2004-04-09 22:50:22 +0000252 // Okay, we can only really hoist these out if their operands are not
253 // defined in the conditional region.
Gabor Greiff7ea3632008-06-10 22:03:26 +0000254 for (User::op_iterator i = I->op_begin(), e = I->op_end(); i != e; ++i)
255 if (!DominatesMergePoint(*i, BB, 0))
Chris Lattner570751c2004-04-09 22:50:22 +0000256 return false;
Chris Lattner9c078662004-10-14 05:13:36 +0000257 // Okay, it's safe to do this! Remember this instruction.
258 AggressiveInsts->insert(I);
Chris Lattner570751c2004-04-09 22:50:22 +0000259 }
260
Chris Lattner723c66d2004-02-11 03:36:04 +0000261 return true;
262}
Chris Lattner01d1ee32002-05-21 20:50:24 +0000263
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000264/// GetConstantInt - Extract ConstantInt from value, looking through IntToPtr
265/// and PointerNullValue. Return NULL if value is not a constant int.
Chris Lattner28acc132010-12-13 03:30:12 +0000266static ConstantInt *GetConstantInt(Value *V, const TargetData *TD) {
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000267 // Normal constant int.
268 ConstantInt *CI = dyn_cast<ConstantInt>(V);
Duncan Sands1df98592010-02-16 11:11:14 +0000269 if (CI || !TD || !isa<Constant>(V) || !V->getType()->isPointerTy())
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000270 return CI;
271
272 // This is some kind of pointer constant. Turn it into a pointer-sized
273 // ConstantInt if possible.
274 const IntegerType *PtrTy = TD->getIntPtrType(V->getContext());
275
276 // Null pointer means 0, see SelectionDAGBuilder::getValue(const Value*).
277 if (isa<ConstantPointerNull>(V))
278 return ConstantInt::get(PtrTy, 0);
279
280 // IntToPtr const int.
281 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(V))
282 if (CE->getOpcode() == Instruction::IntToPtr)
283 if (ConstantInt *CI = dyn_cast<ConstantInt>(CE->getOperand(0))) {
284 // The constant is very likely to have the right type already.
285 if (CI->getType() == PtrTy)
286 return CI;
287 else
288 return cast<ConstantInt>
289 (ConstantExpr::getIntegerCast(CI, PtrTy, /*isSigned=*/false));
290 }
291 return 0;
292}
293
Chris Lattner0aa749b2010-12-13 04:26:26 +0000294/// GatherConstantCompares - Given a potentially 'or'd or 'and'd together
295/// collection of icmp eq/ne instructions that compare a value against a
296/// constant, return the value being compared, and stick the constant into the
297/// Values vector.
Chris Lattner28acc132010-12-13 03:30:12 +0000298static Value *
Chris Lattner0aa749b2010-12-13 04:26:26 +0000299GatherConstantCompares(Value *V, std::vector<ConstantInt*> &Vals, Value *&Extra,
300 const TargetData *TD, bool isEQ) {
301 Instruction *I = dyn_cast<Instruction>(V);
302 if (I == 0) return 0;
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000303
Chris Lattner7312a222010-12-13 04:50:38 +0000304 // If this is an icmp against a constant, handle this as one of the cases.
Chris Lattner0aa749b2010-12-13 04:26:26 +0000305 if (ICmpInst *ICI = dyn_cast<ICmpInst>(I)) {
306 if (ICI->getPredicate() == (isEQ ? ICmpInst::ICMP_EQ : ICmpInst::ICMP_NE))
307 if (ConstantInt *C = GetConstantInt(I->getOperand(1), TD)) {
308 Vals.push_back(C);
309 return I->getOperand(0);
310 }
Chris Lattner662269d2010-12-13 04:18:32 +0000311 return 0;
312 }
313
Chris Lattner7312a222010-12-13 04:50:38 +0000314 // Otherwise, we can only handle an | or &, depending on isEQ.
Chris Lattner0aa749b2010-12-13 04:26:26 +0000315 if (I->getOpcode() != (isEQ ? Instruction::Or : Instruction::And))
Chris Lattner662269d2010-12-13 04:18:32 +0000316 return 0;
Chris Lattner662269d2010-12-13 04:18:32 +0000317
Chris Lattner7312a222010-12-13 04:50:38 +0000318 unsigned NumValsBeforeLHS = Vals.size();
Chris Lattner0aa749b2010-12-13 04:26:26 +0000319 if (Value *LHS = GatherConstantCompares(I->getOperand(0), Vals, Extra, TD,
320 isEQ)) {
Chris Lattner7312a222010-12-13 04:50:38 +0000321 unsigned NumVals = Vals.size();
Chris Lattner0aa749b2010-12-13 04:26:26 +0000322 if (Value *RHS = GatherConstantCompares(I->getOperand(1), Vals, Extra, TD,
323 isEQ)) {
324 if (LHS == RHS)
325 return LHS;
326 }
Chris Lattner7312a222010-12-13 04:50:38 +0000327 Vals.resize(NumVals);
328
329 // The RHS of the or/and can't be folded in and we haven't used "Extra" yet,
330 // set it and return success.
331 if (Extra == 0 || Extra == I->getOperand(1)) {
332 Extra = I->getOperand(1);
333 return LHS;
334 }
335
336 Vals.resize(NumValsBeforeLHS);
337 return 0;
Anton Korobeynikov07e6e562008-02-20 11:26:25 +0000338 }
Chris Lattner7312a222010-12-13 04:50:38 +0000339
340 // If the LHS can't be folded in, but Extra is available and RHS can, try to
341 // use LHS as Extra.
342 if (Extra == 0 || Extra == I->getOperand(0)) {
343 Extra = I->getOperand(0);
344 if (Value *RHS = GatherConstantCompares(I->getOperand(1), Vals, Extra, TD,
345 isEQ))
346 return RHS;
347 Vals.resize(NumValsBeforeLHS);
348 Extra = 0;
349 }
350
Chris Lattner0d560082004-02-24 05:38:11 +0000351 return 0;
352}
Chris Lattner0aa749b2010-12-13 04:26:26 +0000353
Eli Friedman080efb82008-12-16 20:54:32 +0000354static void EraseTerminatorInstAndDCECond(TerminatorInst *TI) {
355 Instruction* Cond = 0;
356 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
357 Cond = dyn_cast<Instruction>(SI->getCondition());
358 } else if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
359 if (BI->isConditional())
360 Cond = dyn_cast<Instruction>(BI->getCondition());
Frits van Bommel7ac40c32010-12-05 18:29:03 +0000361 } else if (IndirectBrInst *IBI = dyn_cast<IndirectBrInst>(TI)) {
362 Cond = dyn_cast<Instruction>(IBI->getAddress());
Eli Friedman080efb82008-12-16 20:54:32 +0000363 }
364
365 TI->eraseFromParent();
366 if (Cond) RecursivelyDeleteTriviallyDeadInstructions(Cond);
367}
368
Chris Lattner9fd49552008-11-27 23:25:44 +0000369/// isValueEqualityComparison - Return true if the specified terminator checks
370/// to see if a value is equal to constant integer value.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000371Value *SimplifyCFGOpt::isValueEqualityComparison(TerminatorInst *TI) {
372 Value *CV = 0;
Chris Lattner4bebf082004-03-16 19:45:22 +0000373 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
374 // Do not permit merging of large switch instructions into their
375 // predecessors unless there is only one predecessor.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000376 if (SI->getNumSuccessors()*std::distance(pred_begin(SI->getParent()),
377 pred_end(SI->getParent())) <= 128)
378 CV = SI->getCondition();
379 } else if (BranchInst *BI = dyn_cast<BranchInst>(TI))
Chris Lattner542f1492004-02-28 21:28:10 +0000380 if (BI->isConditional() && BI->getCondition()->hasOneUse())
Reid Spencere4d87aa2006-12-23 06:05:41 +0000381 if (ICmpInst *ICI = dyn_cast<ICmpInst>(BI->getCondition()))
382 if ((ICI->getPredicate() == ICmpInst::ICMP_EQ ||
383 ICI->getPredicate() == ICmpInst::ICMP_NE) &&
Chris Lattner28acc132010-12-13 03:30:12 +0000384 GetConstantInt(ICI->getOperand(1), TD))
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000385 CV = ICI->getOperand(0);
386
387 // Unwrap any lossless ptrtoint cast.
388 if (TD && CV && CV->getType() == TD->getIntPtrType(CV->getContext()))
389 if (PtrToIntInst *PTII = dyn_cast<PtrToIntInst>(CV))
390 CV = PTII->getOperand(0);
391 return CV;
Chris Lattner542f1492004-02-28 21:28:10 +0000392}
393
Bill Wendling5049fa62009-01-19 23:43:56 +0000394/// GetValueEqualityComparisonCases - Given a value comparison instruction,
395/// decode all of the 'cases' that it represents and return the 'default' block.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000396BasicBlock *SimplifyCFGOpt::
Misha Brukmanfd939082005-04-21 23:48:37 +0000397GetValueEqualityComparisonCases(TerminatorInst *TI,
Chris Lattner542f1492004-02-28 21:28:10 +0000398 std::vector<std::pair<ConstantInt*,
399 BasicBlock*> > &Cases) {
400 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
401 Cases.reserve(SI->getNumCases());
402 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
Chris Lattnerbe54dcc2005-02-26 18:33:28 +0000403 Cases.push_back(std::make_pair(SI->getCaseValue(i), SI->getSuccessor(i)));
Chris Lattner542f1492004-02-28 21:28:10 +0000404 return SI->getDefaultDest();
405 }
406
407 BranchInst *BI = cast<BranchInst>(TI);
Reid Spencere4d87aa2006-12-23 06:05:41 +0000408 ICmpInst *ICI = cast<ICmpInst>(BI->getCondition());
Chris Lattner28acc132010-12-13 03:30:12 +0000409 Cases.push_back(std::make_pair(GetConstantInt(ICI->getOperand(1), TD),
Reid Spencere4d87aa2006-12-23 06:05:41 +0000410 BI->getSuccessor(ICI->getPredicate() ==
411 ICmpInst::ICMP_NE)));
412 return BI->getSuccessor(ICI->getPredicate() == ICmpInst::ICMP_EQ);
Chris Lattner542f1492004-02-28 21:28:10 +0000413}
414
415
Bill Wendling5049fa62009-01-19 23:43:56 +0000416/// EliminateBlockCases - Given a vector of bb/value pairs, remove any entries
417/// in the list that match the specified block.
Misha Brukmanfd939082005-04-21 23:48:37 +0000418static void EliminateBlockCases(BasicBlock *BB,
Chris Lattner623369a2005-02-24 06:17:52 +0000419 std::vector<std::pair<ConstantInt*, BasicBlock*> > &Cases) {
420 for (unsigned i = 0, e = Cases.size(); i != e; ++i)
421 if (Cases[i].second == BB) {
422 Cases.erase(Cases.begin()+i);
423 --i; --e;
424 }
425}
426
Bill Wendling5049fa62009-01-19 23:43:56 +0000427/// ValuesOverlap - Return true if there are any keys in C1 that exist in C2 as
428/// well.
Chris Lattner623369a2005-02-24 06:17:52 +0000429static bool
430ValuesOverlap(std::vector<std::pair<ConstantInt*, BasicBlock*> > &C1,
431 std::vector<std::pair<ConstantInt*, BasicBlock*> > &C2) {
432 std::vector<std::pair<ConstantInt*, BasicBlock*> > *V1 = &C1, *V2 = &C2;
433
434 // Make V1 be smaller than V2.
435 if (V1->size() > V2->size())
436 std::swap(V1, V2);
437
438 if (V1->size() == 0) return false;
439 if (V1->size() == 1) {
440 // Just scan V2.
441 ConstantInt *TheVal = (*V1)[0].first;
442 for (unsigned i = 0, e = V2->size(); i != e; ++i)
443 if (TheVal == (*V2)[i].first)
444 return true;
445 }
446
447 // Otherwise, just sort both lists and compare element by element.
Chris Lattnerfca20f52010-12-13 03:24:30 +0000448 array_pod_sort(V1->begin(), V1->end());
449 array_pod_sort(V2->begin(), V2->end());
Chris Lattner623369a2005-02-24 06:17:52 +0000450 unsigned i1 = 0, i2 = 0, e1 = V1->size(), e2 = V2->size();
451 while (i1 != e1 && i2 != e2) {
452 if ((*V1)[i1].first == (*V2)[i2].first)
453 return true;
454 if ((*V1)[i1].first < (*V2)[i2].first)
455 ++i1;
456 else
457 ++i2;
458 }
459 return false;
460}
461
Bill Wendling5049fa62009-01-19 23:43:56 +0000462/// SimplifyEqualityComparisonWithOnlyPredecessor - If TI is known to be a
463/// terminator instruction and its block is known to only have a single
464/// predecessor block, check to see if that predecessor is also a value
465/// comparison with the same value, and if that comparison determines the
466/// outcome of this comparison. If so, simplify TI. This does a very limited
467/// form of jump threading.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000468bool SimplifyCFGOpt::
469SimplifyEqualityComparisonWithOnlyPredecessor(TerminatorInst *TI,
470 BasicBlock *Pred) {
Chris Lattner623369a2005-02-24 06:17:52 +0000471 Value *PredVal = isValueEqualityComparison(Pred->getTerminator());
472 if (!PredVal) return false; // Not a value comparison in predecessor.
473
474 Value *ThisVal = isValueEqualityComparison(TI);
475 assert(ThisVal && "This isn't a value comparison!!");
476 if (ThisVal != PredVal) return false; // Different predicates.
477
478 // Find out information about when control will move from Pred to TI's block.
479 std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
480 BasicBlock *PredDef = GetValueEqualityComparisonCases(Pred->getTerminator(),
481 PredCases);
482 EliminateBlockCases(PredDef, PredCases); // Remove default from cases.
Misha Brukmanfd939082005-04-21 23:48:37 +0000483
Chris Lattner623369a2005-02-24 06:17:52 +0000484 // Find information about how control leaves this block.
485 std::vector<std::pair<ConstantInt*, BasicBlock*> > ThisCases;
486 BasicBlock *ThisDef = GetValueEqualityComparisonCases(TI, ThisCases);
487 EliminateBlockCases(ThisDef, ThisCases); // Remove default from cases.
488
489 // If TI's block is the default block from Pred's comparison, potentially
490 // simplify TI based on this knowledge.
491 if (PredDef == TI->getParent()) {
492 // If we are here, we know that the value is none of those cases listed in
493 // PredCases. If there are any cases in ThisCases that are in PredCases, we
494 // can simplify TI.
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000495 if (!ValuesOverlap(PredCases, ThisCases))
496 return false;
497
498 if (isa<BranchInst>(TI)) {
499 // Okay, one of the successors of this condbr is dead. Convert it to a
500 // uncond br.
501 assert(ThisCases.size() == 1 && "Branch can only have one case!");
502 // Insert the new branch.
503 Instruction *NI = BranchInst::Create(ThisDef, TI);
504 (void) NI;
Chris Lattner623369a2005-02-24 06:17:52 +0000505
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000506 // Remove PHI node entries for the dead edge.
507 ThisCases[0].second->removePredecessor(TI->getParent());
Chris Lattner623369a2005-02-24 06:17:52 +0000508
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000509 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
510 << "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
Chris Lattner623369a2005-02-24 06:17:52 +0000511
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000512 EraseTerminatorInstAndDCECond(TI);
513 return true;
Chris Lattner623369a2005-02-24 06:17:52 +0000514 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000515
516 SwitchInst *SI = cast<SwitchInst>(TI);
517 // Okay, TI has cases that are statically dead, prune them away.
518 SmallPtrSet<Constant*, 16> DeadCases;
Chris Lattner623369a2005-02-24 06:17:52 +0000519 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000520 DeadCases.insert(PredCases[i].first);
Chris Lattner623369a2005-02-24 06:17:52 +0000521
David Greene89d6fd32010-01-05 01:26:52 +0000522 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000523 << "Through successor TI: " << *TI);
Chris Lattner623369a2005-02-24 06:17:52 +0000524
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000525 for (unsigned i = SI->getNumCases()-1; i != 0; --i)
526 if (DeadCases.count(SI->getCaseValue(i))) {
527 SI->getSuccessor(i)->removePredecessor(TI->getParent());
528 SI->removeCase(i);
529 }
530
531 DEBUG(dbgs() << "Leaving: " << *TI << "\n");
Chris Lattner623369a2005-02-24 06:17:52 +0000532 return true;
533 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000534
535 // Otherwise, TI's block must correspond to some matched value. Find out
536 // which value (or set of values) this is.
537 ConstantInt *TIV = 0;
538 BasicBlock *TIBB = TI->getParent();
539 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
540 if (PredCases[i].second == TIBB) {
541 if (TIV != 0)
542 return false; // Cannot handle multiple values coming to this block.
543 TIV = PredCases[i].first;
544 }
545 assert(TIV && "No edge from pred to succ?");
546
547 // Okay, we found the one constant that our value can be if we get into TI's
548 // BB. Find out which successor will unconditionally be branched to.
549 BasicBlock *TheRealDest = 0;
550 for (unsigned i = 0, e = ThisCases.size(); i != e; ++i)
551 if (ThisCases[i].first == TIV) {
552 TheRealDest = ThisCases[i].second;
553 break;
554 }
555
556 // If not handled by any explicit cases, it is handled by the default case.
557 if (TheRealDest == 0) TheRealDest = ThisDef;
558
559 // Remove PHI node entries for dead edges.
560 BasicBlock *CheckEdge = TheRealDest;
561 for (succ_iterator SI = succ_begin(TIBB), e = succ_end(TIBB); SI != e; ++SI)
562 if (*SI != CheckEdge)
563 (*SI)->removePredecessor(TIBB);
564 else
565 CheckEdge = 0;
566
567 // Insert the new branch.
568 Instruction *NI = BranchInst::Create(TheRealDest, TI);
569 (void) NI;
570
571 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
572 << "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
573
574 EraseTerminatorInstAndDCECond(TI);
575 return true;
Chris Lattner623369a2005-02-24 06:17:52 +0000576}
577
Dale Johannesenc81f5442009-03-12 21:01:11 +0000578namespace {
579 /// ConstantIntOrdering - This class implements a stable ordering of constant
580 /// integers that does not depend on their address. This is important for
581 /// applications that sort ConstantInt's to ensure uniqueness.
582 struct ConstantIntOrdering {
583 bool operator()(const ConstantInt *LHS, const ConstantInt *RHS) const {
584 return LHS->getValue().ult(RHS->getValue());
585 }
586 };
587}
Dale Johannesena9537cf2009-03-12 01:00:26 +0000588
Chris Lattner6d4d21e2010-12-13 02:00:58 +0000589static int ConstantIntSortPredicate(const void *P1, const void *P2) {
590 const ConstantInt *LHS = *(const ConstantInt**)P1;
591 const ConstantInt *RHS = *(const ConstantInt**)P2;
592 return LHS->getValue().ult(RHS->getValue());
593}
594
Bill Wendling5049fa62009-01-19 23:43:56 +0000595/// FoldValueComparisonIntoPredecessors - The specified terminator is a value
596/// equality comparison instruction (either a switch or a branch on "X == c").
597/// See if any of the predecessors of the terminator block are value comparisons
598/// on the same value. If so, and if safe to do so, fold them together.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000599bool SimplifyCFGOpt::FoldValueComparisonIntoPredecessors(TerminatorInst *TI) {
Chris Lattner542f1492004-02-28 21:28:10 +0000600 BasicBlock *BB = TI->getParent();
601 Value *CV = isValueEqualityComparison(TI); // CondVal
602 assert(CV && "Not a comparison?");
603 bool Changed = false;
604
Chris Lattner82442432008-02-18 07:42:56 +0000605 SmallVector<BasicBlock*, 16> Preds(pred_begin(BB), pred_end(BB));
Chris Lattner542f1492004-02-28 21:28:10 +0000606 while (!Preds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +0000607 BasicBlock *Pred = Preds.pop_back_val();
Misha Brukmanfd939082005-04-21 23:48:37 +0000608
Chris Lattner542f1492004-02-28 21:28:10 +0000609 // See if the predecessor is a comparison with the same value.
610 TerminatorInst *PTI = Pred->getTerminator();
611 Value *PCV = isValueEqualityComparison(PTI); // PredCondVal
612
613 if (PCV == CV && SafeToMergeTerminators(TI, PTI)) {
614 // Figure out which 'cases' to copy from SI to PSI.
615 std::vector<std::pair<ConstantInt*, BasicBlock*> > BBCases;
616 BasicBlock *BBDefault = GetValueEqualityComparisonCases(TI, BBCases);
617
618 std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
619 BasicBlock *PredDefault = GetValueEqualityComparisonCases(PTI, PredCases);
620
621 // Based on whether the default edge from PTI goes to BB or not, fill in
622 // PredCases and PredDefault with the new switch cases we would like to
623 // build.
Chris Lattner82442432008-02-18 07:42:56 +0000624 SmallVector<BasicBlock*, 8> NewSuccessors;
Chris Lattner542f1492004-02-28 21:28:10 +0000625
626 if (PredDefault == BB) {
627 // If this is the default destination from PTI, only the edges in TI
628 // that don't occur in PTI, or that branch to BB will be activated.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000629 std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
Chris Lattner542f1492004-02-28 21:28:10 +0000630 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
631 if (PredCases[i].second != BB)
632 PTIHandled.insert(PredCases[i].first);
633 else {
634 // The default destination is BB, we don't need explicit targets.
635 std::swap(PredCases[i], PredCases.back());
636 PredCases.pop_back();
637 --i; --e;
638 }
639
640 // Reconstruct the new switch statement we will be building.
641 if (PredDefault != BBDefault) {
642 PredDefault->removePredecessor(Pred);
643 PredDefault = BBDefault;
644 NewSuccessors.push_back(BBDefault);
645 }
646 for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
647 if (!PTIHandled.count(BBCases[i].first) &&
648 BBCases[i].second != BBDefault) {
649 PredCases.push_back(BBCases[i]);
650 NewSuccessors.push_back(BBCases[i].second);
651 }
652
653 } else {
654 // If this is not the default destination from PSI, only the edges
655 // in SI that occur in PSI with a destination of BB will be
656 // activated.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000657 std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
Chris Lattner542f1492004-02-28 21:28:10 +0000658 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
659 if (PredCases[i].second == BB) {
660 PTIHandled.insert(PredCases[i].first);
661 std::swap(PredCases[i], PredCases.back());
662 PredCases.pop_back();
663 --i; --e;
664 }
665
666 // Okay, now we know which constants were sent to BB from the
667 // predecessor. Figure out where they will all go now.
668 for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
669 if (PTIHandled.count(BBCases[i].first)) {
670 // If this is one we are capable of getting...
671 PredCases.push_back(BBCases[i]);
672 NewSuccessors.push_back(BBCases[i].second);
673 PTIHandled.erase(BBCases[i].first);// This constant is taken care of
674 }
675
676 // If there are any constants vectored to BB that TI doesn't handle,
677 // they must go to the default destination of TI.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000678 for (std::set<ConstantInt*, ConstantIntOrdering>::iterator I =
679 PTIHandled.begin(),
Chris Lattner542f1492004-02-28 21:28:10 +0000680 E = PTIHandled.end(); I != E; ++I) {
681 PredCases.push_back(std::make_pair(*I, BBDefault));
682 NewSuccessors.push_back(BBDefault);
683 }
684 }
685
686 // Okay, at this point, we know which new successor Pred will get. Make
687 // sure we update the number of entries in the PHI nodes for these
688 // successors.
689 for (unsigned i = 0, e = NewSuccessors.size(); i != e; ++i)
690 AddPredecessorToBlock(NewSuccessors[i], Pred, BB);
691
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000692 // Convert pointer to int before we switch.
Duncan Sands1df98592010-02-16 11:11:14 +0000693 if (CV->getType()->isPointerTy()) {
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000694 assert(TD && "Cannot switch on pointer without TargetData");
695 CV = new PtrToIntInst(CV, TD->getIntPtrType(CV->getContext()),
696 "magicptr", PTI);
697 }
698
Chris Lattner542f1492004-02-28 21:28:10 +0000699 // Now that the successors are updated, create the new Switch instruction.
Gabor Greifb1dbcd82008-05-15 10:04:30 +0000700 SwitchInst *NewSI = SwitchInst::Create(CV, PredDefault,
701 PredCases.size(), PTI);
Chris Lattner542f1492004-02-28 21:28:10 +0000702 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
703 NewSI->addCase(PredCases[i].first, PredCases[i].second);
Chris Lattner13b2f762005-01-01 16:02:12 +0000704
Eli Friedman080efb82008-12-16 20:54:32 +0000705 EraseTerminatorInstAndDCECond(PTI);
Chris Lattner13b2f762005-01-01 16:02:12 +0000706
Chris Lattner542f1492004-02-28 21:28:10 +0000707 // Okay, last check. If BB is still a successor of PSI, then we must
708 // have an infinite loop case. If so, add an infinitely looping block
709 // to handle the case to preserve the behavior of the code.
710 BasicBlock *InfLoopBlock = 0;
711 for (unsigned i = 0, e = NewSI->getNumSuccessors(); i != e; ++i)
712 if (NewSI->getSuccessor(i) == BB) {
713 if (InfLoopBlock == 0) {
Chris Lattner093a4382008-07-13 22:23:11 +0000714 // Insert it at the end of the function, because it's either code,
Chris Lattner542f1492004-02-28 21:28:10 +0000715 // or it won't matter if it's hot. :)
Owen Anderson1d0be152009-08-13 21:58:54 +0000716 InfLoopBlock = BasicBlock::Create(BB->getContext(),
717 "infloop", BB->getParent());
Gabor Greif051a9502008-04-06 20:25:17 +0000718 BranchInst::Create(InfLoopBlock, InfLoopBlock);
Chris Lattner542f1492004-02-28 21:28:10 +0000719 }
720 NewSI->setSuccessor(i, InfLoopBlock);
721 }
Misha Brukmanfd939082005-04-21 23:48:37 +0000722
Chris Lattner542f1492004-02-28 21:28:10 +0000723 Changed = true;
724 }
725 }
726 return Changed;
727}
728
Dale Johannesenc1f10402009-06-15 20:59:27 +0000729// isSafeToHoistInvoke - If we would need to insert a select that uses the
730// value of this invoke (comments in HoistThenElseCodeToIf explain why we
731// would need to do this), we can't hoist the invoke, as there is nowhere
732// to put the select in this case.
733static bool isSafeToHoistInvoke(BasicBlock *BB1, BasicBlock *BB2,
734 Instruction *I1, Instruction *I2) {
735 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
736 PHINode *PN;
737 for (BasicBlock::iterator BBI = SI->begin();
738 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
739 Value *BB1V = PN->getIncomingValueForBlock(BB1);
740 Value *BB2V = PN->getIncomingValueForBlock(BB2);
741 if (BB1V != BB2V && (BB1V==I1 || BB2V==I2)) {
742 return false;
743 }
744 }
745 }
746 return true;
747}
748
Chris Lattner6306d072005-08-03 17:59:45 +0000749/// HoistThenElseCodeToIf - Given a conditional branch that goes to BB1 and
Chris Lattner37dc9382004-11-30 00:29:14 +0000750/// BB2, hoist any common code in the two blocks up into the branch block. The
751/// caller of this function guarantees that BI's block dominates BB1 and BB2.
752static bool HoistThenElseCodeToIf(BranchInst *BI) {
753 // This does very trivial matching, with limited scanning, to find identical
754 // instructions in the two blocks. In particular, we don't want to get into
755 // O(M*N) situations here where M and N are the sizes of BB1 and BB2. As
756 // such, we currently just scan for obviously identical instructions in an
757 // identical order.
758 BasicBlock *BB1 = BI->getSuccessor(0); // The true destination.
759 BasicBlock *BB2 = BI->getSuccessor(1); // The false destination
760
Devang Patel65085cf2009-02-04 00:03:08 +0000761 BasicBlock::iterator BB1_Itr = BB1->begin();
762 BasicBlock::iterator BB2_Itr = BB2->begin();
763
764 Instruction *I1 = BB1_Itr++, *I2 = BB2_Itr++;
765 while (isa<DbgInfoIntrinsic>(I1))
766 I1 = BB1_Itr++;
767 while (isa<DbgInfoIntrinsic>(I2))
768 I2 = BB2_Itr++;
Dale Johannesenc1f10402009-06-15 20:59:27 +0000769 if (I1->getOpcode() != I2->getOpcode() || isa<PHINode>(I1) ||
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000770 !I1->isIdenticalToWhenDefined(I2) ||
Dale Johannesenc1f10402009-06-15 20:59:27 +0000771 (isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2)))
Chris Lattner37dc9382004-11-30 00:29:14 +0000772 return false;
773
774 // If we get here, we can hoist at least one instruction.
775 BasicBlock *BIParent = BI->getParent();
Chris Lattner37dc9382004-11-30 00:29:14 +0000776
777 do {
778 // If we are hoisting the terminator instruction, don't move one (making a
779 // broken BB), instead clone it, and remove BI.
780 if (isa<TerminatorInst>(I1))
781 goto HoistTerminator;
Misha Brukmanfd939082005-04-21 23:48:37 +0000782
Chris Lattner37dc9382004-11-30 00:29:14 +0000783 // For a normal instruction, we just move one to right before the branch,
784 // then replace all uses of the other with the first. Finally, we remove
785 // the now redundant second instruction.
786 BIParent->getInstList().splice(BI, BB1->getInstList(), I1);
787 if (!I2->use_empty())
788 I2->replaceAllUsesWith(I1);
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000789 I1->intersectOptionalDataWith(I2);
Chris Lattner37dc9382004-11-30 00:29:14 +0000790 BB2->getInstList().erase(I2);
Misha Brukmanfd939082005-04-21 23:48:37 +0000791
Devang Patel65085cf2009-02-04 00:03:08 +0000792 I1 = BB1_Itr++;
793 while (isa<DbgInfoIntrinsic>(I1))
794 I1 = BB1_Itr++;
795 I2 = BB2_Itr++;
796 while (isa<DbgInfoIntrinsic>(I2))
797 I2 = BB2_Itr++;
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000798 } while (I1->getOpcode() == I2->getOpcode() &&
799 I1->isIdenticalToWhenDefined(I2));
Chris Lattner37dc9382004-11-30 00:29:14 +0000800
801 return true;
802
803HoistTerminator:
Dale Johannesenc1f10402009-06-15 20:59:27 +0000804 // It may not be possible to hoist an invoke.
805 if (isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2))
806 return true;
807
Chris Lattner37dc9382004-11-30 00:29:14 +0000808 // Okay, it is safe to hoist the terminator.
Nick Lewycky67760642009-09-27 07:38:41 +0000809 Instruction *NT = I1->clone();
Chris Lattner37dc9382004-11-30 00:29:14 +0000810 BIParent->getInstList().insert(BI, NT);
Benjamin Kramerf0127052010-01-05 13:12:22 +0000811 if (!NT->getType()->isVoidTy()) {
Chris Lattner37dc9382004-11-30 00:29:14 +0000812 I1->replaceAllUsesWith(NT);
813 I2->replaceAllUsesWith(NT);
Chris Lattner86cc4232007-02-11 01:37:51 +0000814 NT->takeName(I1);
Chris Lattner37dc9382004-11-30 00:29:14 +0000815 }
816
817 // Hoisting one of the terminators from our successor is a great thing.
818 // Unfortunately, the successors of the if/else blocks may have PHI nodes in
819 // them. If they do, all PHI entries for BB1/BB2 must agree for all PHI
820 // nodes, so we insert select instruction to compute the final result.
821 std::map<std::pair<Value*,Value*>, SelectInst*> InsertedSelects;
822 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
823 PHINode *PN;
824 for (BasicBlock::iterator BBI = SI->begin();
Chris Lattner0f535c62004-11-30 07:47:34 +0000825 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
Chris Lattner37dc9382004-11-30 00:29:14 +0000826 Value *BB1V = PN->getIncomingValueForBlock(BB1);
827 Value *BB2V = PN->getIncomingValueForBlock(BB2);
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000828 if (BB1V == BB2V) continue;
829
830 // These values do not agree. Insert a select instruction before NT
831 // that determines the right value.
832 SelectInst *&SI = InsertedSelects[std::make_pair(BB1V, BB2V)];
833 if (SI == 0)
834 SI = SelectInst::Create(BI->getCondition(), BB1V, BB2V,
835 BB1V->getName()+"."+BB2V->getName(), NT);
836 // Make the PHI node use the select for all incoming values for BB1/BB2
837 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
838 if (PN->getIncomingBlock(i) == BB1 || PN->getIncomingBlock(i) == BB2)
839 PN->setIncomingValue(i, SI);
Chris Lattner37dc9382004-11-30 00:29:14 +0000840 }
841 }
842
843 // Update any PHI nodes in our new successors.
844 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI)
845 AddPredecessorToBlock(*SI, BIParent, BB1);
Misha Brukmanfd939082005-04-21 23:48:37 +0000846
Eli Friedman080efb82008-12-16 20:54:32 +0000847 EraseTerminatorInstAndDCECond(BI);
Chris Lattner37dc9382004-11-30 00:29:14 +0000848 return true;
849}
850
Evan Cheng4d09efd2008-06-07 08:52:29 +0000851/// SpeculativelyExecuteBB - Given a conditional branch that goes to BB1
852/// and an BB2 and the only successor of BB1 is BB2, hoist simple code
853/// (for now, restricted to a single instruction that's side effect free) from
854/// the BB1 into the branch block to speculatively execute it.
855static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *BB1) {
856 // Only speculatively execution a single instruction (not counting the
857 // terminator) for now.
Devang Patel06b1e672009-03-06 06:00:17 +0000858 Instruction *HInst = NULL;
859 Instruction *Term = BB1->getTerminator();
860 for (BasicBlock::iterator BBI = BB1->begin(), BBE = BB1->end();
861 BBI != BBE; ++BBI) {
862 Instruction *I = BBI;
863 // Skip debug info.
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000864 if (isa<DbgInfoIntrinsic>(I)) continue;
865 if (I == Term) break;
Devang Patel06b1e672009-03-06 06:00:17 +0000866
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000867 if (HInst)
Devang Patel06b1e672009-03-06 06:00:17 +0000868 return false;
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000869 HInst = I;
Devang Patel06b1e672009-03-06 06:00:17 +0000870 }
871 if (!HInst)
872 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000873
Evan Cheng797d9512008-06-11 19:18:20 +0000874 // Be conservative for now. FP select instruction can often be expensive.
875 Value *BrCond = BI->getCondition();
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000876 if (isa<FCmpInst>(BrCond))
Evan Cheng797d9512008-06-11 19:18:20 +0000877 return false;
878
Evan Cheng4d09efd2008-06-07 08:52:29 +0000879 // If BB1 is actually on the false edge of the conditional branch, remember
880 // to swap the select operands later.
881 bool Invert = false;
882 if (BB1 != BI->getSuccessor(0)) {
883 assert(BB1 == BI->getSuccessor(1) && "No edge from 'if' block?");
884 Invert = true;
885 }
886
887 // Turn
888 // BB:
889 // %t1 = icmp
890 // br i1 %t1, label %BB1, label %BB2
891 // BB1:
892 // %t3 = add %t2, c
893 // br label BB2
894 // BB2:
895 // =>
896 // BB:
897 // %t1 = icmp
898 // %t4 = add %t2, c
899 // %t3 = select i1 %t1, %t2, %t3
Devang Patel06b1e672009-03-06 06:00:17 +0000900 switch (HInst->getOpcode()) {
Evan Cheng4d09efd2008-06-07 08:52:29 +0000901 default: return false; // Not safe / profitable to hoist.
902 case Instruction::Add:
903 case Instruction::Sub:
Dan Gohmanae3a0be2009-06-04 22:49:04 +0000904 // Not worth doing for vector ops.
Duncan Sands1df98592010-02-16 11:11:14 +0000905 if (HInst->getType()->isVectorTy())
Chris Lattner9dd3b612009-01-18 23:22:07 +0000906 return false;
907 break;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000908 case Instruction::And:
909 case Instruction::Or:
910 case Instruction::Xor:
911 case Instruction::Shl:
912 case Instruction::LShr:
913 case Instruction::AShr:
Chris Lattner9dd3b612009-01-18 23:22:07 +0000914 // Don't mess with vector operations.
Duncan Sands1df98592010-02-16 11:11:14 +0000915 if (HInst->getType()->isVectorTy())
Evan Chenge5334ea2008-06-25 07:50:12 +0000916 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000917 break; // These are all cheap and non-trapping instructions.
918 }
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000919
920 // If the instruction is obviously dead, don't try to predicate it.
Devang Patel06b1e672009-03-06 06:00:17 +0000921 if (HInst->use_empty()) {
922 HInst->eraseFromParent();
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000923 return true;
924 }
Evan Cheng4d09efd2008-06-07 08:52:29 +0000925
926 // Can we speculatively execute the instruction? And what is the value
927 // if the condition is false? Consider the phi uses, if the incoming value
928 // from the "if" block are all the same V, then V is the value of the
929 // select if the condition is false.
930 BasicBlock *BIParent = BI->getParent();
931 SmallVector<PHINode*, 4> PHIUses;
932 Value *FalseV = NULL;
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000933
934 BasicBlock *BB2 = BB1->getTerminator()->getSuccessor(0);
Devang Patel06b1e672009-03-06 06:00:17 +0000935 for (Value::use_iterator UI = HInst->use_begin(), E = HInst->use_end();
Evan Cheng4d09efd2008-06-07 08:52:29 +0000936 UI != E; ++UI) {
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000937 // Ignore any user that is not a PHI node in BB2. These can only occur in
938 // unreachable blocks, because they would not be dominated by the instr.
Gabor Greif20361b92010-07-22 11:43:44 +0000939 PHINode *PN = dyn_cast<PHINode>(*UI);
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000940 if (!PN || PN->getParent() != BB2)
941 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000942 PHIUses.push_back(PN);
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000943
Evan Cheng4d09efd2008-06-07 08:52:29 +0000944 Value *PHIV = PN->getIncomingValueForBlock(BIParent);
945 if (!FalseV)
946 FalseV = PHIV;
947 else if (FalseV != PHIV)
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000948 return false; // Inconsistent value when condition is false.
Evan Cheng4d09efd2008-06-07 08:52:29 +0000949 }
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000950
951 assert(FalseV && "Must have at least one user, and it must be a PHI");
Evan Cheng4d09efd2008-06-07 08:52:29 +0000952
Evan Cheng502a4f52008-06-12 21:15:59 +0000953 // Do not hoist the instruction if any of its operands are defined but not
954 // used in this BB. The transformation will prevent the operand from
955 // being sunk into the use block.
Devang Patel06b1e672009-03-06 06:00:17 +0000956 for (User::op_iterator i = HInst->op_begin(), e = HInst->op_end();
957 i != e; ++i) {
Evan Cheng502a4f52008-06-12 21:15:59 +0000958 Instruction *OpI = dyn_cast<Instruction>(*i);
959 if (OpI && OpI->getParent() == BIParent &&
960 !OpI->isUsedInBasicBlock(BIParent))
961 return false;
962 }
963
Devang Patel3d0a9a32008-09-18 22:50:42 +0000964 // If we get here, we can hoist the instruction. Try to place it
Dale Johannesen990afed2009-03-13 01:05:24 +0000965 // before the icmp instruction preceding the conditional branch.
Devang Patel3d0a9a32008-09-18 22:50:42 +0000966 BasicBlock::iterator InsertPos = BI;
Dale Johannesen990afed2009-03-13 01:05:24 +0000967 if (InsertPos != BIParent->begin())
968 --InsertPos;
969 // Skip debug info between condition and branch.
970 while (InsertPos != BIParent->begin() && isa<DbgInfoIntrinsic>(InsertPos))
Devang Patel3d0a9a32008-09-18 22:50:42 +0000971 --InsertPos;
Devang Patel20da1f02008-10-03 18:57:37 +0000972 if (InsertPos == BrCond && !isa<PHINode>(BrCond)) {
Devang Patel3d0a9a32008-09-18 22:50:42 +0000973 SmallPtrSet<Instruction *, 4> BB1Insns;
974 for(BasicBlock::iterator BB1I = BB1->begin(), BB1E = BB1->end();
975 BB1I != BB1E; ++BB1I)
976 BB1Insns.insert(BB1I);
977 for(Value::use_iterator UI = BrCond->use_begin(), UE = BrCond->use_end();
978 UI != UE; ++UI) {
979 Instruction *Use = cast<Instruction>(*UI);
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000980 if (!BB1Insns.count(Use)) continue;
981
982 // If BrCond uses the instruction that place it just before
983 // branch instruction.
984 InsertPos = BI;
985 break;
Devang Patel3d0a9a32008-09-18 22:50:42 +0000986 }
987 } else
988 InsertPos = BI;
Devang Patel06b1e672009-03-06 06:00:17 +0000989 BIParent->getInstList().splice(InsertPos, BB1->getInstList(), HInst);
Evan Cheng4d09efd2008-06-07 08:52:29 +0000990
991 // Create a select whose true value is the speculatively executed value and
992 // false value is the previously determined FalseV.
993 SelectInst *SI;
994 if (Invert)
Devang Patel06b1e672009-03-06 06:00:17 +0000995 SI = SelectInst::Create(BrCond, FalseV, HInst,
996 FalseV->getName() + "." + HInst->getName(), BI);
Evan Cheng4d09efd2008-06-07 08:52:29 +0000997 else
Devang Patel06b1e672009-03-06 06:00:17 +0000998 SI = SelectInst::Create(BrCond, HInst, FalseV,
999 HInst->getName() + "." + FalseV->getName(), BI);
Evan Cheng4d09efd2008-06-07 08:52:29 +00001000
1001 // Make the PHI node use the select for all incoming values for "then" and
1002 // "if" blocks.
1003 for (unsigned i = 0, e = PHIUses.size(); i != e; ++i) {
1004 PHINode *PN = PHIUses[i];
1005 for (unsigned j = 0, ee = PN->getNumIncomingValues(); j != ee; ++j)
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001006 if (PN->getIncomingBlock(j) == BB1 || PN->getIncomingBlock(j) == BIParent)
Evan Cheng4d09efd2008-06-07 08:52:29 +00001007 PN->setIncomingValue(j, SI);
1008 }
1009
Evan Cheng502a4f52008-06-12 21:15:59 +00001010 ++NumSpeculations;
Evan Cheng4d09efd2008-06-07 08:52:29 +00001011 return true;
1012}
1013
Chris Lattner2e42e362005-09-20 00:43:16 +00001014/// BlockIsSimpleEnoughToThreadThrough - Return true if we can thread a branch
1015/// across this block.
1016static bool BlockIsSimpleEnoughToThreadThrough(BasicBlock *BB) {
1017 BranchInst *BI = cast<BranchInst>(BB->getTerminator());
Chris Lattnere9487f02005-09-20 01:48:40 +00001018 unsigned Size = 0;
1019
Devang Patel9200c892009-03-10 18:00:05 +00001020 for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
Dale Johannesen8483e542009-03-12 23:18:09 +00001021 if (isa<DbgInfoIntrinsic>(BBI))
1022 continue;
Chris Lattnere9487f02005-09-20 01:48:40 +00001023 if (Size > 10) return false; // Don't clone large BB's.
Dale Johannesen8483e542009-03-12 23:18:09 +00001024 ++Size;
Chris Lattner2e42e362005-09-20 00:43:16 +00001025
Dale Johannesen8483e542009-03-12 23:18:09 +00001026 // We can only support instructions that do not define values that are
Chris Lattnere9487f02005-09-20 01:48:40 +00001027 // live outside of the current basic block.
1028 for (Value::use_iterator UI = BBI->use_begin(), E = BBI->use_end();
1029 UI != E; ++UI) {
1030 Instruction *U = cast<Instruction>(*UI);
1031 if (U->getParent() != BB || isa<PHINode>(U)) return false;
1032 }
Chris Lattner2e42e362005-09-20 00:43:16 +00001033
1034 // Looks ok, continue checking.
1035 }
Chris Lattnere9487f02005-09-20 01:48:40 +00001036
Chris Lattner2e42e362005-09-20 00:43:16 +00001037 return true;
1038}
1039
Chris Lattnereaba3a12005-09-19 23:49:37 +00001040/// FoldCondBranchOnPHI - If we have a conditional branch on a PHI node value
1041/// that is defined in the same block as the branch and if any PHI entries are
1042/// constants, thread edges corresponding to that entry to be branches to their
1043/// ultimate destination.
1044static bool FoldCondBranchOnPHI(BranchInst *BI) {
1045 BasicBlock *BB = BI->getParent();
1046 PHINode *PN = dyn_cast<PHINode>(BI->getCondition());
Chris Lattner9c88d982005-09-19 23:57:04 +00001047 // NOTE: we currently cannot transform this case if the PHI node is used
1048 // outside of the block.
Chris Lattner2e42e362005-09-20 00:43:16 +00001049 if (!PN || PN->getParent() != BB || !PN->hasOneUse())
1050 return false;
Chris Lattnereaba3a12005-09-19 23:49:37 +00001051
1052 // Degenerate case of a single entry PHI.
1053 if (PN->getNumIncomingValues() == 1) {
Chris Lattner29874e02008-12-03 19:44:02 +00001054 FoldSingleEntryPHINodes(PN->getParent());
Chris Lattnereaba3a12005-09-19 23:49:37 +00001055 return true;
1056 }
1057
1058 // Now we know that this block has multiple preds and two succs.
Chris Lattner2e42e362005-09-20 00:43:16 +00001059 if (!BlockIsSimpleEnoughToThreadThrough(BB)) return false;
Chris Lattnereaba3a12005-09-19 23:49:37 +00001060
1061 // Okay, this is a simple enough basic block. See if any phi values are
1062 // constants.
Zhou Sheng6b6b6ef2007-01-11 12:24:14 +00001063 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001064 ConstantInt *CB = dyn_cast<ConstantInt>(PN->getIncomingValue(i));
1065 if (CB == 0 || !CB->getType()->isIntegerTy(1)) continue;
1066
1067 // Okay, we now know that all edges from PredBB should be revectored to
1068 // branch to RealDest.
1069 BasicBlock *PredBB = PN->getIncomingBlock(i);
1070 BasicBlock *RealDest = BI->getSuccessor(!CB->getZExtValue());
1071
1072 if (RealDest == BB) continue; // Skip self loops.
1073
1074 // The dest block might have PHI nodes, other predecessors and other
1075 // difficult cases. Instead of being smart about this, just insert a new
1076 // block that jumps to the destination block, effectively splitting
1077 // the edge we are about to create.
1078 BasicBlock *EdgeBB = BasicBlock::Create(BB->getContext(),
1079 RealDest->getName()+".critedge",
1080 RealDest->getParent(), RealDest);
1081 BranchInst::Create(RealDest, EdgeBB);
1082 PHINode *PN;
1083 for (BasicBlock::iterator BBI = RealDest->begin();
1084 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
1085 Value *V = PN->getIncomingValueForBlock(BB);
1086 PN->addIncoming(V, EdgeBB);
Chris Lattnereaba3a12005-09-19 23:49:37 +00001087 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001088
1089 // BB may have instructions that are being threaded over. Clone these
1090 // instructions into EdgeBB. We know that there will be no uses of the
1091 // cloned instructions outside of EdgeBB.
1092 BasicBlock::iterator InsertPt = EdgeBB->begin();
1093 DenseMap<Value*, Value*> TranslateMap; // Track translated values.
1094 for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
1095 if (PHINode *PN = dyn_cast<PHINode>(BBI)) {
1096 TranslateMap[PN] = PN->getIncomingValueForBlock(PredBB);
1097 continue;
1098 }
1099 // Clone the instruction.
1100 Instruction *N = BBI->clone();
1101 if (BBI->hasName()) N->setName(BBI->getName()+".c");
1102
1103 // Update operands due to translation.
1104 for (User::op_iterator i = N->op_begin(), e = N->op_end();
1105 i != e; ++i) {
1106 DenseMap<Value*, Value*>::iterator PI = TranslateMap.find(*i);
1107 if (PI != TranslateMap.end())
1108 *i = PI->second;
1109 }
1110
1111 // Check for trivial simplification.
1112 if (Constant *C = ConstantFoldInstruction(N)) {
1113 TranslateMap[BBI] = C;
1114 delete N; // Constant folded away, don't need actual inst
1115 } else {
1116 // Insert the new instruction into its new home.
1117 EdgeBB->getInstList().insert(InsertPt, N);
1118 if (!BBI->use_empty())
1119 TranslateMap[BBI] = N;
1120 }
1121 }
1122
1123 // Loop over all of the edges from PredBB to BB, changing them to branch
1124 // to EdgeBB instead.
1125 TerminatorInst *PredBBTI = PredBB->getTerminator();
1126 for (unsigned i = 0, e = PredBBTI->getNumSuccessors(); i != e; ++i)
1127 if (PredBBTI->getSuccessor(i) == BB) {
1128 BB->removePredecessor(PredBB);
1129 PredBBTI->setSuccessor(i, EdgeBB);
1130 }
1131
1132 // Recurse, simplifying any other constants.
1133 return FoldCondBranchOnPHI(BI) | true;
Zhou Sheng6b6b6ef2007-01-11 12:24:14 +00001134 }
Chris Lattnereaba3a12005-09-19 23:49:37 +00001135
1136 return false;
1137}
1138
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001139/// FoldTwoEntryPHINode - Given a BB that starts with the specified two-entry
1140/// PHI node, see if we can eliminate it.
1141static bool FoldTwoEntryPHINode(PHINode *PN) {
1142 // Ok, this is a two entry PHI node. Check to see if this is a simple "if
1143 // statement", which has a very simple dominance structure. Basically, we
1144 // are trying to find the condition that is being branched on, which
1145 // subsequently causes this merge to happen. We really want control
1146 // dependence information for this check, but simplifycfg can't keep it up
1147 // to date, and this catches most of the cases we care about anyway.
1148 //
1149 BasicBlock *BB = PN->getParent();
1150 BasicBlock *IfTrue, *IfFalse;
1151 Value *IfCond = GetIfCondition(BB, IfTrue, IfFalse);
1152 if (!IfCond) return false;
1153
Chris Lattner822a8792006-11-18 19:19:36 +00001154 // Okay, we found that we can merge this two-entry phi node into a select.
1155 // Doing so would require us to fold *all* two entry phi nodes in this block.
1156 // At some point this becomes non-profitable (particularly if the target
1157 // doesn't support cmov's). Only do this transformation if there are two or
1158 // fewer PHI nodes in this block.
1159 unsigned NumPhis = 0;
1160 for (BasicBlock::iterator I = BB->begin(); isa<PHINode>(I); ++NumPhis, ++I)
1161 if (NumPhis > 2)
1162 return false;
1163
David Greene89d6fd32010-01-05 01:26:52 +00001164 DEBUG(dbgs() << "FOUND IF CONDITION! " << *IfCond << " T: "
Daniel Dunbarce63ffb2009-07-25 00:23:56 +00001165 << IfTrue->getName() << " F: " << IfFalse->getName() << "\n");
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001166
1167 // Loop over the PHI's seeing if we can promote them all to select
1168 // instructions. While we are at it, keep track of the instructions
1169 // that need to be moved to the dominating block.
1170 std::set<Instruction*> AggressiveInsts;
1171
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001172 BasicBlock::iterator AfterPHIIt = BB->begin();
1173 while (isa<PHINode>(AfterPHIIt)) {
1174 PHINode *PN = cast<PHINode>(AfterPHIIt++);
1175 if (PN->getIncomingValue(0) == PN->getIncomingValue(1)) {
1176 if (PN->getIncomingValue(0) != PN)
1177 PN->replaceAllUsesWith(PN->getIncomingValue(0));
1178 else
Owen Anderson9e9a0d52009-07-30 23:03:37 +00001179 PN->replaceAllUsesWith(UndefValue::get(PN->getType()));
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001180 } else if (!DominatesMergePoint(PN->getIncomingValue(0), BB,
1181 &AggressiveInsts) ||
1182 !DominatesMergePoint(PN->getIncomingValue(1), BB,
1183 &AggressiveInsts)) {
Chris Lattner055dc102005-09-23 07:23:18 +00001184 return false;
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001185 }
1186 }
1187
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001188 // If we all PHI nodes are promotable, check to make sure that all
1189 // instructions in the predecessor blocks can be promoted as well. If
1190 // not, we won't be able to get rid of the control flow, so it's not
1191 // worth promoting to select instructions.
1192 BasicBlock *DomBlock = 0, *IfBlock1 = 0, *IfBlock2 = 0;
1193 PN = cast<PHINode>(BB->begin());
1194 BasicBlock *Pred = PN->getIncomingBlock(0);
1195 if (cast<BranchInst>(Pred->getTerminator())->isUnconditional()) {
1196 IfBlock1 = Pred;
1197 DomBlock = *pred_begin(Pred);
1198 for (BasicBlock::iterator I = Pred->begin();
1199 !isa<TerminatorInst>(I); ++I)
Devang Patel383d7ed2009-02-03 22:12:02 +00001200 if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001201 // This is not an aggressive instruction that we can promote.
1202 // Because of this, we won't be able to get rid of the control
1203 // flow, so the xform is not worth it.
1204 return false;
1205 }
1206 }
1207
1208 Pred = PN->getIncomingBlock(1);
1209 if (cast<BranchInst>(Pred->getTerminator())->isUnconditional()) {
1210 IfBlock2 = Pred;
1211 DomBlock = *pred_begin(Pred);
1212 for (BasicBlock::iterator I = Pred->begin();
1213 !isa<TerminatorInst>(I); ++I)
Devang Patel383d7ed2009-02-03 22:12:02 +00001214 if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001215 // This is not an aggressive instruction that we can promote.
1216 // Because of this, we won't be able to get rid of the control
1217 // flow, so the xform is not worth it.
1218 return false;
1219 }
1220 }
1221
1222 // If we can still promote the PHI nodes after this gauntlet of tests,
1223 // do all of the PHI's now.
1224
1225 // Move all 'aggressive' instructions, which are defined in the
1226 // conditional parts of the if's up to the dominating block.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001227 if (IfBlock1)
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001228 DomBlock->getInstList().splice(DomBlock->getTerminator(),
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001229 IfBlock1->getInstList(), IfBlock1->begin(),
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001230 IfBlock1->getTerminator());
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001231 if (IfBlock2)
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001232 DomBlock->getInstList().splice(DomBlock->getTerminator(),
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001233 IfBlock2->getInstList(), IfBlock2->begin(),
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001234 IfBlock2->getTerminator());
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001235
1236 while (PHINode *PN = dyn_cast<PHINode>(BB->begin())) {
1237 // Change the PHI node into a select instruction.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001238 Value *TrueVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfFalse);
1239 Value *FalseVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfTrue);
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001240
Gabor Greif051a9502008-04-06 20:25:17 +00001241 Value *NV = SelectInst::Create(IfCond, TrueVal, FalseVal, "", AfterPHIIt);
Chris Lattner86cc4232007-02-11 01:37:51 +00001242 PN->replaceAllUsesWith(NV);
1243 NV->takeName(PN);
1244
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001245 BB->getInstList().erase(PN);
1246 }
1247 return true;
1248}
Chris Lattnereaba3a12005-09-19 23:49:37 +00001249
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001250/// SimplifyCondBranchToTwoReturns - If we found a conditional branch that goes
1251/// to two returning blocks, try to merge them together into one return,
1252/// introducing a select if the return values disagree.
1253static bool SimplifyCondBranchToTwoReturns(BranchInst *BI) {
1254 assert(BI->isConditional() && "Must be a conditional branch");
1255 BasicBlock *TrueSucc = BI->getSuccessor(0);
1256 BasicBlock *FalseSucc = BI->getSuccessor(1);
1257 ReturnInst *TrueRet = cast<ReturnInst>(TrueSucc->getTerminator());
1258 ReturnInst *FalseRet = cast<ReturnInst>(FalseSucc->getTerminator());
1259
1260 // Check to ensure both blocks are empty (just a return) or optionally empty
1261 // with PHI nodes. If there are other instructions, merging would cause extra
1262 // computation on one path or the other.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001263 if (!TrueSucc->getFirstNonPHIOrDbg()->isTerminator())
Devang Patel2cc86a12009-02-05 00:30:42 +00001264 return false;
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001265 if (!FalseSucc->getFirstNonPHIOrDbg()->isTerminator())
Devang Patel2cc86a12009-02-05 00:30:42 +00001266 return false;
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001267
1268 // Okay, we found a branch that is going to two return nodes. If
1269 // there is no return value for this function, just change the
1270 // branch into a return.
1271 if (FalseRet->getNumOperands() == 0) {
1272 TrueSucc->removePredecessor(BI->getParent());
1273 FalseSucc->removePredecessor(BI->getParent());
Owen Anderson1d0be152009-08-13 21:58:54 +00001274 ReturnInst::Create(BI->getContext(), 0, BI);
Eli Friedman080efb82008-12-16 20:54:32 +00001275 EraseTerminatorInstAndDCECond(BI);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001276 return true;
1277 }
1278
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001279 // Otherwise, figure out what the true and false return values are
1280 // so we can insert a new select instruction.
1281 Value *TrueValue = TrueRet->getReturnValue();
1282 Value *FalseValue = FalseRet->getReturnValue();
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001283
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001284 // Unwrap any PHI nodes in the return blocks.
1285 if (PHINode *TVPN = dyn_cast_or_null<PHINode>(TrueValue))
1286 if (TVPN->getParent() == TrueSucc)
1287 TrueValue = TVPN->getIncomingValueForBlock(BI->getParent());
1288 if (PHINode *FVPN = dyn_cast_or_null<PHINode>(FalseValue))
1289 if (FVPN->getParent() == FalseSucc)
1290 FalseValue = FVPN->getIncomingValueForBlock(BI->getParent());
1291
1292 // In order for this transformation to be safe, we must be able to
1293 // unconditionally execute both operands to the return. This is
1294 // normally the case, but we could have a potentially-trapping
1295 // constant expression that prevents this transformation from being
1296 // safe.
1297 if (ConstantExpr *TCV = dyn_cast_or_null<ConstantExpr>(TrueValue))
1298 if (TCV->canTrap())
1299 return false;
1300 if (ConstantExpr *FCV = dyn_cast_or_null<ConstantExpr>(FalseValue))
1301 if (FCV->canTrap())
1302 return false;
1303
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001304 // Okay, we collected all the mapped values and checked them for sanity, and
1305 // defined to really do this transformation. First, update the CFG.
1306 TrueSucc->removePredecessor(BI->getParent());
1307 FalseSucc->removePredecessor(BI->getParent());
1308
1309 // Insert select instructions where needed.
1310 Value *BrCond = BI->getCondition();
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001311 if (TrueValue) {
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001312 // Insert a select if the results differ.
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001313 if (TrueValue == FalseValue || isa<UndefValue>(FalseValue)) {
1314 } else if (isa<UndefValue>(TrueValue)) {
1315 TrueValue = FalseValue;
1316 } else {
1317 TrueValue = SelectInst::Create(BrCond, TrueValue,
1318 FalseValue, "retval", BI);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001319 }
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001320 }
1321
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001322 Value *RI = !TrueValue ?
Owen Anderson1d0be152009-08-13 21:58:54 +00001323 ReturnInst::Create(BI->getContext(), BI) :
1324 ReturnInst::Create(BI->getContext(), TrueValue, BI);
Daniel Dunbare317bcc2009-08-23 10:29:55 +00001325 (void) RI;
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001326
David Greene89d6fd32010-01-05 01:26:52 +00001327 DEBUG(dbgs() << "\nCHANGING BRANCH TO TWO RETURNS INTO SELECT:"
Chris Lattnerbdff5482009-08-23 04:37:46 +00001328 << "\n " << *BI << "NewRet = " << *RI
1329 << "TRUEBLOCK: " << *TrueSucc << "FALSEBLOCK: "<< *FalseSucc);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001330
Eli Friedman080efb82008-12-16 20:54:32 +00001331 EraseTerminatorInstAndDCECond(BI);
1332
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001333 return true;
1334}
1335
Chris Lattner1347e872008-07-13 21:12:01 +00001336/// FoldBranchToCommonDest - If this basic block is ONLY a setcc and a branch,
1337/// and if a predecessor branches to us and one of our successors, fold the
1338/// setcc into the predecessor and use logical operations to pick the right
1339/// destination.
Dan Gohman4b35f832009-06-27 21:30:38 +00001340bool llvm::FoldBranchToCommonDest(BranchInst *BI) {
Chris Lattner093a4382008-07-13 22:23:11 +00001341 BasicBlock *BB = BI->getParent();
Chris Lattner1347e872008-07-13 21:12:01 +00001342 Instruction *Cond = dyn_cast<Instruction>(BI->getCondition());
Owen Andersone84178a2010-07-14 19:52:16 +00001343 if (Cond == 0 || (!isa<CmpInst>(Cond) && !isa<BinaryOperator>(Cond)) ||
1344 Cond->getParent() != BB || !Cond->hasOneUse())
1345 return false;
Chris Lattner093a4382008-07-13 22:23:11 +00001346
Chris Lattner1347e872008-07-13 21:12:01 +00001347 // Only allow this if the condition is a simple instruction that can be
1348 // executed unconditionally. It must be in the same block as the branch, and
1349 // must be at the front of the block.
Devang Pateld0a203d2009-02-04 21:39:48 +00001350 BasicBlock::iterator FrontIt = BB->front();
1351 // Ignore dbg intrinsics.
1352 while(isa<DbgInfoIntrinsic>(FrontIt))
1353 ++FrontIt;
Owen Andersone84178a2010-07-14 19:52:16 +00001354
1355 // Allow a single instruction to be hoisted in addition to the compare
1356 // that feeds the branch. We later ensure that any values that _it_ uses
1357 // were also live in the predecessor, so that we don't unnecessarily create
1358 // register pressure or inhibit out-of-order execution.
1359 Instruction *BonusInst = 0;
1360 if (&*FrontIt != Cond &&
Owen Anderson2722dfa2010-07-15 16:38:22 +00001361 FrontIt->hasOneUse() && *FrontIt->use_begin() == Cond &&
1362 FrontIt->isSafeToSpeculativelyExecute()) {
Owen Andersone84178a2010-07-14 19:52:16 +00001363 BonusInst = &*FrontIt;
1364 ++FrontIt;
Devang Pateld0a203d2009-02-04 21:39:48 +00001365 }
Chris Lattner6ff645b2009-01-19 23:03:13 +00001366
Owen Andersone84178a2010-07-14 19:52:16 +00001367 // Only a single bonus inst is allowed.
1368 if (&*FrontIt != Cond)
1369 return false;
1370
Chris Lattner1347e872008-07-13 21:12:01 +00001371 // Make sure the instruction after the condition is the cond branch.
1372 BasicBlock::iterator CondIt = Cond; ++CondIt;
Devang Pateld0a203d2009-02-04 21:39:48 +00001373 // Ingore dbg intrinsics.
1374 while(isa<DbgInfoIntrinsic>(CondIt))
1375 ++CondIt;
1376 if (&*CondIt != BI) {
1377 assert (!isa<DbgInfoIntrinsic>(CondIt) && "Hey do not forget debug info!");
Chris Lattner1347e872008-07-13 21:12:01 +00001378 return false;
Devang Pateld0a203d2009-02-04 21:39:48 +00001379 }
Chris Lattner6ff645b2009-01-19 23:03:13 +00001380
1381 // Cond is known to be a compare or binary operator. Check to make sure that
1382 // neither operand is a potentially-trapping constant expression.
1383 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(0)))
1384 if (CE->canTrap())
1385 return false;
1386 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(1)))
1387 if (CE->canTrap())
1388 return false;
1389
Chris Lattner1347e872008-07-13 21:12:01 +00001390
1391 // Finally, don't infinitely unroll conditional loops.
1392 BasicBlock *TrueDest = BI->getSuccessor(0);
1393 BasicBlock *FalseDest = BI->getSuccessor(1);
1394 if (TrueDest == BB || FalseDest == BB)
1395 return false;
1396
1397 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
1398 BasicBlock *PredBlock = *PI;
1399 BranchInst *PBI = dyn_cast<BranchInst>(PredBlock->getTerminator());
Chris Lattner6ff645b2009-01-19 23:03:13 +00001400
Chris Lattner093a4382008-07-13 22:23:11 +00001401 // Check that we have two conditional branches. If there is a PHI node in
1402 // the common successor, verify that the same value flows in from both
1403 // blocks.
Chris Lattner1347e872008-07-13 21:12:01 +00001404 if (PBI == 0 || PBI->isUnconditional() ||
1405 !SafeToMergeTerminators(BI, PBI))
1406 continue;
1407
Owen Andersone84178a2010-07-14 19:52:16 +00001408 // Ensure that any values used in the bonus instruction are also used
1409 // by the terminator of the predecessor. This means that those values
1410 // must already have been resolved, so we won't be inhibiting the
1411 // out-of-order core by speculating them earlier.
1412 if (BonusInst) {
1413 // Collect the values used by the bonus inst
1414 SmallPtrSet<Value*, 4> UsedValues;
1415 for (Instruction::op_iterator OI = BonusInst->op_begin(),
1416 OE = BonusInst->op_end(); OI != OE; ++OI) {
1417 Value* V = *OI;
1418 if (!isa<Constant>(V))
1419 UsedValues.insert(V);
1420 }
1421
1422 SmallVector<std::pair<Value*, unsigned>, 4> Worklist;
1423 Worklist.push_back(std::make_pair(PBI->getOperand(0), 0));
1424
1425 // Walk up to four levels back up the use-def chain of the predecessor's
1426 // terminator to see if all those values were used. The choice of four
1427 // levels is arbitrary, to provide a compile-time-cost bound.
1428 while (!Worklist.empty()) {
1429 std::pair<Value*, unsigned> Pair = Worklist.back();
1430 Worklist.pop_back();
1431
1432 if (Pair.second >= 4) continue;
1433 UsedValues.erase(Pair.first);
1434 if (UsedValues.empty()) break;
1435
1436 if (Instruction* I = dyn_cast<Instruction>(Pair.first)) {
1437 for (Instruction::op_iterator OI = I->op_begin(), OE = I->op_end();
1438 OI != OE; ++OI)
1439 Worklist.push_back(std::make_pair(OI->get(), Pair.second+1));
1440 }
1441 }
1442
1443 if (!UsedValues.empty()) return false;
1444 }
1445
Chris Lattner36989092008-07-13 21:20:19 +00001446 Instruction::BinaryOps Opc;
1447 bool InvertPredCond = false;
1448
1449 if (PBI->getSuccessor(0) == TrueDest)
1450 Opc = Instruction::Or;
1451 else if (PBI->getSuccessor(1) == FalseDest)
1452 Opc = Instruction::And;
1453 else if (PBI->getSuccessor(0) == FalseDest)
1454 Opc = Instruction::And, InvertPredCond = true;
1455 else if (PBI->getSuccessor(1) == TrueDest)
1456 Opc = Instruction::Or, InvertPredCond = true;
1457 else
1458 continue;
1459
David Greene89d6fd32010-01-05 01:26:52 +00001460 DEBUG(dbgs() << "FOLDING BRANCH TO COMMON DEST:\n" << *PBI << *BB);
Chris Lattner6ff645b2009-01-19 23:03:13 +00001461
Chris Lattner36989092008-07-13 21:20:19 +00001462 // If we need to invert the condition in the pred block to match, do so now.
1463 if (InvertPredCond) {
Chris Lattner1347e872008-07-13 21:12:01 +00001464 Value *NewCond =
Dan Gohman4ae51262009-08-12 16:23:25 +00001465 BinaryOperator::CreateNot(PBI->getCondition(),
Chris Lattner36989092008-07-13 21:20:19 +00001466 PBI->getCondition()->getName()+".not", PBI);
Chris Lattner1347e872008-07-13 21:12:01 +00001467 PBI->setCondition(NewCond);
1468 BasicBlock *OldTrue = PBI->getSuccessor(0);
1469 BasicBlock *OldFalse = PBI->getSuccessor(1);
1470 PBI->setSuccessor(0, OldFalse);
1471 PBI->setSuccessor(1, OldTrue);
1472 }
Chris Lattner70087f32008-07-13 21:15:11 +00001473
Owen Andersone84178a2010-07-14 19:52:16 +00001474 // If we have a bonus inst, clone it into the predecessor block.
1475 Instruction *NewBonus = 0;
1476 if (BonusInst) {
1477 NewBonus = BonusInst->clone();
1478 PredBlock->getInstList().insert(PBI, NewBonus);
1479 NewBonus->takeName(BonusInst);
1480 BonusInst->setName(BonusInst->getName()+".old");
1481 }
1482
Chris Lattner36989092008-07-13 21:20:19 +00001483 // Clone Cond into the predecessor basic block, and or/and the
1484 // two conditions together.
Nick Lewycky67760642009-09-27 07:38:41 +00001485 Instruction *New = Cond->clone();
Owen Andersone84178a2010-07-14 19:52:16 +00001486 if (BonusInst) New->replaceUsesOfWith(BonusInst, NewBonus);
Chris Lattner36989092008-07-13 21:20:19 +00001487 PredBlock->getInstList().insert(PBI, New);
1488 New->takeName(Cond);
1489 Cond->setName(New->getName()+".old");
Chris Lattner70087f32008-07-13 21:15:11 +00001490
Chris Lattner36989092008-07-13 21:20:19 +00001491 Value *NewCond = BinaryOperator::Create(Opc, PBI->getCondition(),
1492 New, "or.cond", PBI);
1493 PBI->setCondition(NewCond);
1494 if (PBI->getSuccessor(0) == BB) {
1495 AddPredecessorToBlock(TrueDest, PredBlock, BB);
1496 PBI->setSuccessor(0, TrueDest);
Chris Lattner1347e872008-07-13 21:12:01 +00001497 }
Chris Lattner36989092008-07-13 21:20:19 +00001498 if (PBI->getSuccessor(1) == BB) {
1499 AddPredecessorToBlock(FalseDest, PredBlock, BB);
1500 PBI->setSuccessor(1, FalseDest);
1501 }
1502 return true;
Chris Lattner1347e872008-07-13 21:12:01 +00001503 }
1504 return false;
1505}
1506
Chris Lattner867661a2008-07-13 21:53:26 +00001507/// SimplifyCondBranchToCondBranch - If we have a conditional branch as a
1508/// predecessor of another block, this function tries to simplify it. We know
1509/// that PBI and BI are both conditional branches, and BI is in one of the
1510/// successor blocks of PBI - PBI branches to BI.
1511static bool SimplifyCondBranchToCondBranch(BranchInst *PBI, BranchInst *BI) {
1512 assert(PBI->isConditional() && BI->isConditional());
1513 BasicBlock *BB = BI->getParent();
Dan Gohman4ae51262009-08-12 16:23:25 +00001514
Chris Lattner867661a2008-07-13 21:53:26 +00001515 // If this block ends with a branch instruction, and if there is a
1516 // predecessor that ends on a branch of the same condition, make
1517 // this conditional branch redundant.
1518 if (PBI->getCondition() == BI->getCondition() &&
1519 PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
1520 // Okay, the outcome of this conditional branch is statically
1521 // knowable. If this block had a single pred, handle specially.
1522 if (BB->getSinglePredecessor()) {
1523 // Turn this into a branch on constant.
1524 bool CondIsTrue = PBI->getSuccessor(0) == BB;
Owen Anderson1d0be152009-08-13 21:58:54 +00001525 BI->setCondition(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
1526 CondIsTrue));
Chris Lattner867661a2008-07-13 21:53:26 +00001527 return true; // Nuke the branch on constant.
1528 }
1529
1530 // Otherwise, if there are multiple predecessors, insert a PHI that merges
1531 // in the constant and simplify the block result. Subsequent passes of
1532 // simplifycfg will thread the block.
1533 if (BlockIsSimpleEnoughToThreadThrough(BB)) {
Owen Anderson1d0be152009-08-13 21:58:54 +00001534 PHINode *NewPN = PHINode::Create(Type::getInt1Ty(BB->getContext()),
Chris Lattner867661a2008-07-13 21:53:26 +00001535 BI->getCondition()->getName() + ".pr",
1536 BB->begin());
Chris Lattnereb388af2008-07-13 21:55:46 +00001537 // Okay, we're going to insert the PHI node. Since PBI is not the only
1538 // predecessor, compute the PHI'd conditional value for all of the preds.
1539 // Any predecessor where the condition is not computable we keep symbolic.
Gabor Greif62539832010-07-12 10:59:23 +00001540 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
1541 BasicBlock *P = *PI;
1542 if ((PBI = dyn_cast<BranchInst>(P->getTerminator())) &&
Chris Lattner867661a2008-07-13 21:53:26 +00001543 PBI != BI && PBI->isConditional() &&
1544 PBI->getCondition() == BI->getCondition() &&
1545 PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
1546 bool CondIsTrue = PBI->getSuccessor(0) == BB;
Owen Anderson1d0be152009-08-13 21:58:54 +00001547 NewPN->addIncoming(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
Gabor Greif62539832010-07-12 10:59:23 +00001548 CondIsTrue), P);
Chris Lattner867661a2008-07-13 21:53:26 +00001549 } else {
Gabor Greif62539832010-07-12 10:59:23 +00001550 NewPN->addIncoming(BI->getCondition(), P);
Chris Lattner867661a2008-07-13 21:53:26 +00001551 }
Gabor Greif62539832010-07-12 10:59:23 +00001552 }
Chris Lattner867661a2008-07-13 21:53:26 +00001553
1554 BI->setCondition(NewPN);
Chris Lattner867661a2008-07-13 21:53:26 +00001555 return true;
1556 }
1557 }
1558
1559 // If this is a conditional branch in an empty block, and if any
1560 // predecessors is a conditional branch to one of our destinations,
1561 // fold the conditions into logical ops and one cond br.
Zhou Shenga8d57fe2009-02-26 06:56:37 +00001562 BasicBlock::iterator BBI = BB->begin();
1563 // Ignore dbg intrinsics.
1564 while (isa<DbgInfoIntrinsic>(BBI))
1565 ++BBI;
1566 if (&*BBI != BI)
Chris Lattnerb8245122008-07-13 22:04:41 +00001567 return false;
Chris Lattner63bf29b2009-01-20 01:15:41 +00001568
1569
1570 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(BI->getCondition()))
1571 if (CE->canTrap())
1572 return false;
Chris Lattnerb8245122008-07-13 22:04:41 +00001573
1574 int PBIOp, BIOp;
1575 if (PBI->getSuccessor(0) == BI->getSuccessor(0))
1576 PBIOp = BIOp = 0;
1577 else if (PBI->getSuccessor(0) == BI->getSuccessor(1))
1578 PBIOp = 0, BIOp = 1;
1579 else if (PBI->getSuccessor(1) == BI->getSuccessor(0))
1580 PBIOp = 1, BIOp = 0;
1581 else if (PBI->getSuccessor(1) == BI->getSuccessor(1))
1582 PBIOp = BIOp = 1;
1583 else
1584 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001585
Chris Lattnerb8245122008-07-13 22:04:41 +00001586 // Check to make sure that the other destination of this branch
1587 // isn't BB itself. If so, this is an infinite loop that will
1588 // keep getting unwound.
1589 if (PBI->getSuccessor(PBIOp) == BB)
1590 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001591
Chris Lattnerb8245122008-07-13 22:04:41 +00001592 // Do not perform this transformation if it would require
1593 // insertion of a large number of select instructions. For targets
1594 // without predication/cmovs, this is a big pessimization.
1595 BasicBlock *CommonDest = PBI->getSuccessor(PBIOp);
Chris Lattner867661a2008-07-13 21:53:26 +00001596
Chris Lattnerb8245122008-07-13 22:04:41 +00001597 unsigned NumPhis = 0;
1598 for (BasicBlock::iterator II = CommonDest->begin();
1599 isa<PHINode>(II); ++II, ++NumPhis)
1600 if (NumPhis > 2) // Disable this xform.
1601 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001602
Chris Lattnerb8245122008-07-13 22:04:41 +00001603 // Finally, if everything is ok, fold the branches to logical ops.
1604 BasicBlock *OtherDest = BI->getSuccessor(BIOp ^ 1);
1605
David Greene89d6fd32010-01-05 01:26:52 +00001606 DEBUG(dbgs() << "FOLDING BRs:" << *PBI->getParent()
Chris Lattnerbdff5482009-08-23 04:37:46 +00001607 << "AND: " << *BI->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001608
Chris Lattner093a4382008-07-13 22:23:11 +00001609
1610 // If OtherDest *is* BB, then BB is a basic block with a single conditional
1611 // branch in it, where one edge (OtherDest) goes back to itself but the other
1612 // exits. We don't *know* that the program avoids the infinite loop
1613 // (even though that seems likely). If we do this xform naively, we'll end up
1614 // recursively unpeeling the loop. Since we know that (after the xform is
1615 // done) that the block *is* infinite if reached, we just make it an obviously
1616 // infinite loop with no cond branch.
1617 if (OtherDest == BB) {
1618 // Insert it at the end of the function, because it's either code,
1619 // or it won't matter if it's hot. :)
Owen Anderson1d0be152009-08-13 21:58:54 +00001620 BasicBlock *InfLoopBlock = BasicBlock::Create(BB->getContext(),
1621 "infloop", BB->getParent());
Chris Lattner093a4382008-07-13 22:23:11 +00001622 BranchInst::Create(InfLoopBlock, InfLoopBlock);
1623 OtherDest = InfLoopBlock;
1624 }
1625
David Greene89d6fd32010-01-05 01:26:52 +00001626 DEBUG(dbgs() << *PBI->getParent()->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001627
1628 // BI may have other predecessors. Because of this, we leave
1629 // it alone, but modify PBI.
1630
1631 // Make sure we get to CommonDest on True&True directions.
1632 Value *PBICond = PBI->getCondition();
1633 if (PBIOp)
Dan Gohman4ae51262009-08-12 16:23:25 +00001634 PBICond = BinaryOperator::CreateNot(PBICond,
Chris Lattnerb8245122008-07-13 22:04:41 +00001635 PBICond->getName()+".not",
1636 PBI);
1637 Value *BICond = BI->getCondition();
1638 if (BIOp)
Dan Gohman4ae51262009-08-12 16:23:25 +00001639 BICond = BinaryOperator::CreateNot(BICond,
Chris Lattnerb8245122008-07-13 22:04:41 +00001640 BICond->getName()+".not",
1641 PBI);
1642 // Merge the conditions.
1643 Value *Cond = BinaryOperator::CreateOr(PBICond, BICond, "brmerge", PBI);
1644
1645 // Modify PBI to branch on the new condition to the new dests.
1646 PBI->setCondition(Cond);
1647 PBI->setSuccessor(0, CommonDest);
1648 PBI->setSuccessor(1, OtherDest);
1649
1650 // OtherDest may have phi nodes. If so, add an entry from PBI's
1651 // block that are identical to the entries for BI's block.
1652 PHINode *PN;
1653 for (BasicBlock::iterator II = OtherDest->begin();
1654 (PN = dyn_cast<PHINode>(II)); ++II) {
1655 Value *V = PN->getIncomingValueForBlock(BB);
1656 PN->addIncoming(V, PBI->getParent());
1657 }
1658
1659 // We know that the CommonDest already had an edge from PBI to
1660 // it. If it has PHIs though, the PHIs may have different
1661 // entries for BB and PBI's BB. If so, insert a select to make
1662 // them agree.
1663 for (BasicBlock::iterator II = CommonDest->begin();
1664 (PN = dyn_cast<PHINode>(II)); ++II) {
1665 Value *BIV = PN->getIncomingValueForBlock(BB);
1666 unsigned PBBIdx = PN->getBasicBlockIndex(PBI->getParent());
1667 Value *PBIV = PN->getIncomingValue(PBBIdx);
1668 if (BIV != PBIV) {
1669 // Insert a select in PBI to pick the right value.
1670 Value *NV = SelectInst::Create(PBICond, PBIV, BIV,
1671 PBIV->getName()+".mux", PBI);
1672 PN->setIncomingValue(PBBIdx, NV);
Chris Lattner867661a2008-07-13 21:53:26 +00001673 }
1674 }
Chris Lattnerb8245122008-07-13 22:04:41 +00001675
David Greene89d6fd32010-01-05 01:26:52 +00001676 DEBUG(dbgs() << "INTO: " << *PBI->getParent());
1677 DEBUG(dbgs() << *PBI->getParent()->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001678
1679 // This basic block is probably dead. We know it has at least
1680 // one fewer predecessor.
1681 return true;
Chris Lattner867661a2008-07-13 21:53:26 +00001682}
1683
Frits van Bommel7ac40c32010-12-05 18:29:03 +00001684// SimplifyIndirectBrOnSelect - Replaces
1685// (indirectbr (select cond, blockaddress(@fn, BlockA),
1686// blockaddress(@fn, BlockB)))
1687// with
1688// (br cond, BlockA, BlockB).
1689static bool SimplifyIndirectBrOnSelect(IndirectBrInst *IBI, SelectInst *SI) {
1690 // Check that both operands of the select are block addresses.
1691 BlockAddress *TBA = dyn_cast<BlockAddress>(SI->getTrueValue());
1692 BlockAddress *FBA = dyn_cast<BlockAddress>(SI->getFalseValue());
1693 if (!TBA || !FBA)
1694 return false;
1695
1696 // Extract the actual blocks.
1697 BasicBlock *TrueBB = TBA->getBasicBlock();
1698 BasicBlock *FalseBB = FBA->getBasicBlock();
1699
1700 // Remove any superfluous successor edges from the CFG.
1701 // First, figure out which successors to preserve.
1702 // If TrueBB and FalseBB are equal, only try to preserve one copy of that
1703 // successor.
1704 BasicBlock *KeepEdge1 = TrueBB;
1705 BasicBlock *KeepEdge2 = TrueBB != FalseBB ? FalseBB : 0;
1706
1707 // Then remove the rest.
1708 for (unsigned I = 0, E = IBI->getNumSuccessors(); I != E; ++I) {
1709 BasicBlock *Succ = IBI->getSuccessor(I);
1710 // Make sure only to keep exactly one copy of each edge.
1711 if (Succ == KeepEdge1)
1712 KeepEdge1 = 0;
1713 else if (Succ == KeepEdge2)
1714 KeepEdge2 = 0;
1715 else
1716 Succ->removePredecessor(IBI->getParent());
1717 }
1718
1719 // Insert an appropriate new terminator.
1720 if ((KeepEdge1 == 0) && (KeepEdge2 == 0)) {
1721 if (TrueBB == FalseBB)
1722 // We were only looking for one successor, and it was present.
1723 // Create an unconditional branch to it.
1724 BranchInst::Create(TrueBB, IBI);
1725 else
1726 // We found both of the successors we were looking for.
1727 // Create a conditional branch sharing the condition of the select.
1728 BranchInst::Create(TrueBB, FalseBB, SI->getCondition(), IBI);
1729 } else if (KeepEdge1 && (KeepEdge2 || TrueBB == FalseBB)) {
1730 // Neither of the selected blocks were successors, so this
1731 // indirectbr must be unreachable.
1732 new UnreachableInst(IBI->getContext(), IBI);
1733 } else {
1734 // One of the selected values was a successor, but the other wasn't.
1735 // Insert an unconditional branch to the one that was found;
1736 // the edge to the one that wasn't must be unreachable.
1737 if (KeepEdge1 == 0)
1738 // Only TrueBB was found.
1739 BranchInst::Create(TrueBB, IBI);
1740 else
1741 // Only FalseBB was found.
1742 BranchInst::Create(FalseBB, IBI);
1743 }
1744
1745 EraseTerminatorInstAndDCECond(IBI);
1746 return true;
1747}
1748
Chris Lattner61c77442010-12-13 03:18:54 +00001749/// TryToSimplifyUncondBranchWithICmpInIt - This is called when we find an icmp
1750/// instruction (a seteq/setne with a constant) as the only instruction in a
1751/// block that ends with an uncond branch. We are looking for a very specific
1752/// pattern that occurs when "A == 1 || A == 2 || A == 3" gets simplified. In
1753/// this case, we merge the first two "or's of icmp" into a switch, but then the
1754/// default value goes to an uncond block with a seteq in it, we get something
1755/// like:
1756///
1757/// switch i8 %A, label %DEFAULT [ i8 1, label %end i8 2, label %end ]
1758/// DEFAULT:
1759/// %tmp = icmp eq i8 %A, 92
1760/// br label %end
1761/// end:
1762/// ... = phi i1 [ true, %entry ], [ %tmp, %DEFAULT ], [ true, %entry ]
1763///
1764/// We prefer to split the edge to 'end' so that there is a true/false entry to
1765/// the PHI, merging the third icmp into the switch.
1766static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI) {
1767 BasicBlock *BB = ICI->getParent();
1768 // If the block has any PHIs in it or the icmp has multiple uses, it is too
1769 // complex.
1770 if (isa<PHINode>(BB->begin()) || !ICI->hasOneUse()) return false;
1771
1772 Value *V = ICI->getOperand(0);
1773 ConstantInt *Cst = cast<ConstantInt>(ICI->getOperand(1));
1774
1775 // The pattern we're looking for is where our only predecessor is a switch on
1776 // 'V' and this block is the default case for the switch. In this case we can
1777 // fold the compared value into the switch to simplify things.
1778 BasicBlock *Pred = BB->getSinglePredecessor();
1779 if (Pred == 0 || !isa<SwitchInst>(Pred->getTerminator())) return false;
1780
1781 SwitchInst *SI = cast<SwitchInst>(Pred->getTerminator());
1782 if (SI->getCondition() != V)
1783 return false;
1784
1785 // If BB is reachable on a non-default case, then we simply know the value of
1786 // V in this block. Substitute it and constant fold the icmp instruction
1787 // away.
1788 if (SI->getDefaultDest() != BB) {
1789 ConstantInt *VVal = SI->findCaseDest(BB);
1790 assert(VVal && "Should have a unique destination value");
1791 ICI->setOperand(0, VVal);
1792
1793 if (Constant *C = ConstantFoldInstruction(ICI)) {
1794 ICI->replaceAllUsesWith(C);
1795 ICI->eraseFromParent();
1796 }
1797 // BB is now empty, so it is likely to simplify away.
1798 return SimplifyCFG(BB) | true;
1799 }
1800
Chris Lattnerabf70672010-12-13 03:43:57 +00001801 // Ok, the block is reachable from the default dest. If the constant we're
1802 // comparing exists in one of the other edges, then we can constant fold ICI
1803 // and zap it.
1804 if (SI->findCaseValue(Cst) != 0) {
1805 Value *V;
1806 if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
1807 V = ConstantInt::getFalse(BB->getContext());
1808 else
1809 V = ConstantInt::getTrue(BB->getContext());
1810
1811 ICI->replaceAllUsesWith(V);
1812 ICI->eraseFromParent();
1813 // BB is now empty, so it is likely to simplify away.
1814 return SimplifyCFG(BB) | true;
1815 }
1816
Chris Lattner61c77442010-12-13 03:18:54 +00001817 // The use of the icmp has to be in the 'end' block, by the only PHI node in
1818 // the block.
1819 BasicBlock *SuccBlock = BB->getTerminator()->getSuccessor(0);
1820 PHINode *PHIUse = dyn_cast<PHINode>(ICI->use_back());
1821 if (PHIUse == 0 || PHIUse != &SuccBlock->front() ||
1822 isa<PHINode>(++BasicBlock::iterator(PHIUse)))
1823 return false;
1824
1825 // If the icmp is a SETEQ, then the default dest gets false, the new edge gets
1826 // true in the PHI.
1827 Constant *DefaultCst = ConstantInt::getTrue(BB->getContext());
1828 Constant *NewCst = ConstantInt::getFalse(BB->getContext());
1829
1830 if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
1831 std::swap(DefaultCst, NewCst);
1832
1833 // Replace ICI (which is used by the PHI for the default value) with true or
1834 // false depending on if it is EQ or NE.
1835 ICI->replaceAllUsesWith(DefaultCst);
1836 ICI->eraseFromParent();
1837
1838 // Okay, the switch goes to this block on a default value. Add an edge from
1839 // the switch to the merge point on the compared value.
1840 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "switch.edge",
1841 BB->getParent(), BB);
1842 SI->addCase(Cst, NewBB);
1843
1844 // NewBB branches to the phi block, add the uncond branch and the phi entry.
1845 BranchInst::Create(SuccBlock, NewBB);
1846 PHIUse->addIncoming(NewCst, NewBB);
1847 return true;
1848}
1849
Chris Lattner97fdb892010-12-13 05:03:41 +00001850/// SimplifyBranchOnICmpChain - The specified branch is a conditional branch.
1851/// Check to see if it is branching on an or/and chain of icmp instructions, and
1852/// fold it into a switch instruction if so.
1853static bool SimplifyBranchOnICmpChain(BranchInst *BI, const TargetData *TD) {
1854 Instruction *Cond = dyn_cast<Instruction>(BI->getCondition());
1855 if (Cond == 0) return false;
1856
1857
1858 // Change br (X == 0 | X == 1), T, F into a switch instruction.
1859 // If this is a bunch of seteq's or'd together, or if it's a bunch of
1860 // 'setne's and'ed together, collect them.
1861 Value *CompVal = 0;
1862 std::vector<ConstantInt*> Values;
1863 bool TrueWhenEqual = true;
1864 Value *ExtraCase = 0;
1865
1866 if (Cond->getOpcode() == Instruction::Or) {
1867 CompVal = GatherConstantCompares(Cond, Values, ExtraCase, TD, true);
1868 } else if (Cond->getOpcode() == Instruction::And) {
1869 CompVal = GatherConstantCompares(Cond, Values, ExtraCase, TD, false);
1870 TrueWhenEqual = false;
1871 }
1872
1873 // If we didn't have a multiply compared value, fail.
1874 if (CompVal == 0) return false;
1875
1876 // There might be duplicate constants in the list, which the switch
1877 // instruction can't handle, remove them now.
1878 array_pod_sort(Values.begin(), Values.end(), ConstantIntSortPredicate);
1879 Values.erase(std::unique(Values.begin(), Values.end()), Values.end());
1880
1881 // If Extra was used, we require at least two switch values to do the
1882 // transformation. A switch with one value is just an cond branch.
1883 if (ExtraCase && Values.size() < 2) return false;
1884
1885 // Figure out which block is which destination.
1886 BasicBlock *DefaultBB = BI->getSuccessor(1);
1887 BasicBlock *EdgeBB = BI->getSuccessor(0);
1888 if (!TrueWhenEqual) std::swap(DefaultBB, EdgeBB);
1889
1890 BasicBlock *BB = BI->getParent();
1891
1892 // If there are any extra values that couldn't be folded into the switch
1893 // then we evaluate them with an explicit branch first. Split the block
1894 // right before the condbr to handle it.
1895 if (ExtraCase) {
1896 BasicBlock *NewBB = BB->splitBasicBlock(BI, "switch.early.test");
1897 // Remove the uncond branch added to the old block.
1898 TerminatorInst *OldTI = BB->getTerminator();
1899
1900 BranchInst::Create(EdgeBB, NewBB, ExtraCase, OldTI);
1901 OldTI->eraseFromParent();
1902 BB = NewBB;
1903 }
1904
1905 // Convert pointer to int before we switch.
1906 if (CompVal->getType()->isPointerTy()) {
1907 assert(TD && "Cannot switch on pointer without TargetData");
1908 CompVal = new PtrToIntInst(CompVal,
1909 TD->getIntPtrType(CompVal->getContext()),
1910 "magicptr", BI);
1911 }
1912
1913 // Create the new switch instruction now.
1914 SwitchInst *New =
1915 SwitchInst::Create(CompVal, DefaultBB, Values.size(), BI);
1916
1917 // Add all of the 'cases' to the switch instruction.
1918 for (unsigned i = 0, e = Values.size(); i != e; ++i)
1919 New->addCase(Values[i], EdgeBB);
1920
1921 // We added edges from PI to the EdgeBB. As such, if there were any
1922 // PHI nodes in EdgeBB, they need entries to be added corresponding to
1923 // the number of edges added.
1924 for (BasicBlock::iterator BBI = EdgeBB->begin();
1925 isa<PHINode>(BBI); ++BBI) {
1926 PHINode *PN = cast<PHINode>(BBI);
1927 Value *InVal = PN->getIncomingValueForBlock(BB);
1928 for (unsigned i = 0, e = Values.size()-1; i != e; ++i)
1929 PN->addIncoming(InVal, BB);
1930 }
1931
1932 // Erase the old branch instruction.
1933 EraseTerminatorInstAndDCECond(BI);
1934 return true;
1935}
1936
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00001937bool SimplifyCFGOpt::run(BasicBlock *BB) {
Chris Lattnerdc3602b2003-08-24 18:36:16 +00001938 bool Changed = false;
Duncan Sands5f284752010-10-24 12:23:30 +00001939 Function *Fn = BB->getParent();
Chris Lattner01d1ee32002-05-21 20:50:24 +00001940
Duncan Sands5f284752010-10-24 12:23:30 +00001941 assert(BB && Fn && "Block not embedded in function!");
Chris Lattner01d1ee32002-05-21 20:50:24 +00001942 assert(BB->getTerminator() && "Degenerate basic block encountered!");
Chris Lattner01d1ee32002-05-21 20:50:24 +00001943
Dan Gohmane2c6d132010-08-14 00:29:42 +00001944 // Remove basic blocks that have no predecessors (except the entry block)...
1945 // or that just have themself as a predecessor. These are unreachable.
Duncan Sands5f284752010-10-24 12:23:30 +00001946 if ((pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) ||
Dan Gohmane2c6d132010-08-14 00:29:42 +00001947 BB->getSinglePredecessor() == BB) {
David Greene89d6fd32010-01-05 01:26:52 +00001948 DEBUG(dbgs() << "Removing BB: \n" << *BB);
Chris Lattner71af9b02008-12-03 06:40:52 +00001949 DeleteDeadBlock(BB);
Chris Lattner01d1ee32002-05-21 20:50:24 +00001950 return true;
1951 }
1952
Chris Lattner694e37f2003-08-17 19:41:53 +00001953 // Check to see if we can constant propagate this terminator instruction
1954 // away...
Chris Lattnerdc3602b2003-08-24 18:36:16 +00001955 Changed |= ConstantFoldTerminator(BB);
Chris Lattner694e37f2003-08-17 19:41:53 +00001956
Dan Gohman2c635662009-10-30 22:39:04 +00001957 // Check for and eliminate duplicate PHI nodes in this block.
1958 Changed |= EliminateDuplicatePHINodes(BB);
1959
Chris Lattnerddb97a22010-12-13 05:10:48 +00001960 // Merge basic blocks into their predecessor if there is only one distinct
1961 // pred, and if there is only one distinct successor of the predecessor, and
1962 // if there are no PHI nodes.
1963 //
1964 if (MergeBlockIntoPredecessor(BB))
1965 return true;
1966
Dan Gohman882d87d2008-03-11 21:53:06 +00001967 // If there is a trivial two-entry PHI node in this basic block, and we can
1968 // eliminate it, do so now.
1969 if (PHINode *PN = dyn_cast<PHINode>(BB->begin()))
1970 if (PN->getNumIncomingValues() == 2)
1971 Changed |= FoldTwoEntryPHINode(PN);
1972
Chris Lattner19831ec2004-02-16 06:35:48 +00001973 // If this is a returning block with only PHI nodes in it, fold the return
1974 // instruction into any unconditional branch predecessors.
Chris Lattner147af6b2004-04-02 18:13:43 +00001975 //
1976 // If any predecessor is a conditional branch that just selects among
1977 // different return values, fold the replace the branch/return with a select
1978 // and return.
Chris Lattner19831ec2004-02-16 06:35:48 +00001979 if (ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator())) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001980 if (BB->getFirstNonPHIOrDbg()->isTerminator()) {
Chris Lattner147af6b2004-04-02 18:13:43 +00001981 // Find predecessors that end with branches.
Chris Lattner82442432008-02-18 07:42:56 +00001982 SmallVector<BasicBlock*, 8> UncondBranchPreds;
1983 SmallVector<BranchInst*, 8> CondBranchPreds;
Chris Lattner19831ec2004-02-16 06:35:48 +00001984 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
Gabor Greif58969352010-07-09 15:25:09 +00001985 BasicBlock *P = *PI;
1986 TerminatorInst *PTI = P->getTerminator();
Anton Korobeynikov07e6e562008-02-20 11:26:25 +00001987 if (BranchInst *BI = dyn_cast<BranchInst>(PTI)) {
Chris Lattner19831ec2004-02-16 06:35:48 +00001988 if (BI->isUnconditional())
Gabor Greif58969352010-07-09 15:25:09 +00001989 UncondBranchPreds.push_back(P);
Chris Lattner147af6b2004-04-02 18:13:43 +00001990 else
1991 CondBranchPreds.push_back(BI);
Anton Korobeynikov07e6e562008-02-20 11:26:25 +00001992 }
Chris Lattner19831ec2004-02-16 06:35:48 +00001993 }
Misha Brukmanfd939082005-04-21 23:48:37 +00001994
Chris Lattner19831ec2004-02-16 06:35:48 +00001995 // If we found some, do the transformation!
Bill Wendling1c855032009-03-03 19:25:16 +00001996 if (!UncondBranchPreds.empty()) {
Chris Lattner19831ec2004-02-16 06:35:48 +00001997 while (!UncondBranchPreds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +00001998 BasicBlock *Pred = UncondBranchPreds.pop_back_val();
David Greene89d6fd32010-01-05 01:26:52 +00001999 DEBUG(dbgs() << "FOLDING: " << *BB
Chris Lattnerbdff5482009-08-23 04:37:46 +00002000 << "INTO UNCOND BRANCH PRED: " << *Pred);
Chris Lattner19831ec2004-02-16 06:35:48 +00002001 Instruction *UncondBranch = Pred->getTerminator();
2002 // Clone the return and add it to the end of the predecessor.
Nick Lewycky67760642009-09-27 07:38:41 +00002003 Instruction *NewRet = RI->clone();
Chris Lattner19831ec2004-02-16 06:35:48 +00002004 Pred->getInstList().push_back(NewRet);
2005
2006 // If the return instruction returns a value, and if the value was a
2007 // PHI node in "BB", propagate the right value into the return.
Gabor Greiff7ea3632008-06-10 22:03:26 +00002008 for (User::op_iterator i = NewRet->op_begin(), e = NewRet->op_end();
2009 i != e; ++i)
2010 if (PHINode *PN = dyn_cast<PHINode>(*i))
Chris Lattner19831ec2004-02-16 06:35:48 +00002011 if (PN->getParent() == BB)
Gabor Greiff7ea3632008-06-10 22:03:26 +00002012 *i = PN->getIncomingValueForBlock(Pred);
Chris Lattnerffba5822008-04-28 00:19:07 +00002013
Chris Lattner19831ec2004-02-16 06:35:48 +00002014 // Update any PHI nodes in the returning block to realize that we no
2015 // longer branch to them.
2016 BB->removePredecessor(Pred);
2017 Pred->getInstList().erase(UncondBranch);
2018 }
2019
2020 // If we eliminated all predecessors of the block, delete the block now.
2021 if (pred_begin(BB) == pred_end(BB))
2022 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00002023 Fn->getBasicBlockList().erase(BB);
Chris Lattner19831ec2004-02-16 06:35:48 +00002024
Chris Lattner19831ec2004-02-16 06:35:48 +00002025 return true;
2026 }
Chris Lattner147af6b2004-04-02 18:13:43 +00002027
2028 // Check out all of the conditional branches going to this return
2029 // instruction. If any of them just select between returns, change the
2030 // branch itself into a select/return pair.
2031 while (!CondBranchPreds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +00002032 BranchInst *BI = CondBranchPreds.pop_back_val();
Chris Lattner147af6b2004-04-02 18:13:43 +00002033
2034 // Check to see if the non-BB successor is also a return block.
Chris Lattnerc9e495c2008-04-24 00:01:19 +00002035 if (isa<ReturnInst>(BI->getSuccessor(0)->getTerminator()) &&
2036 isa<ReturnInst>(BI->getSuccessor(1)->getTerminator()) &&
2037 SimplifyCondBranchToTwoReturns(BI))
2038 return true;
Chris Lattner147af6b2004-04-02 18:13:43 +00002039 }
Chris Lattner19831ec2004-02-16 06:35:48 +00002040 }
Reid Spencer3ed469c2006-11-02 20:25:50 +00002041 } else if (isa<UnwindInst>(BB->begin())) {
Chris Lattnere14ea082004-02-24 05:54:22 +00002042 // Check to see if the first instruction in this block is just an unwind.
2043 // If so, replace any invoke instructions which use this as an exception
Chris Lattner11f15db2009-10-13 18:13:05 +00002044 // destination with call instructions.
Chris Lattnere14ea082004-02-24 05:54:22 +00002045 //
Chris Lattner82442432008-02-18 07:42:56 +00002046 SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
Chris Lattnere14ea082004-02-24 05:54:22 +00002047 while (!Preds.empty()) {
2048 BasicBlock *Pred = Preds.back();
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002049 InvokeInst *II = dyn_cast<InvokeInst>(Pred->getTerminator());
2050 if (II && II->getUnwindDest() == BB) {
2051 // Insert a new branch instruction before the invoke, because this
2052 // is now a fall through.
2053 BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
2054 Pred->getInstList().remove(II); // Take out of symbol table
Misha Brukmanfd939082005-04-21 23:48:37 +00002055
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002056 // Insert the call now.
2057 SmallVector<Value*,8> Args(II->op_begin(), II->op_end()-3);
2058 CallInst *CI = CallInst::Create(II->getCalledValue(),
2059 Args.begin(), Args.end(),
2060 II->getName(), BI);
2061 CI->setCallingConv(II->getCallingConv());
2062 CI->setAttributes(II->getAttributes());
2063 // If the invoke produced a value, the Call now does instead.
2064 II->replaceAllUsesWith(CI);
2065 delete II;
2066 Changed = true;
2067 }
Misha Brukmanfd939082005-04-21 23:48:37 +00002068
Chris Lattnere14ea082004-02-24 05:54:22 +00002069 Preds.pop_back();
2070 }
Chris Lattner8e509dd2004-02-24 16:09:21 +00002071
Duncan Sands5f284752010-10-24 12:23:30 +00002072 // If this block is now dead (and isn't the entry block), remove it.
2073 if (pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) {
Chris Lattner8e509dd2004-02-24 16:09:21 +00002074 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00002075 Fn->getBasicBlockList().erase(BB);
Chris Lattner8e509dd2004-02-24 16:09:21 +00002076 return true;
2077 }
2078
Chris Lattner623369a2005-02-24 06:17:52 +00002079 } else if (SwitchInst *SI = dyn_cast<SwitchInst>(BB->getTerminator())) {
2080 if (isValueEqualityComparison(SI)) {
2081 // If we only have one predecessor, and if it is a branch on this value,
2082 // see if that predecessor totally determines the outcome of this switch.
2083 if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
2084 if (SimplifyEqualityComparisonWithOnlyPredecessor(SI, OnlyPred))
2085 return SimplifyCFG(BB) || 1;
2086
2087 // If the block only contains the switch, see if we can fold the block
2088 // away into any preds.
Zhou Sheng9a7c7432009-02-25 15:34:27 +00002089 BasicBlock::iterator BBI = BB->begin();
2090 // Ignore dbg intrinsics.
2091 while (isa<DbgInfoIntrinsic>(BBI))
2092 ++BBI;
2093 if (SI == &*BBI)
Chris Lattner623369a2005-02-24 06:17:52 +00002094 if (FoldValueComparisonIntoPredecessors(SI))
2095 return SimplifyCFG(BB) || 1;
2096 }
Chris Lattner542f1492004-02-28 21:28:10 +00002097 } else if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator())) {
Chris Lattner7e663482005-08-03 00:11:16 +00002098 if (BI->isUnconditional()) {
Chris Lattnerdcb54ce2010-12-13 01:28:06 +00002099 // If the Terminator is the only non-phi instruction, simplify the block.
Chris Lattner61c77442010-12-13 03:18:54 +00002100 BasicBlock::iterator I = BB->getFirstNonPHIOrDbg();
Chris Lattnerdcb54ce2010-12-13 01:28:06 +00002101 if (I->isTerminator() && BB != &Fn->getEntryBlock() &&
2102 TryToSimplifyUncondBranchFromEmptyBlock(BB))
2103 return true;
Chris Lattner7e663482005-08-03 00:11:16 +00002104
Chris Lattner61c77442010-12-13 03:18:54 +00002105 // If the only instruction in the block is a seteq/setne comparison
2106 // against a constant, try to simplify the block.
2107 if (ICmpInst *ICI = dyn_cast<ICmpInst>(I))
2108 if (ICI->isEquality() && isa<ConstantInt>(ICI->getOperand(1))) {
2109 for (++I; isa<DbgInfoIntrinsic>(I); ++I)
2110 ;
2111 if (I->isTerminator() &&
2112 TryToSimplifyUncondBranchWithICmpInIt(ICI))
2113 return true;
2114 }
2115
Chris Lattner7e663482005-08-03 00:11:16 +00002116 } else { // Conditional branch
Reid Spencer3ed469c2006-11-02 20:25:50 +00002117 if (isValueEqualityComparison(BI)) {
Chris Lattner623369a2005-02-24 06:17:52 +00002118 // If we only have one predecessor, and if it is a branch on this value,
2119 // see if that predecessor totally determines the outcome of this
2120 // switch.
2121 if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
2122 if (SimplifyEqualityComparisonWithOnlyPredecessor(BI, OnlyPred))
Bill Wendlingc69b4a52010-03-14 10:40:28 +00002123 return SimplifyCFG(BB) | true;
Chris Lattner623369a2005-02-24 06:17:52 +00002124
Chris Lattnere67fa052004-05-01 23:35:43 +00002125 // This block must be empty, except for the setcond inst, if it exists.
Devang Patel556b20a2009-02-04 01:06:11 +00002126 // Ignore dbg intrinsics.
Chris Lattnere67fa052004-05-01 23:35:43 +00002127 BasicBlock::iterator I = BB->begin();
Devang Pateld0a203d2009-02-04 21:39:48 +00002128 // Ignore dbg intrinsics.
Devang Patel556b20a2009-02-04 01:06:11 +00002129 while (isa<DbgInfoIntrinsic>(I))
Devang Pateld0a203d2009-02-04 21:39:48 +00002130 ++I;
2131 if (&*I == BI) {
Chris Lattnere67fa052004-05-01 23:35:43 +00002132 if (FoldValueComparisonIntoPredecessors(BI))
2133 return SimplifyCFG(BB) | true;
Devang Pateld0a203d2009-02-04 21:39:48 +00002134 } else if (&*I == cast<Instruction>(BI->getCondition())){
2135 ++I;
2136 // Ignore dbg intrinsics.
2137 while (isa<DbgInfoIntrinsic>(I))
2138 ++I;
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002139 if (&*I == BI && FoldValueComparisonIntoPredecessors(BI))
2140 return SimplifyCFG(BB) | true;
Devang Pateld0a203d2009-02-04 21:39:48 +00002141 }
Chris Lattnere67fa052004-05-01 23:35:43 +00002142 }
Devang Pateld0a203d2009-02-04 21:39:48 +00002143
Chris Lattner97fdb892010-12-13 05:03:41 +00002144 // Try to turn "br (X == 0 | X == 1), T, F" into a switch instruction.
2145 if (SimplifyBranchOnICmpChain(BI, TD))
2146 return true;
2147
Chris Lattnereaba3a12005-09-19 23:49:37 +00002148 // If this is a branch on a phi node in the current block, thread control
2149 // through this block if any PHI node entries are constants.
2150 if (PHINode *PN = dyn_cast<PHINode>(BI->getCondition()))
2151 if (PN->getParent() == BI->getParent())
2152 if (FoldCondBranchOnPHI(BI))
2153 return SimplifyCFG(BB) | true;
Chris Lattnere67fa052004-05-01 23:35:43 +00002154
2155 // If this basic block is ONLY a setcc and a branch, and if a predecessor
2156 // branches to us and one of our successors, fold the setcc into the
2157 // predecessor and use logical operations to pick the right destination.
Chris Lattner1347e872008-07-13 21:12:01 +00002158 if (FoldBranchToCommonDest(BI))
Bill Wendlingc69b4a52010-03-14 10:40:28 +00002159 return SimplifyCFG(BB) | true;
Chris Lattnere67fa052004-05-01 23:35:43 +00002160
Chris Lattner867661a2008-07-13 21:53:26 +00002161 // Scan predecessor blocks for conditional branches.
Chris Lattner2e42e362005-09-20 00:43:16 +00002162 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI)
2163 if (BranchInst *PBI = dyn_cast<BranchInst>((*PI)->getTerminator()))
Chris Lattner867661a2008-07-13 21:53:26 +00002164 if (PBI != BI && PBI->isConditional())
2165 if (SimplifyCondBranchToCondBranch(PBI, BI))
2166 return SimplifyCFG(BB) | true;
Chris Lattnerd52c2612004-02-24 07:23:58 +00002167 }
Chris Lattner698f96f2004-10-18 04:07:22 +00002168 } else if (isa<UnreachableInst>(BB->getTerminator())) {
2169 // If there are any instructions immediately before the unreachable that can
2170 // be removed, do so.
2171 Instruction *Unreachable = BB->getTerminator();
2172 while (Unreachable != BB->begin()) {
2173 BasicBlock::iterator BBI = Unreachable;
2174 --BBI;
Chris Lattnerf8131c92008-10-29 17:46:26 +00002175 // Do not delete instructions that can have side effects, like calls
2176 // (which may never return) and volatile loads and stores.
Dale Johannesen80b8a622009-03-12 17:42:45 +00002177 if (isa<CallInst>(BBI) && !isa<DbgInfoIntrinsic>(BBI)) break;
Chris Lattnerf8131c92008-10-29 17:46:26 +00002178
2179 if (StoreInst *SI = dyn_cast<StoreInst>(BBI))
2180 if (SI->isVolatile())
2181 break;
2182
2183 if (LoadInst *LI = dyn_cast<LoadInst>(BBI))
2184 if (LI->isVolatile())
2185 break;
2186
Chris Lattner698f96f2004-10-18 04:07:22 +00002187 // Delete this instruction
2188 BB->getInstList().erase(BBI);
2189 Changed = true;
2190 }
2191
2192 // If the unreachable instruction is the first in the block, take a gander
2193 // at all of the predecessors of this instruction, and simplify them.
2194 if (&BB->front() == Unreachable) {
Chris Lattner82442432008-02-18 07:42:56 +00002195 SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
Chris Lattner698f96f2004-10-18 04:07:22 +00002196 for (unsigned i = 0, e = Preds.size(); i != e; ++i) {
2197 TerminatorInst *TI = Preds[i]->getTerminator();
2198
2199 if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
2200 if (BI->isUnconditional()) {
2201 if (BI->getSuccessor(0) == BB) {
Owen Anderson1d0be152009-08-13 21:58:54 +00002202 new UnreachableInst(TI->getContext(), TI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002203 TI->eraseFromParent();
2204 Changed = true;
2205 }
2206 } else {
2207 if (BI->getSuccessor(0) == BB) {
Gabor Greif051a9502008-04-06 20:25:17 +00002208 BranchInst::Create(BI->getSuccessor(1), BI);
Eli Friedman080efb82008-12-16 20:54:32 +00002209 EraseTerminatorInstAndDCECond(BI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002210 } else if (BI->getSuccessor(1) == BB) {
Gabor Greif051a9502008-04-06 20:25:17 +00002211 BranchInst::Create(BI->getSuccessor(0), BI);
Eli Friedman080efb82008-12-16 20:54:32 +00002212 EraseTerminatorInstAndDCECond(BI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002213 Changed = true;
2214 }
2215 }
2216 } else if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
2217 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2218 if (SI->getSuccessor(i) == BB) {
Chris Lattner42eb7522005-05-20 22:19:54 +00002219 BB->removePredecessor(SI->getParent());
Chris Lattner698f96f2004-10-18 04:07:22 +00002220 SI->removeCase(i);
2221 --i; --e;
2222 Changed = true;
2223 }
2224 // If the default value is unreachable, figure out the most popular
2225 // destination and make it the default.
2226 if (SI->getSuccessor(0) == BB) {
2227 std::map<BasicBlock*, unsigned> Popularity;
2228 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2229 Popularity[SI->getSuccessor(i)]++;
2230
2231 // Find the most popular block.
2232 unsigned MaxPop = 0;
2233 BasicBlock *MaxBlock = 0;
2234 for (std::map<BasicBlock*, unsigned>::iterator
2235 I = Popularity.begin(), E = Popularity.end(); I != E; ++I) {
2236 if (I->second > MaxPop) {
2237 MaxPop = I->second;
2238 MaxBlock = I->first;
2239 }
2240 }
2241 if (MaxBlock) {
2242 // Make this the new default, allowing us to delete any explicit
2243 // edges to it.
2244 SI->setSuccessor(0, MaxBlock);
2245 Changed = true;
2246
Chris Lattner42eb7522005-05-20 22:19:54 +00002247 // If MaxBlock has phinodes in it, remove MaxPop-1 entries from
2248 // it.
2249 if (isa<PHINode>(MaxBlock->begin()))
2250 for (unsigned i = 0; i != MaxPop-1; ++i)
2251 MaxBlock->removePredecessor(SI->getParent());
2252
Chris Lattner698f96f2004-10-18 04:07:22 +00002253 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2254 if (SI->getSuccessor(i) == MaxBlock) {
2255 SI->removeCase(i);
2256 --i; --e;
2257 }
2258 }
2259 }
2260 } else if (InvokeInst *II = dyn_cast<InvokeInst>(TI)) {
2261 if (II->getUnwindDest() == BB) {
2262 // Convert the invoke to a call instruction. This would be a good
2263 // place to note that the call does not throw though.
Gabor Greif051a9502008-04-06 20:25:17 +00002264 BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
Chris Lattner698f96f2004-10-18 04:07:22 +00002265 II->removeFromParent(); // Take out of symbol table
Misha Brukmanfd939082005-04-21 23:48:37 +00002266
Chris Lattner698f96f2004-10-18 04:07:22 +00002267 // Insert the call now...
Gabor Greifbd443142010-03-30 19:20:53 +00002268 SmallVector<Value*, 8> Args(II->op_begin(), II->op_end()-3);
Gabor Greif051a9502008-04-06 20:25:17 +00002269 CallInst *CI = CallInst::Create(II->getCalledValue(),
2270 Args.begin(), Args.end(),
2271 II->getName(), BI);
Chris Lattner16d0db22005-05-14 12:21:56 +00002272 CI->setCallingConv(II->getCallingConv());
Devang Patel05988662008-09-25 21:00:45 +00002273 CI->setAttributes(II->getAttributes());
Gabor Greifbd443142010-03-30 19:20:53 +00002274 // If the invoke produced a value, the call does now instead.
Chris Lattner698f96f2004-10-18 04:07:22 +00002275 II->replaceAllUsesWith(CI);
2276 delete II;
2277 Changed = true;
2278 }
2279 }
2280 }
2281
2282 // If this block is now dead, remove it.
Duncan Sands5f284752010-10-24 12:23:30 +00002283 if (pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) {
Chris Lattner698f96f2004-10-18 04:07:22 +00002284 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00002285 Fn->getBasicBlockList().erase(BB);
Chris Lattner698f96f2004-10-18 04:07:22 +00002286 return true;
2287 }
2288 }
Dan Gohman7a499432010-08-16 14:41:14 +00002289 } else if (IndirectBrInst *IBI =
2290 dyn_cast<IndirectBrInst>(BB->getTerminator())) {
Dan Gohmane2c6d132010-08-14 00:29:42 +00002291 // Eliminate redundant destinations.
2292 SmallPtrSet<Value *, 8> Succs;
2293 for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) {
2294 BasicBlock *Dest = IBI->getDestination(i);
Dan Gohman7a499432010-08-16 14:41:14 +00002295 if (!Dest->hasAddressTaken() || !Succs.insert(Dest)) {
Dan Gohmane2c6d132010-08-14 00:29:42 +00002296 Dest->removePredecessor(BB);
2297 IBI->removeDestination(i);
2298 --i; --e;
2299 Changed = true;
2300 }
2301 }
2302
2303 if (IBI->getNumDestinations() == 0) {
2304 // If the indirectbr has no successors, change it to unreachable.
2305 new UnreachableInst(IBI->getContext(), IBI);
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002306 EraseTerminatorInstAndDCECond(IBI);
Dan Gohmane2c6d132010-08-14 00:29:42 +00002307 Changed = true;
2308 } else if (IBI->getNumDestinations() == 1) {
2309 // If the indirectbr has one successor, change it to a direct branch.
2310 BranchInst::Create(IBI->getDestination(0), IBI);
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002311 EraseTerminatorInstAndDCECond(IBI);
Dan Gohmane2c6d132010-08-14 00:29:42 +00002312 Changed = true;
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002313 } else if (SelectInst *SI = dyn_cast<SelectInst>(IBI->getAddress())) {
2314 if (SimplifyIndirectBrOnSelect(IBI, SI))
2315 return SimplifyCFG(BB) | true;
Dan Gohmane2c6d132010-08-14 00:29:42 +00002316 }
Chris Lattner19831ec2004-02-16 06:35:48 +00002317 }
2318
Owen Andersoncfa94192008-07-18 17:49:43 +00002319 // Otherwise, if this block only has a single predecessor, and if that block
2320 // is a conditional branch, see if we can hoist any code from this block up
2321 // into our predecessor.
Chris Lattnerddb97a22010-12-13 05:10:48 +00002322 if (BasicBlock *OnlyPred = BB->getSinglePredecessor()) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002323 BranchInst *BI = dyn_cast<BranchInst>(OnlyPred->getTerminator());
2324 if (BI && BI->isConditional()) {
2325 // Get the other block.
2326 BasicBlock *OtherBB = BI->getSuccessor(BI->getSuccessor(0) == BB);
Chris Lattnerddb97a22010-12-13 05:10:48 +00002327 pred_iterator PI = pred_begin(OtherBB);
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002328 ++PI;
2329
2330 if (PI == pred_end(OtherBB)) {
2331 // We have a conditional branch to two blocks that are only reachable
2332 // from the condbr. We know that the condbr dominates the two blocks,
2333 // so see if there is any identical code in the "then" and "else"
2334 // blocks. If so, we can hoist it up to the branching block.
2335 Changed |= HoistThenElseCodeToIf(BI);
2336 } else {
2337 BasicBlock* OnlySucc = NULL;
2338 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB);
2339 SI != SE; ++SI) {
2340 if (!OnlySucc)
2341 OnlySucc = *SI;
2342 else if (*SI != OnlySucc) {
2343 OnlySucc = 0; // There are multiple distinct successors!
2344 break;
Evan Cheng4d09efd2008-06-07 08:52:29 +00002345 }
Chris Lattner76134372004-12-10 17:42:31 +00002346 }
Chris Lattner37dc9382004-11-30 00:29:14 +00002347
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002348 if (OnlySucc == OtherBB) {
2349 // If BB's only successor is the other successor of the predecessor,
2350 // i.e. a triangle, see if we can hoist any code from this block up
2351 // to the "if" block.
2352 Changed |= SpeculativelyExecuteBB(BI, BB);
Chris Lattner0d560082004-02-24 05:38:11 +00002353 }
2354 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002355 }
2356 }
2357
Chris Lattner694e37f2003-08-17 19:41:53 +00002358 return Changed;
Chris Lattner01d1ee32002-05-21 20:50:24 +00002359}
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00002360
2361/// SimplifyCFG - This function is used to do simplification of a CFG. For
2362/// example, it adjusts branches to branches to eliminate the extra hop, it
2363/// eliminates unreachable basic blocks, and does other "peephole" optimization
2364/// of the CFG. It returns true if a modification was made.
2365///
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00002366bool llvm::SimplifyCFG(BasicBlock *BB, const TargetData *TD) {
2367 return SimplifyCFGOpt(TD).run(BB);
2368}