blob: a1e7486ba50356c6460475a515f4613d59c6ed5b [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
Bill Wendling5049fa62009-01-19 23:43:56 +0000294/// GatherConstantSetEQs - Given a potentially 'or'd together collection of
295/// icmp_eq instructions that compare a value against a constant, return the
296/// value being compared, and stick the constant into the Values vector.
Chris Lattner28acc132010-12-13 03:30:12 +0000297static Value *
298GatherConstantSetEQs(Value *V, std::vector<ConstantInt*> &Values,
299 const TargetData *TD) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000300 Instruction *Inst = dyn_cast<Instruction>(V);
301 if (Inst == 0) return 0;
302
303 if (Inst->getOpcode() == Instruction::ICmp &&
304 cast<ICmpInst>(Inst)->getPredicate() == ICmpInst::ICMP_EQ) {
Chris Lattner28acc132010-12-13 03:30:12 +0000305 if (ConstantInt *C = GetConstantInt(Inst->getOperand(1), TD)) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000306 Values.push_back(C);
307 return Inst->getOperand(0);
Chris Lattner0d560082004-02-24 05:38:11 +0000308 }
Chris Lattner662269d2010-12-13 04:18:32 +0000309 return 0;
310 }
311
312 if (Inst->getOpcode() == Instruction::Or) {
Chris Lattner28acc132010-12-13 03:30:12 +0000313 if (Value *LHS = GatherConstantSetEQs(Inst->getOperand(0), Values, TD))
314 if (Value *RHS = GatherConstantSetEQs(Inst->getOperand(1), Values, TD))
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000315 if (LHS == RHS)
316 return LHS;
Anton Korobeynikov07e6e562008-02-20 11:26:25 +0000317 }
Chris Lattner0d560082004-02-24 05:38:11 +0000318 return 0;
319}
320
Bill Wendling5049fa62009-01-19 23:43:56 +0000321/// GatherConstantSetNEs - Given a potentially 'and'd together collection of
322/// setne instructions that compare a value against a constant, return the value
323/// being compared, and stick the constant into the Values vector.
Chris Lattner28acc132010-12-13 03:30:12 +0000324static Value *
325GatherConstantSetNEs(Value *V, std::vector<ConstantInt*> &Values,
326 const TargetData *TD) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000327 Instruction *Inst = dyn_cast<Instruction>(V);
328 if (Inst == 0) return 0;
329
330 if (Inst->getOpcode() == Instruction::ICmp &&
Chris Lattner662269d2010-12-13 04:18:32 +0000331 cast<ICmpInst>(Inst)->getPredicate() == ICmpInst::ICMP_NE) {
Chris Lattner28acc132010-12-13 03:30:12 +0000332 if (ConstantInt *C = GetConstantInt(Inst->getOperand(1), TD)) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000333 Values.push_back(C);
334 return Inst->getOperand(0);
Chris Lattner0d560082004-02-24 05:38:11 +0000335 }
Chris Lattner662269d2010-12-13 04:18:32 +0000336 return 0;
337 }
338
339 if (Inst->getOpcode() == Instruction::And) {
Chris Lattner28acc132010-12-13 03:30:12 +0000340 if (Value *LHS = GatherConstantSetNEs(Inst->getOperand(0), Values, TD))
341 if (Value *RHS = GatherConstantSetNEs(Inst->getOperand(1), Values, TD))
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000342 if (LHS == RHS)
343 return LHS;
Anton Korobeynikov07e6e562008-02-20 11:26:25 +0000344 }
Chris Lattner0d560082004-02-24 05:38:11 +0000345 return 0;
346}
347
Chris Lattner0d560082004-02-24 05:38:11 +0000348
Eli Friedman080efb82008-12-16 20:54:32 +0000349static void EraseTerminatorInstAndDCECond(TerminatorInst *TI) {
350 Instruction* Cond = 0;
351 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
352 Cond = dyn_cast<Instruction>(SI->getCondition());
353 } else if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
354 if (BI->isConditional())
355 Cond = dyn_cast<Instruction>(BI->getCondition());
Frits van Bommel7ac40c32010-12-05 18:29:03 +0000356 } else if (IndirectBrInst *IBI = dyn_cast<IndirectBrInst>(TI)) {
357 Cond = dyn_cast<Instruction>(IBI->getAddress());
Eli Friedman080efb82008-12-16 20:54:32 +0000358 }
359
360 TI->eraseFromParent();
361 if (Cond) RecursivelyDeleteTriviallyDeadInstructions(Cond);
362}
363
Chris Lattner9fd49552008-11-27 23:25:44 +0000364/// isValueEqualityComparison - Return true if the specified terminator checks
365/// to see if a value is equal to constant integer value.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000366Value *SimplifyCFGOpt::isValueEqualityComparison(TerminatorInst *TI) {
367 Value *CV = 0;
Chris Lattner4bebf082004-03-16 19:45:22 +0000368 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
369 // Do not permit merging of large switch instructions into their
370 // predecessors unless there is only one predecessor.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000371 if (SI->getNumSuccessors()*std::distance(pred_begin(SI->getParent()),
372 pred_end(SI->getParent())) <= 128)
373 CV = SI->getCondition();
374 } else if (BranchInst *BI = dyn_cast<BranchInst>(TI))
Chris Lattner542f1492004-02-28 21:28:10 +0000375 if (BI->isConditional() && BI->getCondition()->hasOneUse())
Reid Spencere4d87aa2006-12-23 06:05:41 +0000376 if (ICmpInst *ICI = dyn_cast<ICmpInst>(BI->getCondition()))
377 if ((ICI->getPredicate() == ICmpInst::ICMP_EQ ||
378 ICI->getPredicate() == ICmpInst::ICMP_NE) &&
Chris Lattner28acc132010-12-13 03:30:12 +0000379 GetConstantInt(ICI->getOperand(1), TD))
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000380 CV = ICI->getOperand(0);
381
382 // Unwrap any lossless ptrtoint cast.
383 if (TD && CV && CV->getType() == TD->getIntPtrType(CV->getContext()))
384 if (PtrToIntInst *PTII = dyn_cast<PtrToIntInst>(CV))
385 CV = PTII->getOperand(0);
386 return CV;
Chris Lattner542f1492004-02-28 21:28:10 +0000387}
388
Bill Wendling5049fa62009-01-19 23:43:56 +0000389/// GetValueEqualityComparisonCases - Given a value comparison instruction,
390/// decode all of the 'cases' that it represents and return the 'default' block.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000391BasicBlock *SimplifyCFGOpt::
Misha Brukmanfd939082005-04-21 23:48:37 +0000392GetValueEqualityComparisonCases(TerminatorInst *TI,
Chris Lattner542f1492004-02-28 21:28:10 +0000393 std::vector<std::pair<ConstantInt*,
394 BasicBlock*> > &Cases) {
395 if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
396 Cases.reserve(SI->getNumCases());
397 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
Chris Lattnerbe54dcc2005-02-26 18:33:28 +0000398 Cases.push_back(std::make_pair(SI->getCaseValue(i), SI->getSuccessor(i)));
Chris Lattner542f1492004-02-28 21:28:10 +0000399 return SI->getDefaultDest();
400 }
401
402 BranchInst *BI = cast<BranchInst>(TI);
Reid Spencere4d87aa2006-12-23 06:05:41 +0000403 ICmpInst *ICI = cast<ICmpInst>(BI->getCondition());
Chris Lattner28acc132010-12-13 03:30:12 +0000404 Cases.push_back(std::make_pair(GetConstantInt(ICI->getOperand(1), TD),
Reid Spencere4d87aa2006-12-23 06:05:41 +0000405 BI->getSuccessor(ICI->getPredicate() ==
406 ICmpInst::ICMP_NE)));
407 return BI->getSuccessor(ICI->getPredicate() == ICmpInst::ICMP_EQ);
Chris Lattner542f1492004-02-28 21:28:10 +0000408}
409
410
Bill Wendling5049fa62009-01-19 23:43:56 +0000411/// EliminateBlockCases - Given a vector of bb/value pairs, remove any entries
412/// in the list that match the specified block.
Misha Brukmanfd939082005-04-21 23:48:37 +0000413static void EliminateBlockCases(BasicBlock *BB,
Chris Lattner623369a2005-02-24 06:17:52 +0000414 std::vector<std::pair<ConstantInt*, BasicBlock*> > &Cases) {
415 for (unsigned i = 0, e = Cases.size(); i != e; ++i)
416 if (Cases[i].second == BB) {
417 Cases.erase(Cases.begin()+i);
418 --i; --e;
419 }
420}
421
Bill Wendling5049fa62009-01-19 23:43:56 +0000422/// ValuesOverlap - Return true if there are any keys in C1 that exist in C2 as
423/// well.
Chris Lattner623369a2005-02-24 06:17:52 +0000424static bool
425ValuesOverlap(std::vector<std::pair<ConstantInt*, BasicBlock*> > &C1,
426 std::vector<std::pair<ConstantInt*, BasicBlock*> > &C2) {
427 std::vector<std::pair<ConstantInt*, BasicBlock*> > *V1 = &C1, *V2 = &C2;
428
429 // Make V1 be smaller than V2.
430 if (V1->size() > V2->size())
431 std::swap(V1, V2);
432
433 if (V1->size() == 0) return false;
434 if (V1->size() == 1) {
435 // Just scan V2.
436 ConstantInt *TheVal = (*V1)[0].first;
437 for (unsigned i = 0, e = V2->size(); i != e; ++i)
438 if (TheVal == (*V2)[i].first)
439 return true;
440 }
441
442 // Otherwise, just sort both lists and compare element by element.
Chris Lattnerfca20f52010-12-13 03:24:30 +0000443 array_pod_sort(V1->begin(), V1->end());
444 array_pod_sort(V2->begin(), V2->end());
Chris Lattner623369a2005-02-24 06:17:52 +0000445 unsigned i1 = 0, i2 = 0, e1 = V1->size(), e2 = V2->size();
446 while (i1 != e1 && i2 != e2) {
447 if ((*V1)[i1].first == (*V2)[i2].first)
448 return true;
449 if ((*V1)[i1].first < (*V2)[i2].first)
450 ++i1;
451 else
452 ++i2;
453 }
454 return false;
455}
456
Bill Wendling5049fa62009-01-19 23:43:56 +0000457/// SimplifyEqualityComparisonWithOnlyPredecessor - If TI is known to be a
458/// terminator instruction and its block is known to only have a single
459/// predecessor block, check to see if that predecessor is also a value
460/// comparison with the same value, and if that comparison determines the
461/// outcome of this comparison. If so, simplify TI. This does a very limited
462/// form of jump threading.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000463bool SimplifyCFGOpt::
464SimplifyEqualityComparisonWithOnlyPredecessor(TerminatorInst *TI,
465 BasicBlock *Pred) {
Chris Lattner623369a2005-02-24 06:17:52 +0000466 Value *PredVal = isValueEqualityComparison(Pred->getTerminator());
467 if (!PredVal) return false; // Not a value comparison in predecessor.
468
469 Value *ThisVal = isValueEqualityComparison(TI);
470 assert(ThisVal && "This isn't a value comparison!!");
471 if (ThisVal != PredVal) return false; // Different predicates.
472
473 // Find out information about when control will move from Pred to TI's block.
474 std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
475 BasicBlock *PredDef = GetValueEqualityComparisonCases(Pred->getTerminator(),
476 PredCases);
477 EliminateBlockCases(PredDef, PredCases); // Remove default from cases.
Misha Brukmanfd939082005-04-21 23:48:37 +0000478
Chris Lattner623369a2005-02-24 06:17:52 +0000479 // Find information about how control leaves this block.
480 std::vector<std::pair<ConstantInt*, BasicBlock*> > ThisCases;
481 BasicBlock *ThisDef = GetValueEqualityComparisonCases(TI, ThisCases);
482 EliminateBlockCases(ThisDef, ThisCases); // Remove default from cases.
483
484 // If TI's block is the default block from Pred's comparison, potentially
485 // simplify TI based on this knowledge.
486 if (PredDef == TI->getParent()) {
487 // If we are here, we know that the value is none of those cases listed in
488 // PredCases. If there are any cases in ThisCases that are in PredCases, we
489 // can simplify TI.
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000490 if (!ValuesOverlap(PredCases, ThisCases))
491 return false;
492
493 if (isa<BranchInst>(TI)) {
494 // Okay, one of the successors of this condbr is dead. Convert it to a
495 // uncond br.
496 assert(ThisCases.size() == 1 && "Branch can only have one case!");
497 // Insert the new branch.
498 Instruction *NI = BranchInst::Create(ThisDef, TI);
499 (void) NI;
Chris Lattner623369a2005-02-24 06:17:52 +0000500
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000501 // Remove PHI node entries for the dead edge.
502 ThisCases[0].second->removePredecessor(TI->getParent());
Chris Lattner623369a2005-02-24 06:17:52 +0000503
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000504 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
505 << "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
Chris Lattner623369a2005-02-24 06:17:52 +0000506
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000507 EraseTerminatorInstAndDCECond(TI);
508 return true;
Chris Lattner623369a2005-02-24 06:17:52 +0000509 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000510
511 SwitchInst *SI = cast<SwitchInst>(TI);
512 // Okay, TI has cases that are statically dead, prune them away.
513 SmallPtrSet<Constant*, 16> DeadCases;
Chris Lattner623369a2005-02-24 06:17:52 +0000514 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000515 DeadCases.insert(PredCases[i].first);
Chris Lattner623369a2005-02-24 06:17:52 +0000516
David Greene89d6fd32010-01-05 01:26:52 +0000517 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000518 << "Through successor TI: " << *TI);
Chris Lattner623369a2005-02-24 06:17:52 +0000519
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000520 for (unsigned i = SI->getNumCases()-1; i != 0; --i)
521 if (DeadCases.count(SI->getCaseValue(i))) {
522 SI->getSuccessor(i)->removePredecessor(TI->getParent());
523 SI->removeCase(i);
524 }
525
526 DEBUG(dbgs() << "Leaving: " << *TI << "\n");
Chris Lattner623369a2005-02-24 06:17:52 +0000527 return true;
528 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000529
530 // Otherwise, TI's block must correspond to some matched value. Find out
531 // which value (or set of values) this is.
532 ConstantInt *TIV = 0;
533 BasicBlock *TIBB = TI->getParent();
534 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
535 if (PredCases[i].second == TIBB) {
536 if (TIV != 0)
537 return false; // Cannot handle multiple values coming to this block.
538 TIV = PredCases[i].first;
539 }
540 assert(TIV && "No edge from pred to succ?");
541
542 // Okay, we found the one constant that our value can be if we get into TI's
543 // BB. Find out which successor will unconditionally be branched to.
544 BasicBlock *TheRealDest = 0;
545 for (unsigned i = 0, e = ThisCases.size(); i != e; ++i)
546 if (ThisCases[i].first == TIV) {
547 TheRealDest = ThisCases[i].second;
548 break;
549 }
550
551 // If not handled by any explicit cases, it is handled by the default case.
552 if (TheRealDest == 0) TheRealDest = ThisDef;
553
554 // Remove PHI node entries for dead edges.
555 BasicBlock *CheckEdge = TheRealDest;
556 for (succ_iterator SI = succ_begin(TIBB), e = succ_end(TIBB); SI != e; ++SI)
557 if (*SI != CheckEdge)
558 (*SI)->removePredecessor(TIBB);
559 else
560 CheckEdge = 0;
561
562 // Insert the new branch.
563 Instruction *NI = BranchInst::Create(TheRealDest, TI);
564 (void) NI;
565
566 DEBUG(dbgs() << "Threading pred instr: " << *Pred->getTerminator()
567 << "Through successor TI: " << *TI << "Leaving: " << *NI << "\n");
568
569 EraseTerminatorInstAndDCECond(TI);
570 return true;
Chris Lattner623369a2005-02-24 06:17:52 +0000571}
572
Dale Johannesenc81f5442009-03-12 21:01:11 +0000573namespace {
574 /// ConstantIntOrdering - This class implements a stable ordering of constant
575 /// integers that does not depend on their address. This is important for
576 /// applications that sort ConstantInt's to ensure uniqueness.
577 struct ConstantIntOrdering {
578 bool operator()(const ConstantInt *LHS, const ConstantInt *RHS) const {
579 return LHS->getValue().ult(RHS->getValue());
580 }
581 };
582}
Dale Johannesena9537cf2009-03-12 01:00:26 +0000583
Chris Lattner6d4d21e2010-12-13 02:00:58 +0000584static int ConstantIntSortPredicate(const void *P1, const void *P2) {
585 const ConstantInt *LHS = *(const ConstantInt**)P1;
586 const ConstantInt *RHS = *(const ConstantInt**)P2;
587 return LHS->getValue().ult(RHS->getValue());
588}
589
Bill Wendling5049fa62009-01-19 23:43:56 +0000590/// FoldValueComparisonIntoPredecessors - The specified terminator is a value
591/// equality comparison instruction (either a switch or a branch on "X == c").
592/// See if any of the predecessors of the terminator block are value comparisons
593/// on the same value. If so, and if safe to do so, fold them together.
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000594bool SimplifyCFGOpt::FoldValueComparisonIntoPredecessors(TerminatorInst *TI) {
Chris Lattner542f1492004-02-28 21:28:10 +0000595 BasicBlock *BB = TI->getParent();
596 Value *CV = isValueEqualityComparison(TI); // CondVal
597 assert(CV && "Not a comparison?");
598 bool Changed = false;
599
Chris Lattner82442432008-02-18 07:42:56 +0000600 SmallVector<BasicBlock*, 16> Preds(pred_begin(BB), pred_end(BB));
Chris Lattner542f1492004-02-28 21:28:10 +0000601 while (!Preds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +0000602 BasicBlock *Pred = Preds.pop_back_val();
Misha Brukmanfd939082005-04-21 23:48:37 +0000603
Chris Lattner542f1492004-02-28 21:28:10 +0000604 // See if the predecessor is a comparison with the same value.
605 TerminatorInst *PTI = Pred->getTerminator();
606 Value *PCV = isValueEqualityComparison(PTI); // PredCondVal
607
608 if (PCV == CV && SafeToMergeTerminators(TI, PTI)) {
609 // Figure out which 'cases' to copy from SI to PSI.
610 std::vector<std::pair<ConstantInt*, BasicBlock*> > BBCases;
611 BasicBlock *BBDefault = GetValueEqualityComparisonCases(TI, BBCases);
612
613 std::vector<std::pair<ConstantInt*, BasicBlock*> > PredCases;
614 BasicBlock *PredDefault = GetValueEqualityComparisonCases(PTI, PredCases);
615
616 // Based on whether the default edge from PTI goes to BB or not, fill in
617 // PredCases and PredDefault with the new switch cases we would like to
618 // build.
Chris Lattner82442432008-02-18 07:42:56 +0000619 SmallVector<BasicBlock*, 8> NewSuccessors;
Chris Lattner542f1492004-02-28 21:28:10 +0000620
621 if (PredDefault == BB) {
622 // If this is the default destination from PTI, only the edges in TI
623 // that don't occur in PTI, or that branch to BB will be activated.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000624 std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
Chris Lattner542f1492004-02-28 21:28:10 +0000625 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
626 if (PredCases[i].second != BB)
627 PTIHandled.insert(PredCases[i].first);
628 else {
629 // The default destination is BB, we don't need explicit targets.
630 std::swap(PredCases[i], PredCases.back());
631 PredCases.pop_back();
632 --i; --e;
633 }
634
635 // Reconstruct the new switch statement we will be building.
636 if (PredDefault != BBDefault) {
637 PredDefault->removePredecessor(Pred);
638 PredDefault = BBDefault;
639 NewSuccessors.push_back(BBDefault);
640 }
641 for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
642 if (!PTIHandled.count(BBCases[i].first) &&
643 BBCases[i].second != BBDefault) {
644 PredCases.push_back(BBCases[i]);
645 NewSuccessors.push_back(BBCases[i].second);
646 }
647
648 } else {
649 // If this is not the default destination from PSI, only the edges
650 // in SI that occur in PSI with a destination of BB will be
651 // activated.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000652 std::set<ConstantInt*, ConstantIntOrdering> PTIHandled;
Chris Lattner542f1492004-02-28 21:28:10 +0000653 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
654 if (PredCases[i].second == BB) {
655 PTIHandled.insert(PredCases[i].first);
656 std::swap(PredCases[i], PredCases.back());
657 PredCases.pop_back();
658 --i; --e;
659 }
660
661 // Okay, now we know which constants were sent to BB from the
662 // predecessor. Figure out where they will all go now.
663 for (unsigned i = 0, e = BBCases.size(); i != e; ++i)
664 if (PTIHandled.count(BBCases[i].first)) {
665 // If this is one we are capable of getting...
666 PredCases.push_back(BBCases[i]);
667 NewSuccessors.push_back(BBCases[i].second);
668 PTIHandled.erase(BBCases[i].first);// This constant is taken care of
669 }
670
671 // If there are any constants vectored to BB that TI doesn't handle,
672 // they must go to the default destination of TI.
Dale Johannesenc81f5442009-03-12 21:01:11 +0000673 for (std::set<ConstantInt*, ConstantIntOrdering>::iterator I =
674 PTIHandled.begin(),
Chris Lattner542f1492004-02-28 21:28:10 +0000675 E = PTIHandled.end(); I != E; ++I) {
676 PredCases.push_back(std::make_pair(*I, BBDefault));
677 NewSuccessors.push_back(BBDefault);
678 }
679 }
680
681 // Okay, at this point, we know which new successor Pred will get. Make
682 // sure we update the number of entries in the PHI nodes for these
683 // successors.
684 for (unsigned i = 0, e = NewSuccessors.size(); i != e; ++i)
685 AddPredecessorToBlock(NewSuccessors[i], Pred, BB);
686
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000687 // Convert pointer to int before we switch.
Duncan Sands1df98592010-02-16 11:11:14 +0000688 if (CV->getType()->isPointerTy()) {
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +0000689 assert(TD && "Cannot switch on pointer without TargetData");
690 CV = new PtrToIntInst(CV, TD->getIntPtrType(CV->getContext()),
691 "magicptr", PTI);
692 }
693
Chris Lattner542f1492004-02-28 21:28:10 +0000694 // Now that the successors are updated, create the new Switch instruction.
Gabor Greifb1dbcd82008-05-15 10:04:30 +0000695 SwitchInst *NewSI = SwitchInst::Create(CV, PredDefault,
696 PredCases.size(), PTI);
Chris Lattner542f1492004-02-28 21:28:10 +0000697 for (unsigned i = 0, e = PredCases.size(); i != e; ++i)
698 NewSI->addCase(PredCases[i].first, PredCases[i].second);
Chris Lattner13b2f762005-01-01 16:02:12 +0000699
Eli Friedman080efb82008-12-16 20:54:32 +0000700 EraseTerminatorInstAndDCECond(PTI);
Chris Lattner13b2f762005-01-01 16:02:12 +0000701
Chris Lattner542f1492004-02-28 21:28:10 +0000702 // Okay, last check. If BB is still a successor of PSI, then we must
703 // have an infinite loop case. If so, add an infinitely looping block
704 // to handle the case to preserve the behavior of the code.
705 BasicBlock *InfLoopBlock = 0;
706 for (unsigned i = 0, e = NewSI->getNumSuccessors(); i != e; ++i)
707 if (NewSI->getSuccessor(i) == BB) {
708 if (InfLoopBlock == 0) {
Chris Lattner093a4382008-07-13 22:23:11 +0000709 // Insert it at the end of the function, because it's either code,
Chris Lattner542f1492004-02-28 21:28:10 +0000710 // or it won't matter if it's hot. :)
Owen Anderson1d0be152009-08-13 21:58:54 +0000711 InfLoopBlock = BasicBlock::Create(BB->getContext(),
712 "infloop", BB->getParent());
Gabor Greif051a9502008-04-06 20:25:17 +0000713 BranchInst::Create(InfLoopBlock, InfLoopBlock);
Chris Lattner542f1492004-02-28 21:28:10 +0000714 }
715 NewSI->setSuccessor(i, InfLoopBlock);
716 }
Misha Brukmanfd939082005-04-21 23:48:37 +0000717
Chris Lattner542f1492004-02-28 21:28:10 +0000718 Changed = true;
719 }
720 }
721 return Changed;
722}
723
Dale Johannesenc1f10402009-06-15 20:59:27 +0000724// isSafeToHoistInvoke - If we would need to insert a select that uses the
725// value of this invoke (comments in HoistThenElseCodeToIf explain why we
726// would need to do this), we can't hoist the invoke, as there is nowhere
727// to put the select in this case.
728static bool isSafeToHoistInvoke(BasicBlock *BB1, BasicBlock *BB2,
729 Instruction *I1, Instruction *I2) {
730 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
731 PHINode *PN;
732 for (BasicBlock::iterator BBI = SI->begin();
733 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
734 Value *BB1V = PN->getIncomingValueForBlock(BB1);
735 Value *BB2V = PN->getIncomingValueForBlock(BB2);
736 if (BB1V != BB2V && (BB1V==I1 || BB2V==I2)) {
737 return false;
738 }
739 }
740 }
741 return true;
742}
743
Chris Lattner6306d072005-08-03 17:59:45 +0000744/// HoistThenElseCodeToIf - Given a conditional branch that goes to BB1 and
Chris Lattner37dc9382004-11-30 00:29:14 +0000745/// BB2, hoist any common code in the two blocks up into the branch block. The
746/// caller of this function guarantees that BI's block dominates BB1 and BB2.
747static bool HoistThenElseCodeToIf(BranchInst *BI) {
748 // This does very trivial matching, with limited scanning, to find identical
749 // instructions in the two blocks. In particular, we don't want to get into
750 // O(M*N) situations here where M and N are the sizes of BB1 and BB2. As
751 // such, we currently just scan for obviously identical instructions in an
752 // identical order.
753 BasicBlock *BB1 = BI->getSuccessor(0); // The true destination.
754 BasicBlock *BB2 = BI->getSuccessor(1); // The false destination
755
Devang Patel65085cf2009-02-04 00:03:08 +0000756 BasicBlock::iterator BB1_Itr = BB1->begin();
757 BasicBlock::iterator BB2_Itr = BB2->begin();
758
759 Instruction *I1 = BB1_Itr++, *I2 = BB2_Itr++;
760 while (isa<DbgInfoIntrinsic>(I1))
761 I1 = BB1_Itr++;
762 while (isa<DbgInfoIntrinsic>(I2))
763 I2 = BB2_Itr++;
Dale Johannesenc1f10402009-06-15 20:59:27 +0000764 if (I1->getOpcode() != I2->getOpcode() || isa<PHINode>(I1) ||
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000765 !I1->isIdenticalToWhenDefined(I2) ||
Dale Johannesenc1f10402009-06-15 20:59:27 +0000766 (isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2)))
Chris Lattner37dc9382004-11-30 00:29:14 +0000767 return false;
768
769 // If we get here, we can hoist at least one instruction.
770 BasicBlock *BIParent = BI->getParent();
Chris Lattner37dc9382004-11-30 00:29:14 +0000771
772 do {
773 // If we are hoisting the terminator instruction, don't move one (making a
774 // broken BB), instead clone it, and remove BI.
775 if (isa<TerminatorInst>(I1))
776 goto HoistTerminator;
Misha Brukmanfd939082005-04-21 23:48:37 +0000777
Chris Lattner37dc9382004-11-30 00:29:14 +0000778 // For a normal instruction, we just move one to right before the branch,
779 // then replace all uses of the other with the first. Finally, we remove
780 // the now redundant second instruction.
781 BIParent->getInstList().splice(BI, BB1->getInstList(), I1);
782 if (!I2->use_empty())
783 I2->replaceAllUsesWith(I1);
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000784 I1->intersectOptionalDataWith(I2);
Chris Lattner37dc9382004-11-30 00:29:14 +0000785 BB2->getInstList().erase(I2);
Misha Brukmanfd939082005-04-21 23:48:37 +0000786
Devang Patel65085cf2009-02-04 00:03:08 +0000787 I1 = BB1_Itr++;
788 while (isa<DbgInfoIntrinsic>(I1))
789 I1 = BB1_Itr++;
790 I2 = BB2_Itr++;
791 while (isa<DbgInfoIntrinsic>(I2))
792 I2 = BB2_Itr++;
Dan Gohman58cfa3b2009-08-25 22:11:20 +0000793 } while (I1->getOpcode() == I2->getOpcode() &&
794 I1->isIdenticalToWhenDefined(I2));
Chris Lattner37dc9382004-11-30 00:29:14 +0000795
796 return true;
797
798HoistTerminator:
Dale Johannesenc1f10402009-06-15 20:59:27 +0000799 // It may not be possible to hoist an invoke.
800 if (isa<InvokeInst>(I1) && !isSafeToHoistInvoke(BB1, BB2, I1, I2))
801 return true;
802
Chris Lattner37dc9382004-11-30 00:29:14 +0000803 // Okay, it is safe to hoist the terminator.
Nick Lewycky67760642009-09-27 07:38:41 +0000804 Instruction *NT = I1->clone();
Chris Lattner37dc9382004-11-30 00:29:14 +0000805 BIParent->getInstList().insert(BI, NT);
Benjamin Kramerf0127052010-01-05 13:12:22 +0000806 if (!NT->getType()->isVoidTy()) {
Chris Lattner37dc9382004-11-30 00:29:14 +0000807 I1->replaceAllUsesWith(NT);
808 I2->replaceAllUsesWith(NT);
Chris Lattner86cc4232007-02-11 01:37:51 +0000809 NT->takeName(I1);
Chris Lattner37dc9382004-11-30 00:29:14 +0000810 }
811
812 // Hoisting one of the terminators from our successor is a great thing.
813 // Unfortunately, the successors of the if/else blocks may have PHI nodes in
814 // them. If they do, all PHI entries for BB1/BB2 must agree for all PHI
815 // nodes, so we insert select instruction to compute the final result.
816 std::map<std::pair<Value*,Value*>, SelectInst*> InsertedSelects;
817 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI) {
818 PHINode *PN;
819 for (BasicBlock::iterator BBI = SI->begin();
Chris Lattner0f535c62004-11-30 07:47:34 +0000820 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
Chris Lattner37dc9382004-11-30 00:29:14 +0000821 Value *BB1V = PN->getIncomingValueForBlock(BB1);
822 Value *BB2V = PN->getIncomingValueForBlock(BB2);
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000823 if (BB1V == BB2V) continue;
824
825 // These values do not agree. Insert a select instruction before NT
826 // that determines the right value.
827 SelectInst *&SI = InsertedSelects[std::make_pair(BB1V, BB2V)];
828 if (SI == 0)
829 SI = SelectInst::Create(BI->getCondition(), BB1V, BB2V,
830 BB1V->getName()+"."+BB2V->getName(), NT);
831 // Make the PHI node use the select for all incoming values for BB1/BB2
832 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
833 if (PN->getIncomingBlock(i) == BB1 || PN->getIncomingBlock(i) == BB2)
834 PN->setIncomingValue(i, SI);
Chris Lattner37dc9382004-11-30 00:29:14 +0000835 }
836 }
837
838 // Update any PHI nodes in our new successors.
839 for (succ_iterator SI = succ_begin(BB1), E = succ_end(BB1); SI != E; ++SI)
840 AddPredecessorToBlock(*SI, BIParent, BB1);
Misha Brukmanfd939082005-04-21 23:48:37 +0000841
Eli Friedman080efb82008-12-16 20:54:32 +0000842 EraseTerminatorInstAndDCECond(BI);
Chris Lattner37dc9382004-11-30 00:29:14 +0000843 return true;
844}
845
Evan Cheng4d09efd2008-06-07 08:52:29 +0000846/// SpeculativelyExecuteBB - Given a conditional branch that goes to BB1
847/// and an BB2 and the only successor of BB1 is BB2, hoist simple code
848/// (for now, restricted to a single instruction that's side effect free) from
849/// the BB1 into the branch block to speculatively execute it.
850static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *BB1) {
851 // Only speculatively execution a single instruction (not counting the
852 // terminator) for now.
Devang Patel06b1e672009-03-06 06:00:17 +0000853 Instruction *HInst = NULL;
854 Instruction *Term = BB1->getTerminator();
855 for (BasicBlock::iterator BBI = BB1->begin(), BBE = BB1->end();
856 BBI != BBE; ++BBI) {
857 Instruction *I = BBI;
858 // Skip debug info.
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000859 if (isa<DbgInfoIntrinsic>(I)) continue;
860 if (I == Term) break;
Devang Patel06b1e672009-03-06 06:00:17 +0000861
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000862 if (HInst)
Devang Patel06b1e672009-03-06 06:00:17 +0000863 return false;
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000864 HInst = I;
Devang Patel06b1e672009-03-06 06:00:17 +0000865 }
866 if (!HInst)
867 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000868
Evan Cheng797d9512008-06-11 19:18:20 +0000869 // Be conservative for now. FP select instruction can often be expensive.
870 Value *BrCond = BI->getCondition();
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000871 if (isa<FCmpInst>(BrCond))
Evan Cheng797d9512008-06-11 19:18:20 +0000872 return false;
873
Evan Cheng4d09efd2008-06-07 08:52:29 +0000874 // If BB1 is actually on the false edge of the conditional branch, remember
875 // to swap the select operands later.
876 bool Invert = false;
877 if (BB1 != BI->getSuccessor(0)) {
878 assert(BB1 == BI->getSuccessor(1) && "No edge from 'if' block?");
879 Invert = true;
880 }
881
882 // Turn
883 // BB:
884 // %t1 = icmp
885 // br i1 %t1, label %BB1, label %BB2
886 // BB1:
887 // %t3 = add %t2, c
888 // br label BB2
889 // BB2:
890 // =>
891 // BB:
892 // %t1 = icmp
893 // %t4 = add %t2, c
894 // %t3 = select i1 %t1, %t2, %t3
Devang Patel06b1e672009-03-06 06:00:17 +0000895 switch (HInst->getOpcode()) {
Evan Cheng4d09efd2008-06-07 08:52:29 +0000896 default: return false; // Not safe / profitable to hoist.
897 case Instruction::Add:
898 case Instruction::Sub:
Dan Gohmanae3a0be2009-06-04 22:49:04 +0000899 // Not worth doing for vector ops.
Duncan Sands1df98592010-02-16 11:11:14 +0000900 if (HInst->getType()->isVectorTy())
Chris Lattner9dd3b612009-01-18 23:22:07 +0000901 return false;
902 break;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000903 case Instruction::And:
904 case Instruction::Or:
905 case Instruction::Xor:
906 case Instruction::Shl:
907 case Instruction::LShr:
908 case Instruction::AShr:
Chris Lattner9dd3b612009-01-18 23:22:07 +0000909 // Don't mess with vector operations.
Duncan Sands1df98592010-02-16 11:11:14 +0000910 if (HInst->getType()->isVectorTy())
Evan Chenge5334ea2008-06-25 07:50:12 +0000911 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000912 break; // These are all cheap and non-trapping instructions.
913 }
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000914
915 // If the instruction is obviously dead, don't try to predicate it.
Devang Patel06b1e672009-03-06 06:00:17 +0000916 if (HInst->use_empty()) {
917 HInst->eraseFromParent();
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000918 return true;
919 }
Evan Cheng4d09efd2008-06-07 08:52:29 +0000920
921 // Can we speculatively execute the instruction? And what is the value
922 // if the condition is false? Consider the phi uses, if the incoming value
923 // from the "if" block are all the same V, then V is the value of the
924 // select if the condition is false.
925 BasicBlock *BIParent = BI->getParent();
926 SmallVector<PHINode*, 4> PHIUses;
927 Value *FalseV = NULL;
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000928
929 BasicBlock *BB2 = BB1->getTerminator()->getSuccessor(0);
Devang Patel06b1e672009-03-06 06:00:17 +0000930 for (Value::use_iterator UI = HInst->use_begin(), E = HInst->use_end();
Evan Cheng4d09efd2008-06-07 08:52:29 +0000931 UI != E; ++UI) {
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000932 // Ignore any user that is not a PHI node in BB2. These can only occur in
933 // unreachable blocks, because they would not be dominated by the instr.
Gabor Greif20361b92010-07-22 11:43:44 +0000934 PHINode *PN = dyn_cast<PHINode>(*UI);
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000935 if (!PN || PN->getParent() != BB2)
936 return false;
Evan Cheng4d09efd2008-06-07 08:52:29 +0000937 PHIUses.push_back(PN);
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000938
Evan Cheng4d09efd2008-06-07 08:52:29 +0000939 Value *PHIV = PN->getIncomingValueForBlock(BIParent);
940 if (!FalseV)
941 FalseV = PHIV;
942 else if (FalseV != PHIV)
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000943 return false; // Inconsistent value when condition is false.
Evan Cheng4d09efd2008-06-07 08:52:29 +0000944 }
Chris Lattner6fe73bb2009-01-19 00:36:37 +0000945
946 assert(FalseV && "Must have at least one user, and it must be a PHI");
Evan Cheng4d09efd2008-06-07 08:52:29 +0000947
Evan Cheng502a4f52008-06-12 21:15:59 +0000948 // Do not hoist the instruction if any of its operands are defined but not
949 // used in this BB. The transformation will prevent the operand from
950 // being sunk into the use block.
Devang Patel06b1e672009-03-06 06:00:17 +0000951 for (User::op_iterator i = HInst->op_begin(), e = HInst->op_end();
952 i != e; ++i) {
Evan Cheng502a4f52008-06-12 21:15:59 +0000953 Instruction *OpI = dyn_cast<Instruction>(*i);
954 if (OpI && OpI->getParent() == BIParent &&
955 !OpI->isUsedInBasicBlock(BIParent))
956 return false;
957 }
958
Devang Patel3d0a9a32008-09-18 22:50:42 +0000959 // If we get here, we can hoist the instruction. Try to place it
Dale Johannesen990afed2009-03-13 01:05:24 +0000960 // before the icmp instruction preceding the conditional branch.
Devang Patel3d0a9a32008-09-18 22:50:42 +0000961 BasicBlock::iterator InsertPos = BI;
Dale Johannesen990afed2009-03-13 01:05:24 +0000962 if (InsertPos != BIParent->begin())
963 --InsertPos;
964 // Skip debug info between condition and branch.
965 while (InsertPos != BIParent->begin() && isa<DbgInfoIntrinsic>(InsertPos))
Devang Patel3d0a9a32008-09-18 22:50:42 +0000966 --InsertPos;
Devang Patel20da1f02008-10-03 18:57:37 +0000967 if (InsertPos == BrCond && !isa<PHINode>(BrCond)) {
Devang Patel3d0a9a32008-09-18 22:50:42 +0000968 SmallPtrSet<Instruction *, 4> BB1Insns;
969 for(BasicBlock::iterator BB1I = BB1->begin(), BB1E = BB1->end();
970 BB1I != BB1E; ++BB1I)
971 BB1Insns.insert(BB1I);
972 for(Value::use_iterator UI = BrCond->use_begin(), UE = BrCond->use_end();
973 UI != UE; ++UI) {
974 Instruction *Use = cast<Instruction>(*UI);
Chris Lattner9a2b72a2010-12-13 01:47:07 +0000975 if (!BB1Insns.count(Use)) continue;
976
977 // If BrCond uses the instruction that place it just before
978 // branch instruction.
979 InsertPos = BI;
980 break;
Devang Patel3d0a9a32008-09-18 22:50:42 +0000981 }
982 } else
983 InsertPos = BI;
Devang Patel06b1e672009-03-06 06:00:17 +0000984 BIParent->getInstList().splice(InsertPos, BB1->getInstList(), HInst);
Evan Cheng4d09efd2008-06-07 08:52:29 +0000985
986 // Create a select whose true value is the speculatively executed value and
987 // false value is the previously determined FalseV.
988 SelectInst *SI;
989 if (Invert)
Devang Patel06b1e672009-03-06 06:00:17 +0000990 SI = SelectInst::Create(BrCond, FalseV, HInst,
991 FalseV->getName() + "." + HInst->getName(), BI);
Evan Cheng4d09efd2008-06-07 08:52:29 +0000992 else
Devang Patel06b1e672009-03-06 06:00:17 +0000993 SI = SelectInst::Create(BrCond, HInst, FalseV,
994 HInst->getName() + "." + FalseV->getName(), BI);
Evan Cheng4d09efd2008-06-07 08:52:29 +0000995
996 // Make the PHI node use the select for all incoming values for "then" and
997 // "if" blocks.
998 for (unsigned i = 0, e = PHIUses.size(); i != e; ++i) {
999 PHINode *PN = PHIUses[i];
1000 for (unsigned j = 0, ee = PN->getNumIncomingValues(); j != ee; ++j)
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001001 if (PN->getIncomingBlock(j) == BB1 || PN->getIncomingBlock(j) == BIParent)
Evan Cheng4d09efd2008-06-07 08:52:29 +00001002 PN->setIncomingValue(j, SI);
1003 }
1004
Evan Cheng502a4f52008-06-12 21:15:59 +00001005 ++NumSpeculations;
Evan Cheng4d09efd2008-06-07 08:52:29 +00001006 return true;
1007}
1008
Chris Lattner2e42e362005-09-20 00:43:16 +00001009/// BlockIsSimpleEnoughToThreadThrough - Return true if we can thread a branch
1010/// across this block.
1011static bool BlockIsSimpleEnoughToThreadThrough(BasicBlock *BB) {
1012 BranchInst *BI = cast<BranchInst>(BB->getTerminator());
Chris Lattnere9487f02005-09-20 01:48:40 +00001013 unsigned Size = 0;
1014
Devang Patel9200c892009-03-10 18:00:05 +00001015 for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
Dale Johannesen8483e542009-03-12 23:18:09 +00001016 if (isa<DbgInfoIntrinsic>(BBI))
1017 continue;
Chris Lattnere9487f02005-09-20 01:48:40 +00001018 if (Size > 10) return false; // Don't clone large BB's.
Dale Johannesen8483e542009-03-12 23:18:09 +00001019 ++Size;
Chris Lattner2e42e362005-09-20 00:43:16 +00001020
Dale Johannesen8483e542009-03-12 23:18:09 +00001021 // We can only support instructions that do not define values that are
Chris Lattnere9487f02005-09-20 01:48:40 +00001022 // live outside of the current basic block.
1023 for (Value::use_iterator UI = BBI->use_begin(), E = BBI->use_end();
1024 UI != E; ++UI) {
1025 Instruction *U = cast<Instruction>(*UI);
1026 if (U->getParent() != BB || isa<PHINode>(U)) return false;
1027 }
Chris Lattner2e42e362005-09-20 00:43:16 +00001028
1029 // Looks ok, continue checking.
1030 }
Chris Lattnere9487f02005-09-20 01:48:40 +00001031
Chris Lattner2e42e362005-09-20 00:43:16 +00001032 return true;
1033}
1034
Chris Lattnereaba3a12005-09-19 23:49:37 +00001035/// FoldCondBranchOnPHI - If we have a conditional branch on a PHI node value
1036/// that is defined in the same block as the branch and if any PHI entries are
1037/// constants, thread edges corresponding to that entry to be branches to their
1038/// ultimate destination.
1039static bool FoldCondBranchOnPHI(BranchInst *BI) {
1040 BasicBlock *BB = BI->getParent();
1041 PHINode *PN = dyn_cast<PHINode>(BI->getCondition());
Chris Lattner9c88d982005-09-19 23:57:04 +00001042 // NOTE: we currently cannot transform this case if the PHI node is used
1043 // outside of the block.
Chris Lattner2e42e362005-09-20 00:43:16 +00001044 if (!PN || PN->getParent() != BB || !PN->hasOneUse())
1045 return false;
Chris Lattnereaba3a12005-09-19 23:49:37 +00001046
1047 // Degenerate case of a single entry PHI.
1048 if (PN->getNumIncomingValues() == 1) {
Chris Lattner29874e02008-12-03 19:44:02 +00001049 FoldSingleEntryPHINodes(PN->getParent());
Chris Lattnereaba3a12005-09-19 23:49:37 +00001050 return true;
1051 }
1052
1053 // Now we know that this block has multiple preds and two succs.
Chris Lattner2e42e362005-09-20 00:43:16 +00001054 if (!BlockIsSimpleEnoughToThreadThrough(BB)) return false;
Chris Lattnereaba3a12005-09-19 23:49:37 +00001055
1056 // Okay, this is a simple enough basic block. See if any phi values are
1057 // constants.
Zhou Sheng6b6b6ef2007-01-11 12:24:14 +00001058 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001059 ConstantInt *CB = dyn_cast<ConstantInt>(PN->getIncomingValue(i));
1060 if (CB == 0 || !CB->getType()->isIntegerTy(1)) continue;
1061
1062 // Okay, we now know that all edges from PredBB should be revectored to
1063 // branch to RealDest.
1064 BasicBlock *PredBB = PN->getIncomingBlock(i);
1065 BasicBlock *RealDest = BI->getSuccessor(!CB->getZExtValue());
1066
1067 if (RealDest == BB) continue; // Skip self loops.
1068
1069 // The dest block might have PHI nodes, other predecessors and other
1070 // difficult cases. Instead of being smart about this, just insert a new
1071 // block that jumps to the destination block, effectively splitting
1072 // the edge we are about to create.
1073 BasicBlock *EdgeBB = BasicBlock::Create(BB->getContext(),
1074 RealDest->getName()+".critedge",
1075 RealDest->getParent(), RealDest);
1076 BranchInst::Create(RealDest, EdgeBB);
1077 PHINode *PN;
1078 for (BasicBlock::iterator BBI = RealDest->begin();
1079 (PN = dyn_cast<PHINode>(BBI)); ++BBI) {
1080 Value *V = PN->getIncomingValueForBlock(BB);
1081 PN->addIncoming(V, EdgeBB);
Chris Lattnereaba3a12005-09-19 23:49:37 +00001082 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001083
1084 // BB may have instructions that are being threaded over. Clone these
1085 // instructions into EdgeBB. We know that there will be no uses of the
1086 // cloned instructions outside of EdgeBB.
1087 BasicBlock::iterator InsertPt = EdgeBB->begin();
1088 DenseMap<Value*, Value*> TranslateMap; // Track translated values.
1089 for (BasicBlock::iterator BBI = BB->begin(); &*BBI != BI; ++BBI) {
1090 if (PHINode *PN = dyn_cast<PHINode>(BBI)) {
1091 TranslateMap[PN] = PN->getIncomingValueForBlock(PredBB);
1092 continue;
1093 }
1094 // Clone the instruction.
1095 Instruction *N = BBI->clone();
1096 if (BBI->hasName()) N->setName(BBI->getName()+".c");
1097
1098 // Update operands due to translation.
1099 for (User::op_iterator i = N->op_begin(), e = N->op_end();
1100 i != e; ++i) {
1101 DenseMap<Value*, Value*>::iterator PI = TranslateMap.find(*i);
1102 if (PI != TranslateMap.end())
1103 *i = PI->second;
1104 }
1105
1106 // Check for trivial simplification.
1107 if (Constant *C = ConstantFoldInstruction(N)) {
1108 TranslateMap[BBI] = C;
1109 delete N; // Constant folded away, don't need actual inst
1110 } else {
1111 // Insert the new instruction into its new home.
1112 EdgeBB->getInstList().insert(InsertPt, N);
1113 if (!BBI->use_empty())
1114 TranslateMap[BBI] = N;
1115 }
1116 }
1117
1118 // Loop over all of the edges from PredBB to BB, changing them to branch
1119 // to EdgeBB instead.
1120 TerminatorInst *PredBBTI = PredBB->getTerminator();
1121 for (unsigned i = 0, e = PredBBTI->getNumSuccessors(); i != e; ++i)
1122 if (PredBBTI->getSuccessor(i) == BB) {
1123 BB->removePredecessor(PredBB);
1124 PredBBTI->setSuccessor(i, EdgeBB);
1125 }
1126
1127 // Recurse, simplifying any other constants.
1128 return FoldCondBranchOnPHI(BI) | true;
Zhou Sheng6b6b6ef2007-01-11 12:24:14 +00001129 }
Chris Lattnereaba3a12005-09-19 23:49:37 +00001130
1131 return false;
1132}
1133
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001134/// FoldTwoEntryPHINode - Given a BB that starts with the specified two-entry
1135/// PHI node, see if we can eliminate it.
1136static bool FoldTwoEntryPHINode(PHINode *PN) {
1137 // Ok, this is a two entry PHI node. Check to see if this is a simple "if
1138 // statement", which has a very simple dominance structure. Basically, we
1139 // are trying to find the condition that is being branched on, which
1140 // subsequently causes this merge to happen. We really want control
1141 // dependence information for this check, but simplifycfg can't keep it up
1142 // to date, and this catches most of the cases we care about anyway.
1143 //
1144 BasicBlock *BB = PN->getParent();
1145 BasicBlock *IfTrue, *IfFalse;
1146 Value *IfCond = GetIfCondition(BB, IfTrue, IfFalse);
1147 if (!IfCond) return false;
1148
Chris Lattner822a8792006-11-18 19:19:36 +00001149 // Okay, we found that we can merge this two-entry phi node into a select.
1150 // Doing so would require us to fold *all* two entry phi nodes in this block.
1151 // At some point this becomes non-profitable (particularly if the target
1152 // doesn't support cmov's). Only do this transformation if there are two or
1153 // fewer PHI nodes in this block.
1154 unsigned NumPhis = 0;
1155 for (BasicBlock::iterator I = BB->begin(); isa<PHINode>(I); ++NumPhis, ++I)
1156 if (NumPhis > 2)
1157 return false;
1158
David Greene89d6fd32010-01-05 01:26:52 +00001159 DEBUG(dbgs() << "FOUND IF CONDITION! " << *IfCond << " T: "
Daniel Dunbarce63ffb2009-07-25 00:23:56 +00001160 << IfTrue->getName() << " F: " << IfFalse->getName() << "\n");
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001161
1162 // Loop over the PHI's seeing if we can promote them all to select
1163 // instructions. While we are at it, keep track of the instructions
1164 // that need to be moved to the dominating block.
1165 std::set<Instruction*> AggressiveInsts;
1166
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001167 BasicBlock::iterator AfterPHIIt = BB->begin();
1168 while (isa<PHINode>(AfterPHIIt)) {
1169 PHINode *PN = cast<PHINode>(AfterPHIIt++);
1170 if (PN->getIncomingValue(0) == PN->getIncomingValue(1)) {
1171 if (PN->getIncomingValue(0) != PN)
1172 PN->replaceAllUsesWith(PN->getIncomingValue(0));
1173 else
Owen Anderson9e9a0d52009-07-30 23:03:37 +00001174 PN->replaceAllUsesWith(UndefValue::get(PN->getType()));
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001175 } else if (!DominatesMergePoint(PN->getIncomingValue(0), BB,
1176 &AggressiveInsts) ||
1177 !DominatesMergePoint(PN->getIncomingValue(1), BB,
1178 &AggressiveInsts)) {
Chris Lattner055dc102005-09-23 07:23:18 +00001179 return false;
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001180 }
1181 }
1182
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001183 // If we all PHI nodes are promotable, check to make sure that all
1184 // instructions in the predecessor blocks can be promoted as well. If
1185 // not, we won't be able to get rid of the control flow, so it's not
1186 // worth promoting to select instructions.
1187 BasicBlock *DomBlock = 0, *IfBlock1 = 0, *IfBlock2 = 0;
1188 PN = cast<PHINode>(BB->begin());
1189 BasicBlock *Pred = PN->getIncomingBlock(0);
1190 if (cast<BranchInst>(Pred->getTerminator())->isUnconditional()) {
1191 IfBlock1 = Pred;
1192 DomBlock = *pred_begin(Pred);
1193 for (BasicBlock::iterator I = Pred->begin();
1194 !isa<TerminatorInst>(I); ++I)
Devang Patel383d7ed2009-02-03 22:12:02 +00001195 if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001196 // This is not an aggressive instruction that we can promote.
1197 // Because of this, we won't be able to get rid of the control
1198 // flow, so the xform is not worth it.
1199 return false;
1200 }
1201 }
1202
1203 Pred = PN->getIncomingBlock(1);
1204 if (cast<BranchInst>(Pred->getTerminator())->isUnconditional()) {
1205 IfBlock2 = Pred;
1206 DomBlock = *pred_begin(Pred);
1207 for (BasicBlock::iterator I = Pred->begin();
1208 !isa<TerminatorInst>(I); ++I)
Devang Patel383d7ed2009-02-03 22:12:02 +00001209 if (!AggressiveInsts.count(I) && !isa<DbgInfoIntrinsic>(I)) {
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001210 // This is not an aggressive instruction that we can promote.
1211 // Because of this, we won't be able to get rid of the control
1212 // flow, so the xform is not worth it.
1213 return false;
1214 }
1215 }
1216
1217 // If we can still promote the PHI nodes after this gauntlet of tests,
1218 // do all of the PHI's now.
1219
1220 // Move all 'aggressive' instructions, which are defined in the
1221 // conditional parts of the if's up to the dominating block.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001222 if (IfBlock1)
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001223 DomBlock->getInstList().splice(DomBlock->getTerminator(),
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001224 IfBlock1->getInstList(), IfBlock1->begin(),
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001225 IfBlock1->getTerminator());
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001226 if (IfBlock2)
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001227 DomBlock->getInstList().splice(DomBlock->getTerminator(),
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001228 IfBlock2->getInstList(), IfBlock2->begin(),
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001229 IfBlock2->getTerminator());
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001230
1231 while (PHINode *PN = dyn_cast<PHINode>(BB->begin())) {
1232 // Change the PHI node into a select instruction.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001233 Value *TrueVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfFalse);
1234 Value *FalseVal = PN->getIncomingValue(PN->getIncomingBlock(0) == IfTrue);
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001235
Gabor Greif051a9502008-04-06 20:25:17 +00001236 Value *NV = SelectInst::Create(IfCond, TrueVal, FalseVal, "", AfterPHIIt);
Chris Lattner86cc4232007-02-11 01:37:51 +00001237 PN->replaceAllUsesWith(NV);
1238 NV->takeName(PN);
1239
Chris Lattnerf58c1a52005-09-23 06:39:30 +00001240 BB->getInstList().erase(PN);
1241 }
1242 return true;
1243}
Chris Lattnereaba3a12005-09-19 23:49:37 +00001244
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001245/// SimplifyCondBranchToTwoReturns - If we found a conditional branch that goes
1246/// to two returning blocks, try to merge them together into one return,
1247/// introducing a select if the return values disagree.
1248static bool SimplifyCondBranchToTwoReturns(BranchInst *BI) {
1249 assert(BI->isConditional() && "Must be a conditional branch");
1250 BasicBlock *TrueSucc = BI->getSuccessor(0);
1251 BasicBlock *FalseSucc = BI->getSuccessor(1);
1252 ReturnInst *TrueRet = cast<ReturnInst>(TrueSucc->getTerminator());
1253 ReturnInst *FalseRet = cast<ReturnInst>(FalseSucc->getTerminator());
1254
1255 // Check to ensure both blocks are empty (just a return) or optionally empty
1256 // with PHI nodes. If there are other instructions, merging would cause extra
1257 // computation on one path or the other.
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001258 if (!TrueSucc->getFirstNonPHIOrDbg()->isTerminator())
Devang Patel2cc86a12009-02-05 00:30:42 +00001259 return false;
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001260 if (!FalseSucc->getFirstNonPHIOrDbg()->isTerminator())
Devang Patel2cc86a12009-02-05 00:30:42 +00001261 return false;
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001262
1263 // Okay, we found a branch that is going to two return nodes. If
1264 // there is no return value for this function, just change the
1265 // branch into a return.
1266 if (FalseRet->getNumOperands() == 0) {
1267 TrueSucc->removePredecessor(BI->getParent());
1268 FalseSucc->removePredecessor(BI->getParent());
Owen Anderson1d0be152009-08-13 21:58:54 +00001269 ReturnInst::Create(BI->getContext(), 0, BI);
Eli Friedman080efb82008-12-16 20:54:32 +00001270 EraseTerminatorInstAndDCECond(BI);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001271 return true;
1272 }
1273
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001274 // Otherwise, figure out what the true and false return values are
1275 // so we can insert a new select instruction.
1276 Value *TrueValue = TrueRet->getReturnValue();
1277 Value *FalseValue = FalseRet->getReturnValue();
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001278
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001279 // Unwrap any PHI nodes in the return blocks.
1280 if (PHINode *TVPN = dyn_cast_or_null<PHINode>(TrueValue))
1281 if (TVPN->getParent() == TrueSucc)
1282 TrueValue = TVPN->getIncomingValueForBlock(BI->getParent());
1283 if (PHINode *FVPN = dyn_cast_or_null<PHINode>(FalseValue))
1284 if (FVPN->getParent() == FalseSucc)
1285 FalseValue = FVPN->getIncomingValueForBlock(BI->getParent());
1286
1287 // In order for this transformation to be safe, we must be able to
1288 // unconditionally execute both operands to the return. This is
1289 // normally the case, but we could have a potentially-trapping
1290 // constant expression that prevents this transformation from being
1291 // safe.
1292 if (ConstantExpr *TCV = dyn_cast_or_null<ConstantExpr>(TrueValue))
1293 if (TCV->canTrap())
1294 return false;
1295 if (ConstantExpr *FCV = dyn_cast_or_null<ConstantExpr>(FalseValue))
1296 if (FCV->canTrap())
1297 return false;
1298
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001299 // Okay, we collected all the mapped values and checked them for sanity, and
1300 // defined to really do this transformation. First, update the CFG.
1301 TrueSucc->removePredecessor(BI->getParent());
1302 FalseSucc->removePredecessor(BI->getParent());
1303
1304 // Insert select instructions where needed.
1305 Value *BrCond = BI->getCondition();
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001306 if (TrueValue) {
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001307 // Insert a select if the results differ.
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001308 if (TrueValue == FalseValue || isa<UndefValue>(FalseValue)) {
1309 } else if (isa<UndefValue>(TrueValue)) {
1310 TrueValue = FalseValue;
1311 } else {
1312 TrueValue = SelectInst::Create(BrCond, TrueValue,
1313 FalseValue, "retval", BI);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001314 }
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001315 }
1316
Dan Gohmanfc74abf2008-07-23 00:34:11 +00001317 Value *RI = !TrueValue ?
Owen Anderson1d0be152009-08-13 21:58:54 +00001318 ReturnInst::Create(BI->getContext(), BI) :
1319 ReturnInst::Create(BI->getContext(), TrueValue, BI);
Daniel Dunbare317bcc2009-08-23 10:29:55 +00001320 (void) RI;
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001321
David Greene89d6fd32010-01-05 01:26:52 +00001322 DEBUG(dbgs() << "\nCHANGING BRANCH TO TWO RETURNS INTO SELECT:"
Chris Lattnerbdff5482009-08-23 04:37:46 +00001323 << "\n " << *BI << "NewRet = " << *RI
1324 << "TRUEBLOCK: " << *TrueSucc << "FALSEBLOCK: "<< *FalseSucc);
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001325
Eli Friedman080efb82008-12-16 20:54:32 +00001326 EraseTerminatorInstAndDCECond(BI);
1327
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001328 return true;
1329}
1330
Chris Lattner1347e872008-07-13 21:12:01 +00001331/// FoldBranchToCommonDest - If this basic block is ONLY a setcc and a branch,
1332/// and if a predecessor branches to us and one of our successors, fold the
1333/// setcc into the predecessor and use logical operations to pick the right
1334/// destination.
Dan Gohman4b35f832009-06-27 21:30:38 +00001335bool llvm::FoldBranchToCommonDest(BranchInst *BI) {
Chris Lattner093a4382008-07-13 22:23:11 +00001336 BasicBlock *BB = BI->getParent();
Chris Lattner1347e872008-07-13 21:12:01 +00001337 Instruction *Cond = dyn_cast<Instruction>(BI->getCondition());
Owen Andersone84178a2010-07-14 19:52:16 +00001338 if (Cond == 0 || (!isa<CmpInst>(Cond) && !isa<BinaryOperator>(Cond)) ||
1339 Cond->getParent() != BB || !Cond->hasOneUse())
1340 return false;
Chris Lattner093a4382008-07-13 22:23:11 +00001341
Chris Lattner1347e872008-07-13 21:12:01 +00001342 // Only allow this if the condition is a simple instruction that can be
1343 // executed unconditionally. It must be in the same block as the branch, and
1344 // must be at the front of the block.
Devang Pateld0a203d2009-02-04 21:39:48 +00001345 BasicBlock::iterator FrontIt = BB->front();
1346 // Ignore dbg intrinsics.
1347 while(isa<DbgInfoIntrinsic>(FrontIt))
1348 ++FrontIt;
Owen Andersone84178a2010-07-14 19:52:16 +00001349
1350 // Allow a single instruction to be hoisted in addition to the compare
1351 // that feeds the branch. We later ensure that any values that _it_ uses
1352 // were also live in the predecessor, so that we don't unnecessarily create
1353 // register pressure or inhibit out-of-order execution.
1354 Instruction *BonusInst = 0;
1355 if (&*FrontIt != Cond &&
Owen Anderson2722dfa2010-07-15 16:38:22 +00001356 FrontIt->hasOneUse() && *FrontIt->use_begin() == Cond &&
1357 FrontIt->isSafeToSpeculativelyExecute()) {
Owen Andersone84178a2010-07-14 19:52:16 +00001358 BonusInst = &*FrontIt;
1359 ++FrontIt;
Devang Pateld0a203d2009-02-04 21:39:48 +00001360 }
Chris Lattner6ff645b2009-01-19 23:03:13 +00001361
Owen Andersone84178a2010-07-14 19:52:16 +00001362 // Only a single bonus inst is allowed.
1363 if (&*FrontIt != Cond)
1364 return false;
1365
Chris Lattner1347e872008-07-13 21:12:01 +00001366 // Make sure the instruction after the condition is the cond branch.
1367 BasicBlock::iterator CondIt = Cond; ++CondIt;
Devang Pateld0a203d2009-02-04 21:39:48 +00001368 // Ingore dbg intrinsics.
1369 while(isa<DbgInfoIntrinsic>(CondIt))
1370 ++CondIt;
1371 if (&*CondIt != BI) {
1372 assert (!isa<DbgInfoIntrinsic>(CondIt) && "Hey do not forget debug info!");
Chris Lattner1347e872008-07-13 21:12:01 +00001373 return false;
Devang Pateld0a203d2009-02-04 21:39:48 +00001374 }
Chris Lattner6ff645b2009-01-19 23:03:13 +00001375
1376 // Cond is known to be a compare or binary operator. Check to make sure that
1377 // neither operand is a potentially-trapping constant expression.
1378 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(0)))
1379 if (CE->canTrap())
1380 return false;
1381 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Cond->getOperand(1)))
1382 if (CE->canTrap())
1383 return false;
1384
Chris Lattner1347e872008-07-13 21:12:01 +00001385
1386 // Finally, don't infinitely unroll conditional loops.
1387 BasicBlock *TrueDest = BI->getSuccessor(0);
1388 BasicBlock *FalseDest = BI->getSuccessor(1);
1389 if (TrueDest == BB || FalseDest == BB)
1390 return false;
1391
1392 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
1393 BasicBlock *PredBlock = *PI;
1394 BranchInst *PBI = dyn_cast<BranchInst>(PredBlock->getTerminator());
Chris Lattner6ff645b2009-01-19 23:03:13 +00001395
Chris Lattner093a4382008-07-13 22:23:11 +00001396 // Check that we have two conditional branches. If there is a PHI node in
1397 // the common successor, verify that the same value flows in from both
1398 // blocks.
Chris Lattner1347e872008-07-13 21:12:01 +00001399 if (PBI == 0 || PBI->isUnconditional() ||
1400 !SafeToMergeTerminators(BI, PBI))
1401 continue;
1402
Owen Andersone84178a2010-07-14 19:52:16 +00001403 // Ensure that any values used in the bonus instruction are also used
1404 // by the terminator of the predecessor. This means that those values
1405 // must already have been resolved, so we won't be inhibiting the
1406 // out-of-order core by speculating them earlier.
1407 if (BonusInst) {
1408 // Collect the values used by the bonus inst
1409 SmallPtrSet<Value*, 4> UsedValues;
1410 for (Instruction::op_iterator OI = BonusInst->op_begin(),
1411 OE = BonusInst->op_end(); OI != OE; ++OI) {
1412 Value* V = *OI;
1413 if (!isa<Constant>(V))
1414 UsedValues.insert(V);
1415 }
1416
1417 SmallVector<std::pair<Value*, unsigned>, 4> Worklist;
1418 Worklist.push_back(std::make_pair(PBI->getOperand(0), 0));
1419
1420 // Walk up to four levels back up the use-def chain of the predecessor's
1421 // terminator to see if all those values were used. The choice of four
1422 // levels is arbitrary, to provide a compile-time-cost bound.
1423 while (!Worklist.empty()) {
1424 std::pair<Value*, unsigned> Pair = Worklist.back();
1425 Worklist.pop_back();
1426
1427 if (Pair.second >= 4) continue;
1428 UsedValues.erase(Pair.first);
1429 if (UsedValues.empty()) break;
1430
1431 if (Instruction* I = dyn_cast<Instruction>(Pair.first)) {
1432 for (Instruction::op_iterator OI = I->op_begin(), OE = I->op_end();
1433 OI != OE; ++OI)
1434 Worklist.push_back(std::make_pair(OI->get(), Pair.second+1));
1435 }
1436 }
1437
1438 if (!UsedValues.empty()) return false;
1439 }
1440
Chris Lattner36989092008-07-13 21:20:19 +00001441 Instruction::BinaryOps Opc;
1442 bool InvertPredCond = false;
1443
1444 if (PBI->getSuccessor(0) == TrueDest)
1445 Opc = Instruction::Or;
1446 else if (PBI->getSuccessor(1) == FalseDest)
1447 Opc = Instruction::And;
1448 else if (PBI->getSuccessor(0) == FalseDest)
1449 Opc = Instruction::And, InvertPredCond = true;
1450 else if (PBI->getSuccessor(1) == TrueDest)
1451 Opc = Instruction::Or, InvertPredCond = true;
1452 else
1453 continue;
1454
David Greene89d6fd32010-01-05 01:26:52 +00001455 DEBUG(dbgs() << "FOLDING BRANCH TO COMMON DEST:\n" << *PBI << *BB);
Chris Lattner6ff645b2009-01-19 23:03:13 +00001456
Chris Lattner36989092008-07-13 21:20:19 +00001457 // If we need to invert the condition in the pred block to match, do so now.
1458 if (InvertPredCond) {
Chris Lattner1347e872008-07-13 21:12:01 +00001459 Value *NewCond =
Dan Gohman4ae51262009-08-12 16:23:25 +00001460 BinaryOperator::CreateNot(PBI->getCondition(),
Chris Lattner36989092008-07-13 21:20:19 +00001461 PBI->getCondition()->getName()+".not", PBI);
Chris Lattner1347e872008-07-13 21:12:01 +00001462 PBI->setCondition(NewCond);
1463 BasicBlock *OldTrue = PBI->getSuccessor(0);
1464 BasicBlock *OldFalse = PBI->getSuccessor(1);
1465 PBI->setSuccessor(0, OldFalse);
1466 PBI->setSuccessor(1, OldTrue);
1467 }
Chris Lattner70087f32008-07-13 21:15:11 +00001468
Owen Andersone84178a2010-07-14 19:52:16 +00001469 // If we have a bonus inst, clone it into the predecessor block.
1470 Instruction *NewBonus = 0;
1471 if (BonusInst) {
1472 NewBonus = BonusInst->clone();
1473 PredBlock->getInstList().insert(PBI, NewBonus);
1474 NewBonus->takeName(BonusInst);
1475 BonusInst->setName(BonusInst->getName()+".old");
1476 }
1477
Chris Lattner36989092008-07-13 21:20:19 +00001478 // Clone Cond into the predecessor basic block, and or/and the
1479 // two conditions together.
Nick Lewycky67760642009-09-27 07:38:41 +00001480 Instruction *New = Cond->clone();
Owen Andersone84178a2010-07-14 19:52:16 +00001481 if (BonusInst) New->replaceUsesOfWith(BonusInst, NewBonus);
Chris Lattner36989092008-07-13 21:20:19 +00001482 PredBlock->getInstList().insert(PBI, New);
1483 New->takeName(Cond);
1484 Cond->setName(New->getName()+".old");
Chris Lattner70087f32008-07-13 21:15:11 +00001485
Chris Lattner36989092008-07-13 21:20:19 +00001486 Value *NewCond = BinaryOperator::Create(Opc, PBI->getCondition(),
1487 New, "or.cond", PBI);
1488 PBI->setCondition(NewCond);
1489 if (PBI->getSuccessor(0) == BB) {
1490 AddPredecessorToBlock(TrueDest, PredBlock, BB);
1491 PBI->setSuccessor(0, TrueDest);
Chris Lattner1347e872008-07-13 21:12:01 +00001492 }
Chris Lattner36989092008-07-13 21:20:19 +00001493 if (PBI->getSuccessor(1) == BB) {
1494 AddPredecessorToBlock(FalseDest, PredBlock, BB);
1495 PBI->setSuccessor(1, FalseDest);
1496 }
1497 return true;
Chris Lattner1347e872008-07-13 21:12:01 +00001498 }
1499 return false;
1500}
1501
Chris Lattner867661a2008-07-13 21:53:26 +00001502/// SimplifyCondBranchToCondBranch - If we have a conditional branch as a
1503/// predecessor of another block, this function tries to simplify it. We know
1504/// that PBI and BI are both conditional branches, and BI is in one of the
1505/// successor blocks of PBI - PBI branches to BI.
1506static bool SimplifyCondBranchToCondBranch(BranchInst *PBI, BranchInst *BI) {
1507 assert(PBI->isConditional() && BI->isConditional());
1508 BasicBlock *BB = BI->getParent();
Dan Gohman4ae51262009-08-12 16:23:25 +00001509
Chris Lattner867661a2008-07-13 21:53:26 +00001510 // If this block ends with a branch instruction, and if there is a
1511 // predecessor that ends on a branch of the same condition, make
1512 // this conditional branch redundant.
1513 if (PBI->getCondition() == BI->getCondition() &&
1514 PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
1515 // Okay, the outcome of this conditional branch is statically
1516 // knowable. If this block had a single pred, handle specially.
1517 if (BB->getSinglePredecessor()) {
1518 // Turn this into a branch on constant.
1519 bool CondIsTrue = PBI->getSuccessor(0) == BB;
Owen Anderson1d0be152009-08-13 21:58:54 +00001520 BI->setCondition(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
1521 CondIsTrue));
Chris Lattner867661a2008-07-13 21:53:26 +00001522 return true; // Nuke the branch on constant.
1523 }
1524
1525 // Otherwise, if there are multiple predecessors, insert a PHI that merges
1526 // in the constant and simplify the block result. Subsequent passes of
1527 // simplifycfg will thread the block.
1528 if (BlockIsSimpleEnoughToThreadThrough(BB)) {
Owen Anderson1d0be152009-08-13 21:58:54 +00001529 PHINode *NewPN = PHINode::Create(Type::getInt1Ty(BB->getContext()),
Chris Lattner867661a2008-07-13 21:53:26 +00001530 BI->getCondition()->getName() + ".pr",
1531 BB->begin());
Chris Lattnereb388af2008-07-13 21:55:46 +00001532 // Okay, we're going to insert the PHI node. Since PBI is not the only
1533 // predecessor, compute the PHI'd conditional value for all of the preds.
1534 // Any predecessor where the condition is not computable we keep symbolic.
Gabor Greif62539832010-07-12 10:59:23 +00001535 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
1536 BasicBlock *P = *PI;
1537 if ((PBI = dyn_cast<BranchInst>(P->getTerminator())) &&
Chris Lattner867661a2008-07-13 21:53:26 +00001538 PBI != BI && PBI->isConditional() &&
1539 PBI->getCondition() == BI->getCondition() &&
1540 PBI->getSuccessor(0) != PBI->getSuccessor(1)) {
1541 bool CondIsTrue = PBI->getSuccessor(0) == BB;
Owen Anderson1d0be152009-08-13 21:58:54 +00001542 NewPN->addIncoming(ConstantInt::get(Type::getInt1Ty(BB->getContext()),
Gabor Greif62539832010-07-12 10:59:23 +00001543 CondIsTrue), P);
Chris Lattner867661a2008-07-13 21:53:26 +00001544 } else {
Gabor Greif62539832010-07-12 10:59:23 +00001545 NewPN->addIncoming(BI->getCondition(), P);
Chris Lattner867661a2008-07-13 21:53:26 +00001546 }
Gabor Greif62539832010-07-12 10:59:23 +00001547 }
Chris Lattner867661a2008-07-13 21:53:26 +00001548
1549 BI->setCondition(NewPN);
Chris Lattner867661a2008-07-13 21:53:26 +00001550 return true;
1551 }
1552 }
1553
1554 // If this is a conditional branch in an empty block, and if any
1555 // predecessors is a conditional branch to one of our destinations,
1556 // fold the conditions into logical ops and one cond br.
Zhou Shenga8d57fe2009-02-26 06:56:37 +00001557 BasicBlock::iterator BBI = BB->begin();
1558 // Ignore dbg intrinsics.
1559 while (isa<DbgInfoIntrinsic>(BBI))
1560 ++BBI;
1561 if (&*BBI != BI)
Chris Lattnerb8245122008-07-13 22:04:41 +00001562 return false;
Chris Lattner63bf29b2009-01-20 01:15:41 +00001563
1564
1565 if (ConstantExpr *CE = dyn_cast<ConstantExpr>(BI->getCondition()))
1566 if (CE->canTrap())
1567 return false;
Chris Lattnerb8245122008-07-13 22:04:41 +00001568
1569 int PBIOp, BIOp;
1570 if (PBI->getSuccessor(0) == BI->getSuccessor(0))
1571 PBIOp = BIOp = 0;
1572 else if (PBI->getSuccessor(0) == BI->getSuccessor(1))
1573 PBIOp = 0, BIOp = 1;
1574 else if (PBI->getSuccessor(1) == BI->getSuccessor(0))
1575 PBIOp = 1, BIOp = 0;
1576 else if (PBI->getSuccessor(1) == BI->getSuccessor(1))
1577 PBIOp = BIOp = 1;
1578 else
1579 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001580
Chris Lattnerb8245122008-07-13 22:04:41 +00001581 // Check to make sure that the other destination of this branch
1582 // isn't BB itself. If so, this is an infinite loop that will
1583 // keep getting unwound.
1584 if (PBI->getSuccessor(PBIOp) == BB)
1585 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001586
Chris Lattnerb8245122008-07-13 22:04:41 +00001587 // Do not perform this transformation if it would require
1588 // insertion of a large number of select instructions. For targets
1589 // without predication/cmovs, this is a big pessimization.
1590 BasicBlock *CommonDest = PBI->getSuccessor(PBIOp);
Chris Lattner867661a2008-07-13 21:53:26 +00001591
Chris Lattnerb8245122008-07-13 22:04:41 +00001592 unsigned NumPhis = 0;
1593 for (BasicBlock::iterator II = CommonDest->begin();
1594 isa<PHINode>(II); ++II, ++NumPhis)
1595 if (NumPhis > 2) // Disable this xform.
1596 return false;
Chris Lattner867661a2008-07-13 21:53:26 +00001597
Chris Lattnerb8245122008-07-13 22:04:41 +00001598 // Finally, if everything is ok, fold the branches to logical ops.
1599 BasicBlock *OtherDest = BI->getSuccessor(BIOp ^ 1);
1600
David Greene89d6fd32010-01-05 01:26:52 +00001601 DEBUG(dbgs() << "FOLDING BRs:" << *PBI->getParent()
Chris Lattnerbdff5482009-08-23 04:37:46 +00001602 << "AND: " << *BI->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001603
Chris Lattner093a4382008-07-13 22:23:11 +00001604
1605 // If OtherDest *is* BB, then BB is a basic block with a single conditional
1606 // branch in it, where one edge (OtherDest) goes back to itself but the other
1607 // exits. We don't *know* that the program avoids the infinite loop
1608 // (even though that seems likely). If we do this xform naively, we'll end up
1609 // recursively unpeeling the loop. Since we know that (after the xform is
1610 // done) that the block *is* infinite if reached, we just make it an obviously
1611 // infinite loop with no cond branch.
1612 if (OtherDest == BB) {
1613 // Insert it at the end of the function, because it's either code,
1614 // or it won't matter if it's hot. :)
Owen Anderson1d0be152009-08-13 21:58:54 +00001615 BasicBlock *InfLoopBlock = BasicBlock::Create(BB->getContext(),
1616 "infloop", BB->getParent());
Chris Lattner093a4382008-07-13 22:23:11 +00001617 BranchInst::Create(InfLoopBlock, InfLoopBlock);
1618 OtherDest = InfLoopBlock;
1619 }
1620
David Greene89d6fd32010-01-05 01:26:52 +00001621 DEBUG(dbgs() << *PBI->getParent()->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001622
1623 // BI may have other predecessors. Because of this, we leave
1624 // it alone, but modify PBI.
1625
1626 // Make sure we get to CommonDest on True&True directions.
1627 Value *PBICond = PBI->getCondition();
1628 if (PBIOp)
Dan Gohman4ae51262009-08-12 16:23:25 +00001629 PBICond = BinaryOperator::CreateNot(PBICond,
Chris Lattnerb8245122008-07-13 22:04:41 +00001630 PBICond->getName()+".not",
1631 PBI);
1632 Value *BICond = BI->getCondition();
1633 if (BIOp)
Dan Gohman4ae51262009-08-12 16:23:25 +00001634 BICond = BinaryOperator::CreateNot(BICond,
Chris Lattnerb8245122008-07-13 22:04:41 +00001635 BICond->getName()+".not",
1636 PBI);
1637 // Merge the conditions.
1638 Value *Cond = BinaryOperator::CreateOr(PBICond, BICond, "brmerge", PBI);
1639
1640 // Modify PBI to branch on the new condition to the new dests.
1641 PBI->setCondition(Cond);
1642 PBI->setSuccessor(0, CommonDest);
1643 PBI->setSuccessor(1, OtherDest);
1644
1645 // OtherDest may have phi nodes. If so, add an entry from PBI's
1646 // block that are identical to the entries for BI's block.
1647 PHINode *PN;
1648 for (BasicBlock::iterator II = OtherDest->begin();
1649 (PN = dyn_cast<PHINode>(II)); ++II) {
1650 Value *V = PN->getIncomingValueForBlock(BB);
1651 PN->addIncoming(V, PBI->getParent());
1652 }
1653
1654 // We know that the CommonDest already had an edge from PBI to
1655 // it. If it has PHIs though, the PHIs may have different
1656 // entries for BB and PBI's BB. If so, insert a select to make
1657 // them agree.
1658 for (BasicBlock::iterator II = CommonDest->begin();
1659 (PN = dyn_cast<PHINode>(II)); ++II) {
1660 Value *BIV = PN->getIncomingValueForBlock(BB);
1661 unsigned PBBIdx = PN->getBasicBlockIndex(PBI->getParent());
1662 Value *PBIV = PN->getIncomingValue(PBBIdx);
1663 if (BIV != PBIV) {
1664 // Insert a select in PBI to pick the right value.
1665 Value *NV = SelectInst::Create(PBICond, PBIV, BIV,
1666 PBIV->getName()+".mux", PBI);
1667 PN->setIncomingValue(PBBIdx, NV);
Chris Lattner867661a2008-07-13 21:53:26 +00001668 }
1669 }
Chris Lattnerb8245122008-07-13 22:04:41 +00001670
David Greene89d6fd32010-01-05 01:26:52 +00001671 DEBUG(dbgs() << "INTO: " << *PBI->getParent());
1672 DEBUG(dbgs() << *PBI->getParent()->getParent());
Chris Lattnerb8245122008-07-13 22:04:41 +00001673
1674 // This basic block is probably dead. We know it has at least
1675 // one fewer predecessor.
1676 return true;
Chris Lattner867661a2008-07-13 21:53:26 +00001677}
1678
Frits van Bommel7ac40c32010-12-05 18:29:03 +00001679// SimplifyIndirectBrOnSelect - Replaces
1680// (indirectbr (select cond, blockaddress(@fn, BlockA),
1681// blockaddress(@fn, BlockB)))
1682// with
1683// (br cond, BlockA, BlockB).
1684static bool SimplifyIndirectBrOnSelect(IndirectBrInst *IBI, SelectInst *SI) {
1685 // Check that both operands of the select are block addresses.
1686 BlockAddress *TBA = dyn_cast<BlockAddress>(SI->getTrueValue());
1687 BlockAddress *FBA = dyn_cast<BlockAddress>(SI->getFalseValue());
1688 if (!TBA || !FBA)
1689 return false;
1690
1691 // Extract the actual blocks.
1692 BasicBlock *TrueBB = TBA->getBasicBlock();
1693 BasicBlock *FalseBB = FBA->getBasicBlock();
1694
1695 // Remove any superfluous successor edges from the CFG.
1696 // First, figure out which successors to preserve.
1697 // If TrueBB and FalseBB are equal, only try to preserve one copy of that
1698 // successor.
1699 BasicBlock *KeepEdge1 = TrueBB;
1700 BasicBlock *KeepEdge2 = TrueBB != FalseBB ? FalseBB : 0;
1701
1702 // Then remove the rest.
1703 for (unsigned I = 0, E = IBI->getNumSuccessors(); I != E; ++I) {
1704 BasicBlock *Succ = IBI->getSuccessor(I);
1705 // Make sure only to keep exactly one copy of each edge.
1706 if (Succ == KeepEdge1)
1707 KeepEdge1 = 0;
1708 else if (Succ == KeepEdge2)
1709 KeepEdge2 = 0;
1710 else
1711 Succ->removePredecessor(IBI->getParent());
1712 }
1713
1714 // Insert an appropriate new terminator.
1715 if ((KeepEdge1 == 0) && (KeepEdge2 == 0)) {
1716 if (TrueBB == FalseBB)
1717 // We were only looking for one successor, and it was present.
1718 // Create an unconditional branch to it.
1719 BranchInst::Create(TrueBB, IBI);
1720 else
1721 // We found both of the successors we were looking for.
1722 // Create a conditional branch sharing the condition of the select.
1723 BranchInst::Create(TrueBB, FalseBB, SI->getCondition(), IBI);
1724 } else if (KeepEdge1 && (KeepEdge2 || TrueBB == FalseBB)) {
1725 // Neither of the selected blocks were successors, so this
1726 // indirectbr must be unreachable.
1727 new UnreachableInst(IBI->getContext(), IBI);
1728 } else {
1729 // One of the selected values was a successor, but the other wasn't.
1730 // Insert an unconditional branch to the one that was found;
1731 // the edge to the one that wasn't must be unreachable.
1732 if (KeepEdge1 == 0)
1733 // Only TrueBB was found.
1734 BranchInst::Create(TrueBB, IBI);
1735 else
1736 // Only FalseBB was found.
1737 BranchInst::Create(FalseBB, IBI);
1738 }
1739
1740 EraseTerminatorInstAndDCECond(IBI);
1741 return true;
1742}
1743
Chris Lattner61c77442010-12-13 03:18:54 +00001744/// TryToSimplifyUncondBranchWithICmpInIt - This is called when we find an icmp
1745/// instruction (a seteq/setne with a constant) as the only instruction in a
1746/// block that ends with an uncond branch. We are looking for a very specific
1747/// pattern that occurs when "A == 1 || A == 2 || A == 3" gets simplified. In
1748/// this case, we merge the first two "or's of icmp" into a switch, but then the
1749/// default value goes to an uncond block with a seteq in it, we get something
1750/// like:
1751///
1752/// switch i8 %A, label %DEFAULT [ i8 1, label %end i8 2, label %end ]
1753/// DEFAULT:
1754/// %tmp = icmp eq i8 %A, 92
1755/// br label %end
1756/// end:
1757/// ... = phi i1 [ true, %entry ], [ %tmp, %DEFAULT ], [ true, %entry ]
1758///
1759/// We prefer to split the edge to 'end' so that there is a true/false entry to
1760/// the PHI, merging the third icmp into the switch.
1761static bool TryToSimplifyUncondBranchWithICmpInIt(ICmpInst *ICI) {
1762 BasicBlock *BB = ICI->getParent();
1763 // If the block has any PHIs in it or the icmp has multiple uses, it is too
1764 // complex.
1765 if (isa<PHINode>(BB->begin()) || !ICI->hasOneUse()) return false;
1766
1767 Value *V = ICI->getOperand(0);
1768 ConstantInt *Cst = cast<ConstantInt>(ICI->getOperand(1));
1769
1770 // The pattern we're looking for is where our only predecessor is a switch on
1771 // 'V' and this block is the default case for the switch. In this case we can
1772 // fold the compared value into the switch to simplify things.
1773 BasicBlock *Pred = BB->getSinglePredecessor();
1774 if (Pred == 0 || !isa<SwitchInst>(Pred->getTerminator())) return false;
1775
1776 SwitchInst *SI = cast<SwitchInst>(Pred->getTerminator());
1777 if (SI->getCondition() != V)
1778 return false;
1779
1780 // If BB is reachable on a non-default case, then we simply know the value of
1781 // V in this block. Substitute it and constant fold the icmp instruction
1782 // away.
1783 if (SI->getDefaultDest() != BB) {
1784 ConstantInt *VVal = SI->findCaseDest(BB);
1785 assert(VVal && "Should have a unique destination value");
1786 ICI->setOperand(0, VVal);
1787
1788 if (Constant *C = ConstantFoldInstruction(ICI)) {
1789 ICI->replaceAllUsesWith(C);
1790 ICI->eraseFromParent();
1791 }
1792 // BB is now empty, so it is likely to simplify away.
1793 return SimplifyCFG(BB) | true;
1794 }
1795
Chris Lattnerabf70672010-12-13 03:43:57 +00001796 // Ok, the block is reachable from the default dest. If the constant we're
1797 // comparing exists in one of the other edges, then we can constant fold ICI
1798 // and zap it.
1799 if (SI->findCaseValue(Cst) != 0) {
1800 Value *V;
1801 if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
1802 V = ConstantInt::getFalse(BB->getContext());
1803 else
1804 V = ConstantInt::getTrue(BB->getContext());
1805
1806 ICI->replaceAllUsesWith(V);
1807 ICI->eraseFromParent();
1808 // BB is now empty, so it is likely to simplify away.
1809 return SimplifyCFG(BB) | true;
1810 }
1811
Chris Lattner61c77442010-12-13 03:18:54 +00001812 // The use of the icmp has to be in the 'end' block, by the only PHI node in
1813 // the block.
1814 BasicBlock *SuccBlock = BB->getTerminator()->getSuccessor(0);
1815 PHINode *PHIUse = dyn_cast<PHINode>(ICI->use_back());
1816 if (PHIUse == 0 || PHIUse != &SuccBlock->front() ||
1817 isa<PHINode>(++BasicBlock::iterator(PHIUse)))
1818 return false;
1819
1820 // If the icmp is a SETEQ, then the default dest gets false, the new edge gets
1821 // true in the PHI.
1822 Constant *DefaultCst = ConstantInt::getTrue(BB->getContext());
1823 Constant *NewCst = ConstantInt::getFalse(BB->getContext());
1824
1825 if (ICI->getPredicate() == ICmpInst::ICMP_EQ)
1826 std::swap(DefaultCst, NewCst);
1827
1828 // Replace ICI (which is used by the PHI for the default value) with true or
1829 // false depending on if it is EQ or NE.
1830 ICI->replaceAllUsesWith(DefaultCst);
1831 ICI->eraseFromParent();
1832
1833 // Okay, the switch goes to this block on a default value. Add an edge from
1834 // the switch to the merge point on the compared value.
1835 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "switch.edge",
1836 BB->getParent(), BB);
1837 SI->addCase(Cst, NewBB);
1838
1839 // NewBB branches to the phi block, add the uncond branch and the phi entry.
1840 BranchInst::Create(SuccBlock, NewBB);
1841 PHIUse->addIncoming(NewCst, NewBB);
1842 return true;
1843}
1844
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00001845bool SimplifyCFGOpt::run(BasicBlock *BB) {
Chris Lattnerdc3602b2003-08-24 18:36:16 +00001846 bool Changed = false;
Duncan Sands5f284752010-10-24 12:23:30 +00001847 Function *Fn = BB->getParent();
Chris Lattner01d1ee32002-05-21 20:50:24 +00001848
Duncan Sands5f284752010-10-24 12:23:30 +00001849 assert(BB && Fn && "Block not embedded in function!");
Chris Lattner01d1ee32002-05-21 20:50:24 +00001850 assert(BB->getTerminator() && "Degenerate basic block encountered!");
Chris Lattner01d1ee32002-05-21 20:50:24 +00001851
Dan Gohmane2c6d132010-08-14 00:29:42 +00001852 // Remove basic blocks that have no predecessors (except the entry block)...
1853 // or that just have themself as a predecessor. These are unreachable.
Duncan Sands5f284752010-10-24 12:23:30 +00001854 if ((pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) ||
Dan Gohmane2c6d132010-08-14 00:29:42 +00001855 BB->getSinglePredecessor() == BB) {
David Greene89d6fd32010-01-05 01:26:52 +00001856 DEBUG(dbgs() << "Removing BB: \n" << *BB);
Chris Lattner71af9b02008-12-03 06:40:52 +00001857 DeleteDeadBlock(BB);
Chris Lattner01d1ee32002-05-21 20:50:24 +00001858 return true;
1859 }
1860
Chris Lattner694e37f2003-08-17 19:41:53 +00001861 // Check to see if we can constant propagate this terminator instruction
1862 // away...
Chris Lattnerdc3602b2003-08-24 18:36:16 +00001863 Changed |= ConstantFoldTerminator(BB);
Chris Lattner694e37f2003-08-17 19:41:53 +00001864
Dan Gohman2c635662009-10-30 22:39:04 +00001865 // Check for and eliminate duplicate PHI nodes in this block.
1866 Changed |= EliminateDuplicatePHINodes(BB);
1867
Dan Gohman882d87d2008-03-11 21:53:06 +00001868 // If there is a trivial two-entry PHI node in this basic block, and we can
1869 // eliminate it, do so now.
1870 if (PHINode *PN = dyn_cast<PHINode>(BB->begin()))
1871 if (PN->getNumIncomingValues() == 2)
1872 Changed |= FoldTwoEntryPHINode(PN);
1873
Chris Lattner19831ec2004-02-16 06:35:48 +00001874 // If this is a returning block with only PHI nodes in it, fold the return
1875 // instruction into any unconditional branch predecessors.
Chris Lattner147af6b2004-04-02 18:13:43 +00001876 //
1877 // If any predecessor is a conditional branch that just selects among
1878 // different return values, fold the replace the branch/return with a select
1879 // and return.
Chris Lattner19831ec2004-02-16 06:35:48 +00001880 if (ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator())) {
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001881 if (BB->getFirstNonPHIOrDbg()->isTerminator()) {
Chris Lattner147af6b2004-04-02 18:13:43 +00001882 // Find predecessors that end with branches.
Chris Lattner82442432008-02-18 07:42:56 +00001883 SmallVector<BasicBlock*, 8> UncondBranchPreds;
1884 SmallVector<BranchInst*, 8> CondBranchPreds;
Chris Lattner19831ec2004-02-16 06:35:48 +00001885 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
Gabor Greif58969352010-07-09 15:25:09 +00001886 BasicBlock *P = *PI;
1887 TerminatorInst *PTI = P->getTerminator();
Anton Korobeynikov07e6e562008-02-20 11:26:25 +00001888 if (BranchInst *BI = dyn_cast<BranchInst>(PTI)) {
Chris Lattner19831ec2004-02-16 06:35:48 +00001889 if (BI->isUnconditional())
Gabor Greif58969352010-07-09 15:25:09 +00001890 UncondBranchPreds.push_back(P);
Chris Lattner147af6b2004-04-02 18:13:43 +00001891 else
1892 CondBranchPreds.push_back(BI);
Anton Korobeynikov07e6e562008-02-20 11:26:25 +00001893 }
Chris Lattner19831ec2004-02-16 06:35:48 +00001894 }
Misha Brukmanfd939082005-04-21 23:48:37 +00001895
Chris Lattner19831ec2004-02-16 06:35:48 +00001896 // If we found some, do the transformation!
Bill Wendling1c855032009-03-03 19:25:16 +00001897 if (!UncondBranchPreds.empty()) {
Chris Lattner19831ec2004-02-16 06:35:48 +00001898 while (!UncondBranchPreds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +00001899 BasicBlock *Pred = UncondBranchPreds.pop_back_val();
David Greene89d6fd32010-01-05 01:26:52 +00001900 DEBUG(dbgs() << "FOLDING: " << *BB
Chris Lattnerbdff5482009-08-23 04:37:46 +00001901 << "INTO UNCOND BRANCH PRED: " << *Pred);
Chris Lattner19831ec2004-02-16 06:35:48 +00001902 Instruction *UncondBranch = Pred->getTerminator();
1903 // Clone the return and add it to the end of the predecessor.
Nick Lewycky67760642009-09-27 07:38:41 +00001904 Instruction *NewRet = RI->clone();
Chris Lattner19831ec2004-02-16 06:35:48 +00001905 Pred->getInstList().push_back(NewRet);
1906
1907 // If the return instruction returns a value, and if the value was a
1908 // PHI node in "BB", propagate the right value into the return.
Gabor Greiff7ea3632008-06-10 22:03:26 +00001909 for (User::op_iterator i = NewRet->op_begin(), e = NewRet->op_end();
1910 i != e; ++i)
1911 if (PHINode *PN = dyn_cast<PHINode>(*i))
Chris Lattner19831ec2004-02-16 06:35:48 +00001912 if (PN->getParent() == BB)
Gabor Greiff7ea3632008-06-10 22:03:26 +00001913 *i = PN->getIncomingValueForBlock(Pred);
Chris Lattnerffba5822008-04-28 00:19:07 +00001914
Chris Lattner19831ec2004-02-16 06:35:48 +00001915 // Update any PHI nodes in the returning block to realize that we no
1916 // longer branch to them.
1917 BB->removePredecessor(Pred);
1918 Pred->getInstList().erase(UncondBranch);
1919 }
1920
1921 // If we eliminated all predecessors of the block, delete the block now.
1922 if (pred_begin(BB) == pred_end(BB))
1923 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00001924 Fn->getBasicBlockList().erase(BB);
Chris Lattner19831ec2004-02-16 06:35:48 +00001925
Chris Lattner19831ec2004-02-16 06:35:48 +00001926 return true;
1927 }
Chris Lattner147af6b2004-04-02 18:13:43 +00001928
1929 // Check out all of the conditional branches going to this return
1930 // instruction. If any of them just select between returns, change the
1931 // branch itself into a select/return pair.
1932 while (!CondBranchPreds.empty()) {
Dan Gohmane9d87f42009-05-06 17:22:41 +00001933 BranchInst *BI = CondBranchPreds.pop_back_val();
Chris Lattner147af6b2004-04-02 18:13:43 +00001934
1935 // Check to see if the non-BB successor is also a return block.
Chris Lattnerc9e495c2008-04-24 00:01:19 +00001936 if (isa<ReturnInst>(BI->getSuccessor(0)->getTerminator()) &&
1937 isa<ReturnInst>(BI->getSuccessor(1)->getTerminator()) &&
1938 SimplifyCondBranchToTwoReturns(BI))
1939 return true;
Chris Lattner147af6b2004-04-02 18:13:43 +00001940 }
Chris Lattner19831ec2004-02-16 06:35:48 +00001941 }
Reid Spencer3ed469c2006-11-02 20:25:50 +00001942 } else if (isa<UnwindInst>(BB->begin())) {
Chris Lattnere14ea082004-02-24 05:54:22 +00001943 // Check to see if the first instruction in this block is just an unwind.
1944 // If so, replace any invoke instructions which use this as an exception
Chris Lattner11f15db2009-10-13 18:13:05 +00001945 // destination with call instructions.
Chris Lattnere14ea082004-02-24 05:54:22 +00001946 //
Chris Lattner82442432008-02-18 07:42:56 +00001947 SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
Chris Lattnere14ea082004-02-24 05:54:22 +00001948 while (!Preds.empty()) {
1949 BasicBlock *Pred = Preds.back();
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001950 InvokeInst *II = dyn_cast<InvokeInst>(Pred->getTerminator());
1951 if (II && II->getUnwindDest() == BB) {
1952 // Insert a new branch instruction before the invoke, because this
1953 // is now a fall through.
1954 BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
1955 Pred->getInstList().remove(II); // Take out of symbol table
Misha Brukmanfd939082005-04-21 23:48:37 +00001956
Chris Lattner9a2b72a2010-12-13 01:47:07 +00001957 // Insert the call now.
1958 SmallVector<Value*,8> Args(II->op_begin(), II->op_end()-3);
1959 CallInst *CI = CallInst::Create(II->getCalledValue(),
1960 Args.begin(), Args.end(),
1961 II->getName(), BI);
1962 CI->setCallingConv(II->getCallingConv());
1963 CI->setAttributes(II->getAttributes());
1964 // If the invoke produced a value, the Call now does instead.
1965 II->replaceAllUsesWith(CI);
1966 delete II;
1967 Changed = true;
1968 }
Misha Brukmanfd939082005-04-21 23:48:37 +00001969
Chris Lattnere14ea082004-02-24 05:54:22 +00001970 Preds.pop_back();
1971 }
Chris Lattner8e509dd2004-02-24 16:09:21 +00001972
Duncan Sands5f284752010-10-24 12:23:30 +00001973 // If this block is now dead (and isn't the entry block), remove it.
1974 if (pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) {
Chris Lattner8e509dd2004-02-24 16:09:21 +00001975 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00001976 Fn->getBasicBlockList().erase(BB);
Chris Lattner8e509dd2004-02-24 16:09:21 +00001977 return true;
1978 }
1979
Chris Lattner623369a2005-02-24 06:17:52 +00001980 } else if (SwitchInst *SI = dyn_cast<SwitchInst>(BB->getTerminator())) {
1981 if (isValueEqualityComparison(SI)) {
1982 // If we only have one predecessor, and if it is a branch on this value,
1983 // see if that predecessor totally determines the outcome of this switch.
1984 if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
1985 if (SimplifyEqualityComparisonWithOnlyPredecessor(SI, OnlyPred))
1986 return SimplifyCFG(BB) || 1;
1987
1988 // If the block only contains the switch, see if we can fold the block
1989 // away into any preds.
Zhou Sheng9a7c7432009-02-25 15:34:27 +00001990 BasicBlock::iterator BBI = BB->begin();
1991 // Ignore dbg intrinsics.
1992 while (isa<DbgInfoIntrinsic>(BBI))
1993 ++BBI;
1994 if (SI == &*BBI)
Chris Lattner623369a2005-02-24 06:17:52 +00001995 if (FoldValueComparisonIntoPredecessors(SI))
1996 return SimplifyCFG(BB) || 1;
1997 }
Chris Lattner542f1492004-02-28 21:28:10 +00001998 } else if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator())) {
Chris Lattner7e663482005-08-03 00:11:16 +00001999 if (BI->isUnconditional()) {
Chris Lattnerdcb54ce2010-12-13 01:28:06 +00002000 // If the Terminator is the only non-phi instruction, simplify the block.
Chris Lattner61c77442010-12-13 03:18:54 +00002001 BasicBlock::iterator I = BB->getFirstNonPHIOrDbg();
Chris Lattnerdcb54ce2010-12-13 01:28:06 +00002002 if (I->isTerminator() && BB != &Fn->getEntryBlock() &&
2003 TryToSimplifyUncondBranchFromEmptyBlock(BB))
2004 return true;
Chris Lattner7e663482005-08-03 00:11:16 +00002005
Chris Lattner61c77442010-12-13 03:18:54 +00002006 // If the only instruction in the block is a seteq/setne comparison
2007 // against a constant, try to simplify the block.
2008 if (ICmpInst *ICI = dyn_cast<ICmpInst>(I))
2009 if (ICI->isEquality() && isa<ConstantInt>(ICI->getOperand(1))) {
2010 for (++I; isa<DbgInfoIntrinsic>(I); ++I)
2011 ;
2012 if (I->isTerminator() &&
2013 TryToSimplifyUncondBranchWithICmpInIt(ICI))
2014 return true;
2015 }
2016
Chris Lattner7e663482005-08-03 00:11:16 +00002017 } else { // Conditional branch
Reid Spencer3ed469c2006-11-02 20:25:50 +00002018 if (isValueEqualityComparison(BI)) {
Chris Lattner623369a2005-02-24 06:17:52 +00002019 // If we only have one predecessor, and if it is a branch on this value,
2020 // see if that predecessor totally determines the outcome of this
2021 // switch.
2022 if (BasicBlock *OnlyPred = BB->getSinglePredecessor())
2023 if (SimplifyEqualityComparisonWithOnlyPredecessor(BI, OnlyPred))
Bill Wendlingc69b4a52010-03-14 10:40:28 +00002024 return SimplifyCFG(BB) | true;
Chris Lattner623369a2005-02-24 06:17:52 +00002025
Chris Lattnere67fa052004-05-01 23:35:43 +00002026 // This block must be empty, except for the setcond inst, if it exists.
Devang Patel556b20a2009-02-04 01:06:11 +00002027 // Ignore dbg intrinsics.
Chris Lattnere67fa052004-05-01 23:35:43 +00002028 BasicBlock::iterator I = BB->begin();
Devang Pateld0a203d2009-02-04 21:39:48 +00002029 // Ignore dbg intrinsics.
Devang Patel556b20a2009-02-04 01:06:11 +00002030 while (isa<DbgInfoIntrinsic>(I))
Devang Pateld0a203d2009-02-04 21:39:48 +00002031 ++I;
2032 if (&*I == BI) {
Chris Lattnere67fa052004-05-01 23:35:43 +00002033 if (FoldValueComparisonIntoPredecessors(BI))
2034 return SimplifyCFG(BB) | true;
Devang Pateld0a203d2009-02-04 21:39:48 +00002035 } else if (&*I == cast<Instruction>(BI->getCondition())){
2036 ++I;
2037 // Ignore dbg intrinsics.
2038 while (isa<DbgInfoIntrinsic>(I))
2039 ++I;
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002040 if (&*I == BI && FoldValueComparisonIntoPredecessors(BI))
2041 return SimplifyCFG(BB) | true;
Devang Pateld0a203d2009-02-04 21:39:48 +00002042 }
Chris Lattnere67fa052004-05-01 23:35:43 +00002043 }
Devang Pateld0a203d2009-02-04 21:39:48 +00002044
Chris Lattnereaba3a12005-09-19 23:49:37 +00002045 // If this is a branch on a phi node in the current block, thread control
2046 // through this block if any PHI node entries are constants.
2047 if (PHINode *PN = dyn_cast<PHINode>(BI->getCondition()))
2048 if (PN->getParent() == BI->getParent())
2049 if (FoldCondBranchOnPHI(BI))
2050 return SimplifyCFG(BB) | true;
Chris Lattnere67fa052004-05-01 23:35:43 +00002051
2052 // If this basic block is ONLY a setcc and a branch, and if a predecessor
2053 // branches to us and one of our successors, fold the setcc into the
2054 // predecessor and use logical operations to pick the right destination.
Chris Lattner1347e872008-07-13 21:12:01 +00002055 if (FoldBranchToCommonDest(BI))
Bill Wendlingc69b4a52010-03-14 10:40:28 +00002056 return SimplifyCFG(BB) | true;
Chris Lattnere67fa052004-05-01 23:35:43 +00002057
Chris Lattner867661a2008-07-13 21:53:26 +00002058
2059 // Scan predecessor blocks for conditional branches.
Chris Lattner2e42e362005-09-20 00:43:16 +00002060 for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI)
2061 if (BranchInst *PBI = dyn_cast<BranchInst>((*PI)->getTerminator()))
Chris Lattner867661a2008-07-13 21:53:26 +00002062 if (PBI != BI && PBI->isConditional())
2063 if (SimplifyCondBranchToCondBranch(PBI, BI))
2064 return SimplifyCFG(BB) | true;
Chris Lattnercd4b7092010-12-13 01:57:34 +00002065
2066
2067 // Change br (X == 0 | X == 1), T, F into a switch instruction.
2068 // If this is a bunch of seteq's or'd together, or if it's a bunch of
2069 // 'setne's and'ed together, collect them.
2070 Value *CompVal = 0;
2071 std::vector<ConstantInt*> Values;
Chris Lattner803a29d2010-12-13 04:15:19 +00002072 bool TrueWhenEqual = true;
2073
2074 if (Instruction *Cond = dyn_cast<Instruction>(BI->getCondition())) {
2075 if (Cond->getOpcode() == Instruction::Or) {
2076 CompVal = GatherConstantSetEQs(Cond, Values, TD);
2077 } else if (Cond->getOpcode() == Instruction::And) {
2078 CompVal = GatherConstantSetNEs(Cond, Values, TD);
2079 TrueWhenEqual = false;
2080 }
2081 }
2082
Chris Lattnercd4b7092010-12-13 01:57:34 +00002083 if (CompVal) {
2084 // There might be duplicate constants in the list, which the switch
2085 // instruction can't handle, remove them now.
Chris Lattner6d4d21e2010-12-13 02:00:58 +00002086 array_pod_sort(Values.begin(), Values.end(), ConstantIntSortPredicate);
Chris Lattnercd4b7092010-12-13 01:57:34 +00002087 Values.erase(std::unique(Values.begin(), Values.end()), Values.end());
2088
2089 // Figure out which block is which destination.
2090 BasicBlock *DefaultBB = BI->getSuccessor(1);
2091 BasicBlock *EdgeBB = BI->getSuccessor(0);
2092 if (!TrueWhenEqual) std::swap(DefaultBB, EdgeBB);
2093
2094 // Convert pointer to int before we switch.
2095 if (CompVal->getType()->isPointerTy()) {
2096 assert(TD && "Cannot switch on pointer without TargetData");
2097 CompVal = new PtrToIntInst(CompVal,
2098 TD->getIntPtrType(CompVal->getContext()),
2099 "magicptr", BI);
2100 }
2101
2102 // Create the new switch instruction now.
2103 SwitchInst *New = SwitchInst::Create(CompVal, DefaultBB,
2104 Values.size(), BI);
2105
2106 // Add all of the 'cases' to the switch instruction.
2107 for (unsigned i = 0, e = Values.size(); i != e; ++i)
2108 New->addCase(Values[i], EdgeBB);
2109
2110 // We added edges from PI to the EdgeBB. As such, if there were any
2111 // PHI nodes in EdgeBB, they need entries to be added corresponding to
2112 // the number of edges added.
2113 for (BasicBlock::iterator BBI = EdgeBB->begin();
2114 isa<PHINode>(BBI); ++BBI) {
2115 PHINode *PN = cast<PHINode>(BBI);
2116 Value *InVal = PN->getIncomingValueForBlock(BB);
2117 for (unsigned i = 0, e = Values.size()-1; i != e; ++i)
2118 PN->addIncoming(InVal, BB);
2119 }
2120
2121 // Erase the old branch instruction.
2122 EraseTerminatorInstAndDCECond(BI);
2123 return true;
2124 }
Chris Lattnerd52c2612004-02-24 07:23:58 +00002125 }
Chris Lattner698f96f2004-10-18 04:07:22 +00002126 } else if (isa<UnreachableInst>(BB->getTerminator())) {
2127 // If there are any instructions immediately before the unreachable that can
2128 // be removed, do so.
2129 Instruction *Unreachable = BB->getTerminator();
2130 while (Unreachable != BB->begin()) {
2131 BasicBlock::iterator BBI = Unreachable;
2132 --BBI;
Chris Lattnerf8131c92008-10-29 17:46:26 +00002133 // Do not delete instructions that can have side effects, like calls
2134 // (which may never return) and volatile loads and stores.
Dale Johannesen80b8a622009-03-12 17:42:45 +00002135 if (isa<CallInst>(BBI) && !isa<DbgInfoIntrinsic>(BBI)) break;
Chris Lattnerf8131c92008-10-29 17:46:26 +00002136
2137 if (StoreInst *SI = dyn_cast<StoreInst>(BBI))
2138 if (SI->isVolatile())
2139 break;
2140
2141 if (LoadInst *LI = dyn_cast<LoadInst>(BBI))
2142 if (LI->isVolatile())
2143 break;
2144
Chris Lattner698f96f2004-10-18 04:07:22 +00002145 // Delete this instruction
2146 BB->getInstList().erase(BBI);
2147 Changed = true;
2148 }
2149
2150 // If the unreachable instruction is the first in the block, take a gander
2151 // at all of the predecessors of this instruction, and simplify them.
2152 if (&BB->front() == Unreachable) {
Chris Lattner82442432008-02-18 07:42:56 +00002153 SmallVector<BasicBlock*, 8> Preds(pred_begin(BB), pred_end(BB));
Chris Lattner698f96f2004-10-18 04:07:22 +00002154 for (unsigned i = 0, e = Preds.size(); i != e; ++i) {
2155 TerminatorInst *TI = Preds[i]->getTerminator();
2156
2157 if (BranchInst *BI = dyn_cast<BranchInst>(TI)) {
2158 if (BI->isUnconditional()) {
2159 if (BI->getSuccessor(0) == BB) {
Owen Anderson1d0be152009-08-13 21:58:54 +00002160 new UnreachableInst(TI->getContext(), TI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002161 TI->eraseFromParent();
2162 Changed = true;
2163 }
2164 } else {
2165 if (BI->getSuccessor(0) == BB) {
Gabor Greif051a9502008-04-06 20:25:17 +00002166 BranchInst::Create(BI->getSuccessor(1), BI);
Eli Friedman080efb82008-12-16 20:54:32 +00002167 EraseTerminatorInstAndDCECond(BI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002168 } else if (BI->getSuccessor(1) == BB) {
Gabor Greif051a9502008-04-06 20:25:17 +00002169 BranchInst::Create(BI->getSuccessor(0), BI);
Eli Friedman080efb82008-12-16 20:54:32 +00002170 EraseTerminatorInstAndDCECond(BI);
Chris Lattner698f96f2004-10-18 04:07:22 +00002171 Changed = true;
2172 }
2173 }
2174 } else if (SwitchInst *SI = dyn_cast<SwitchInst>(TI)) {
2175 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2176 if (SI->getSuccessor(i) == BB) {
Chris Lattner42eb7522005-05-20 22:19:54 +00002177 BB->removePredecessor(SI->getParent());
Chris Lattner698f96f2004-10-18 04:07:22 +00002178 SI->removeCase(i);
2179 --i; --e;
2180 Changed = true;
2181 }
2182 // If the default value is unreachable, figure out the most popular
2183 // destination and make it the default.
2184 if (SI->getSuccessor(0) == BB) {
2185 std::map<BasicBlock*, unsigned> Popularity;
2186 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2187 Popularity[SI->getSuccessor(i)]++;
2188
2189 // Find the most popular block.
2190 unsigned MaxPop = 0;
2191 BasicBlock *MaxBlock = 0;
2192 for (std::map<BasicBlock*, unsigned>::iterator
2193 I = Popularity.begin(), E = Popularity.end(); I != E; ++I) {
2194 if (I->second > MaxPop) {
2195 MaxPop = I->second;
2196 MaxBlock = I->first;
2197 }
2198 }
2199 if (MaxBlock) {
2200 // Make this the new default, allowing us to delete any explicit
2201 // edges to it.
2202 SI->setSuccessor(0, MaxBlock);
2203 Changed = true;
2204
Chris Lattner42eb7522005-05-20 22:19:54 +00002205 // If MaxBlock has phinodes in it, remove MaxPop-1 entries from
2206 // it.
2207 if (isa<PHINode>(MaxBlock->begin()))
2208 for (unsigned i = 0; i != MaxPop-1; ++i)
2209 MaxBlock->removePredecessor(SI->getParent());
2210
Chris Lattner698f96f2004-10-18 04:07:22 +00002211 for (unsigned i = 1, e = SI->getNumCases(); i != e; ++i)
2212 if (SI->getSuccessor(i) == MaxBlock) {
2213 SI->removeCase(i);
2214 --i; --e;
2215 }
2216 }
2217 }
2218 } else if (InvokeInst *II = dyn_cast<InvokeInst>(TI)) {
2219 if (II->getUnwindDest() == BB) {
2220 // Convert the invoke to a call instruction. This would be a good
2221 // place to note that the call does not throw though.
Gabor Greif051a9502008-04-06 20:25:17 +00002222 BranchInst *BI = BranchInst::Create(II->getNormalDest(), II);
Chris Lattner698f96f2004-10-18 04:07:22 +00002223 II->removeFromParent(); // Take out of symbol table
Misha Brukmanfd939082005-04-21 23:48:37 +00002224
Chris Lattner698f96f2004-10-18 04:07:22 +00002225 // Insert the call now...
Gabor Greifbd443142010-03-30 19:20:53 +00002226 SmallVector<Value*, 8> Args(II->op_begin(), II->op_end()-3);
Gabor Greif051a9502008-04-06 20:25:17 +00002227 CallInst *CI = CallInst::Create(II->getCalledValue(),
2228 Args.begin(), Args.end(),
2229 II->getName(), BI);
Chris Lattner16d0db22005-05-14 12:21:56 +00002230 CI->setCallingConv(II->getCallingConv());
Devang Patel05988662008-09-25 21:00:45 +00002231 CI->setAttributes(II->getAttributes());
Gabor Greifbd443142010-03-30 19:20:53 +00002232 // If the invoke produced a value, the call does now instead.
Chris Lattner698f96f2004-10-18 04:07:22 +00002233 II->replaceAllUsesWith(CI);
2234 delete II;
2235 Changed = true;
2236 }
2237 }
2238 }
2239
2240 // If this block is now dead, remove it.
Duncan Sands5f284752010-10-24 12:23:30 +00002241 if (pred_begin(BB) == pred_end(BB) && BB != &Fn->getEntryBlock()) {
Chris Lattner698f96f2004-10-18 04:07:22 +00002242 // We know there are no successors, so just nuke the block.
Duncan Sands5f284752010-10-24 12:23:30 +00002243 Fn->getBasicBlockList().erase(BB);
Chris Lattner698f96f2004-10-18 04:07:22 +00002244 return true;
2245 }
2246 }
Dan Gohman7a499432010-08-16 14:41:14 +00002247 } else if (IndirectBrInst *IBI =
2248 dyn_cast<IndirectBrInst>(BB->getTerminator())) {
Dan Gohmane2c6d132010-08-14 00:29:42 +00002249 // Eliminate redundant destinations.
2250 SmallPtrSet<Value *, 8> Succs;
2251 for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) {
2252 BasicBlock *Dest = IBI->getDestination(i);
Dan Gohman7a499432010-08-16 14:41:14 +00002253 if (!Dest->hasAddressTaken() || !Succs.insert(Dest)) {
Dan Gohmane2c6d132010-08-14 00:29:42 +00002254 Dest->removePredecessor(BB);
2255 IBI->removeDestination(i);
2256 --i; --e;
2257 Changed = true;
2258 }
2259 }
2260
2261 if (IBI->getNumDestinations() == 0) {
2262 // If the indirectbr has no successors, change it to unreachable.
2263 new UnreachableInst(IBI->getContext(), IBI);
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002264 EraseTerminatorInstAndDCECond(IBI);
Dan Gohmane2c6d132010-08-14 00:29:42 +00002265 Changed = true;
2266 } else if (IBI->getNumDestinations() == 1) {
2267 // If the indirectbr has one successor, change it to a direct branch.
2268 BranchInst::Create(IBI->getDestination(0), IBI);
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002269 EraseTerminatorInstAndDCECond(IBI);
Dan Gohmane2c6d132010-08-14 00:29:42 +00002270 Changed = true;
Frits van Bommel7ac40c32010-12-05 18:29:03 +00002271 } else if (SelectInst *SI = dyn_cast<SelectInst>(IBI->getAddress())) {
2272 if (SimplifyIndirectBrOnSelect(IBI, SI))
2273 return SimplifyCFG(BB) | true;
Dan Gohmane2c6d132010-08-14 00:29:42 +00002274 }
Chris Lattner19831ec2004-02-16 06:35:48 +00002275 }
2276
Chris Lattner01d1ee32002-05-21 20:50:24 +00002277 // Merge basic blocks into their predecessor if there is only one distinct
2278 // pred, and if there is only one distinct successor of the predecessor, and
2279 // if there are no PHI nodes.
2280 //
Owen Andersoncfa94192008-07-18 17:49:43 +00002281 if (MergeBlockIntoPredecessor(BB))
2282 return true;
2283
2284 // Otherwise, if this block only has a single predecessor, and if that block
2285 // is a conditional branch, see if we can hoist any code from this block up
2286 // into our predecessor.
Chris Lattner2355f942004-02-11 01:17:07 +00002287 pred_iterator PI(pred_begin(BB)), PE(pred_end(BB));
Dan Gohmane2c6d132010-08-14 00:29:42 +00002288 BasicBlock *OnlyPred = 0;
2289 for (; PI != PE; ++PI) { // Search all predecessors, see if they are all same
2290 if (!OnlyPred)
2291 OnlyPred = *PI;
2292 else if (*PI != OnlyPred) {
Chris Lattner2355f942004-02-11 01:17:07 +00002293 OnlyPred = 0; // There are multiple different predecessors...
2294 break;
2295 }
Dan Gohmane2c6d132010-08-14 00:29:42 +00002296 }
Owen Andersoncfa94192008-07-18 17:49:43 +00002297
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002298 if (OnlyPred) {
2299 BranchInst *BI = dyn_cast<BranchInst>(OnlyPred->getTerminator());
2300 if (BI && BI->isConditional()) {
2301 // Get the other block.
2302 BasicBlock *OtherBB = BI->getSuccessor(BI->getSuccessor(0) == BB);
2303 PI = pred_begin(OtherBB);
2304 ++PI;
2305
2306 if (PI == pred_end(OtherBB)) {
2307 // We have a conditional branch to two blocks that are only reachable
2308 // from the condbr. We know that the condbr dominates the two blocks,
2309 // so see if there is any identical code in the "then" and "else"
2310 // blocks. If so, we can hoist it up to the branching block.
2311 Changed |= HoistThenElseCodeToIf(BI);
2312 } else {
2313 BasicBlock* OnlySucc = NULL;
2314 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB);
2315 SI != SE; ++SI) {
2316 if (!OnlySucc)
2317 OnlySucc = *SI;
2318 else if (*SI != OnlySucc) {
2319 OnlySucc = 0; // There are multiple distinct successors!
2320 break;
Evan Cheng4d09efd2008-06-07 08:52:29 +00002321 }
Chris Lattner76134372004-12-10 17:42:31 +00002322 }
Chris Lattner37dc9382004-11-30 00:29:14 +00002323
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002324 if (OnlySucc == OtherBB) {
2325 // If BB's only successor is the other successor of the predecessor,
2326 // i.e. a triangle, see if we can hoist any code from this block up
2327 // to the "if" block.
2328 Changed |= SpeculativelyExecuteBB(BI, BB);
Chris Lattner0d560082004-02-24 05:38:11 +00002329 }
2330 }
Chris Lattner9a2b72a2010-12-13 01:47:07 +00002331 }
2332 }
2333
Chris Lattner694e37f2003-08-17 19:41:53 +00002334 return Changed;
Chris Lattner01d1ee32002-05-21 20:50:24 +00002335}
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00002336
2337/// SimplifyCFG - This function is used to do simplification of a CFG. For
2338/// example, it adjusts branches to branches to eliminate the extra hop, it
2339/// eliminates unreachable basic blocks, and does other "peephole" optimization
2340/// of the CFG. It returns true if a modification was made.
2341///
Jakob Stoklund Olesen58e9ee82010-02-05 22:03:18 +00002342bool llvm::SimplifyCFG(BasicBlock *BB, const TargetData *TD) {
2343 return SimplifyCFGOpt(TD).run(BB);
2344}