blob: 98147ab88e5224e5241602ecb65adc57b601150d [file] [log] [blame]
Chris Lattner4d1e46e2002-05-07 18:07:59 +00001//===-- Local.cpp - Functions to perform local transformations ------------===//
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 Lattner4d1e46e2002-05-07 18:07:59 +00009//
10// This family of functions perform various local transformations to the
11// program.
12//
13//===----------------------------------------------------------------------===//
14
15#include "llvm/Transforms/Utils/Local.h"
Chandler Carruthd04a8d42012-12-03 16:50:05 +000016#include "llvm/ADT/DenseMap.h"
Evgeniy Stepanov3333e662012-12-21 11:18:49 +000017#include "llvm/ADT/STLExtras.h"
Chandler Carruth58a2cbe2013-01-02 10:22:59 +000018#include "llvm/ADT/SmallPtrSet.h"
Chandler Carruthd04a8d42012-12-03 16:50:05 +000019#include "llvm/Analysis/Dominators.h"
20#include "llvm/Analysis/InstructionSimplify.h"
21#include "llvm/Analysis/MemoryBuiltins.h"
22#include "llvm/Analysis/ProfileInfo.h"
23#include "llvm/Analysis/ValueTracking.h"
Chandler Carruth06cb8ed2012-06-29 12:38:19 +000024#include "llvm/DIBuilder.h"
Bill Wendling0bcbd1d2012-06-28 00:05:13 +000025#include "llvm/DebugInfo.h"
Chandler Carruth0b8c9a82013-01-02 11:36:10 +000026#include "llvm/IR/Constants.h"
27#include "llvm/IR/DataLayout.h"
28#include "llvm/IR/DerivedTypes.h"
29#include "llvm/IR/GlobalAlias.h"
30#include "llvm/IR/GlobalVariable.h"
31#include "llvm/IR/IRBuilder.h"
32#include "llvm/IR/Instructions.h"
33#include "llvm/IR/IntrinsicInst.h"
34#include "llvm/IR/Intrinsics.h"
35#include "llvm/IR/MDBuilder.h"
36#include "llvm/IR/Metadata.h"
37#include "llvm/IR/Operator.h"
Chris Lattnerdce94d92009-11-10 05:59:26 +000038#include "llvm/Support/CFG.h"
39#include "llvm/Support/Debug.h"
Chris Lattnerc5f52e62005-09-26 05:27:10 +000040#include "llvm/Support/GetElementPtrTypeIterator.h"
41#include "llvm/Support/MathExtras.h"
Chris Lattner19f2dc42009-12-29 09:12:29 +000042#include "llvm/Support/ValueHandle.h"
Chris Lattnerdce94d92009-11-10 05:59:26 +000043#include "llvm/Support/raw_ostream.h"
Chris Lattnerabbc2dd2003-12-19 05:56:28 +000044using namespace llvm;
Brian Gaeked0fde302003-11-11 22:41:34 +000045
Chris Lattner4d1e46e2002-05-07 18:07:59 +000046//===----------------------------------------------------------------------===//
Chris Lattner3481f242008-11-27 22:57:53 +000047// Local constant propagation.
Chris Lattner4d1e46e2002-05-07 18:07:59 +000048//
49
Frits van Bommel5649ba72011-05-22 16:24:18 +000050/// ConstantFoldTerminator - If a terminator instruction is predicated on a
51/// constant value, convert it into an unconditional branch to the constant
52/// destination. This is a nontrivial operation because the successors of this
53/// basic block must have their PHI nodes updated.
54/// Also calls RecursivelyDeleteTriviallyDeadInstructions() on any branch/switch
55/// conditions and indirectbr addresses this might make dead if
56/// DeleteDeadConditions is true.
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +000057bool llvm::ConstantFoldTerminator(BasicBlock *BB, bool DeleteDeadConditions,
58 const TargetLibraryInfo *TLI) {
Chris Lattner76ae3442002-05-21 20:04:50 +000059 TerminatorInst *T = BB->getTerminator();
Devang Patel62fb3552011-05-18 17:26:46 +000060 IRBuilder<> Builder(T);
Misha Brukmanfd939082005-04-21 23:48:37 +000061
Chris Lattner4d1e46e2002-05-07 18:07:59 +000062 // Branch - See if we are conditional jumping on constant
63 if (BranchInst *BI = dyn_cast<BranchInst>(T)) {
64 if (BI->isUnconditional()) return false; // Can't optimize uncond branch
Gabor Greifc1bb13f2009-01-30 18:21:13 +000065 BasicBlock *Dest1 = BI->getSuccessor(0);
66 BasicBlock *Dest2 = BI->getSuccessor(1);
Chris Lattner4d1e46e2002-05-07 18:07:59 +000067
Zhou Sheng6b6b6ef2007-01-11 12:24:14 +000068 if (ConstantInt *Cond = dyn_cast<ConstantInt>(BI->getCondition())) {
Chris Lattner4d1e46e2002-05-07 18:07:59 +000069 // Are we branching on constant?
70 // YES. Change to unconditional branch...
Reid Spencer579dca12007-01-12 04:24:46 +000071 BasicBlock *Destination = Cond->getZExtValue() ? Dest1 : Dest2;
72 BasicBlock *OldDest = Cond->getZExtValue() ? Dest2 : Dest1;
Chris Lattner4d1e46e2002-05-07 18:07:59 +000073
Misha Brukmanfd939082005-04-21 23:48:37 +000074 //cerr << "Function: " << T->getParent()->getParent()
75 // << "\nRemoving branch from " << T->getParent()
Chris Lattner4d1e46e2002-05-07 18:07:59 +000076 // << "\n\nTo: " << OldDest << endl;
77
78 // Let the basic block know that we are letting go of it. Based on this,
79 // it will adjust it's PHI nodes.
Jay Foad1a039022011-04-19 15:23:29 +000080 OldDest->removePredecessor(BB);
Chris Lattner4d1e46e2002-05-07 18:07:59 +000081
Jay Foad8f9ffbd2011-01-07 20:25:56 +000082 // Replace the conditional branch with an unconditional one.
Devang Patel62fb3552011-05-18 17:26:46 +000083 Builder.CreateBr(Destination);
Jay Foad8f9ffbd2011-01-07 20:25:56 +000084 BI->eraseFromParent();
Chris Lattner4d1e46e2002-05-07 18:07:59 +000085 return true;
Chris Lattner0a4c6782009-11-01 03:40:38 +000086 }
Jakub Staszaka18c5742013-07-22 23:16:36 +000087
Chris Lattner0a4c6782009-11-01 03:40:38 +000088 if (Dest2 == Dest1) { // Conditional branch to same location?
Misha Brukmanfd939082005-04-21 23:48:37 +000089 // This branch matches something like this:
Chris Lattner4d1e46e2002-05-07 18:07:59 +000090 // br bool %cond, label %Dest, label %Dest
91 // and changes it into: br label %Dest
92
93 // Let the basic block know that we are letting go of one copy of it.
94 assert(BI->getParent() && "Terminator not inserted in block!");
95 Dest1->removePredecessor(BI->getParent());
96
Jay Foad8f9ffbd2011-01-07 20:25:56 +000097 // Replace the conditional branch with an unconditional one.
Devang Patel62fb3552011-05-18 17:26:46 +000098 Builder.CreateBr(Dest1);
Frits van Bommel5649ba72011-05-22 16:24:18 +000099 Value *Cond = BI->getCondition();
Jay Foad8f9ffbd2011-01-07 20:25:56 +0000100 BI->eraseFromParent();
Frits van Bommel5649ba72011-05-22 16:24:18 +0000101 if (DeleteDeadConditions)
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000102 RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI);
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000103 return true;
104 }
Chris Lattner0a4c6782009-11-01 03:40:38 +0000105 return false;
106 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000107
Chris Lattner0a4c6782009-11-01 03:40:38 +0000108 if (SwitchInst *SI = dyn_cast<SwitchInst>(T)) {
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000109 // If we are switching on a constant, we can convert the switch into a
110 // single branch instruction!
111 ConstantInt *CI = dyn_cast<ConstantInt>(SI->getCondition());
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000112 BasicBlock *TheOnlyDest = SI->getDefaultDest();
Chris Lattner7d6c24c2003-08-23 23:18:19 +0000113 BasicBlock *DefaultDest = TheOnlyDest;
Chris Lattner694e37f2003-08-17 19:41:53 +0000114
Chris Lattner0a4c6782009-11-01 03:40:38 +0000115 // Figure out which case it goes to.
Stepan Dyatkovskiy3d3abe02012-03-11 06:09:17 +0000116 for (SwitchInst::CaseIt i = SI->case_begin(), e = SI->case_end();
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000117 i != e; ++i) {
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000118 // Found case matching a constant operand?
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000119 if (i.getCaseValue() == CI) {
120 TheOnlyDest = i.getCaseSuccessor();
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000121 break;
122 }
Chris Lattner694e37f2003-08-17 19:41:53 +0000123
Chris Lattner7d6c24c2003-08-23 23:18:19 +0000124 // Check to see if this branch is going to the same place as the default
125 // dest. If so, eliminate it as an explicit compare.
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000126 if (i.getCaseSuccessor() == DefaultDest) {
Manman Renee99c7f2012-09-12 17:04:11 +0000127 MDNode* MD = SI->getMetadata(LLVMContext::MD_prof);
128 // MD should have 2 + NumCases operands.
129 if (MD && MD->getNumOperands() == 2 + SI->getNumCases()) {
130 // Collect branch weights into a vector.
131 SmallVector<uint32_t, 8> Weights;
132 for (unsigned MD_i = 1, MD_e = MD->getNumOperands(); MD_i < MD_e;
133 ++MD_i) {
134 ConstantInt* CI = dyn_cast<ConstantInt>(MD->getOperand(MD_i));
135 assert(CI);
136 Weights.push_back(CI->getValue().getZExtValue());
137 }
138 // Merge weight of this case to the default weight.
139 unsigned idx = i.getCaseIndex();
140 Weights[0] += Weights[idx+1];
141 // Remove weight for this case.
142 std::swap(Weights[idx+1], Weights.back());
143 Weights.pop_back();
144 SI->setMetadata(LLVMContext::MD_prof,
145 MDBuilder(BB->getContext()).
146 createBranchWeights(Weights));
147 }
Chris Lattner0a4c6782009-11-01 03:40:38 +0000148 // Remove this entry.
Chris Lattner7d6c24c2003-08-23 23:18:19 +0000149 DefaultDest->removePredecessor(SI->getParent());
150 SI->removeCase(i);
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000151 --i; --e;
Chris Lattner7d6c24c2003-08-23 23:18:19 +0000152 continue;
153 }
154
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000155 // Otherwise, check to see if the switch only branches to one destination.
156 // We do this by reseting "TheOnlyDest" to null when we find two non-equal
157 // destinations.
Stepan Dyatkovskiyc10fa6c2012-03-08 07:06:20 +0000158 if (i.getCaseSuccessor() != TheOnlyDest) TheOnlyDest = 0;
Chris Lattner694e37f2003-08-17 19:41:53 +0000159 }
160
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000161 if (CI && !TheOnlyDest) {
162 // Branching on a constant, but not any of the cases, go to the default
163 // successor.
164 TheOnlyDest = SI->getDefaultDest();
165 }
166
167 // If we found a single destination that we can fold the switch into, do so
168 // now.
169 if (TheOnlyDest) {
Chris Lattner0a4c6782009-11-01 03:40:38 +0000170 // Insert the new branch.
Devang Patel62fb3552011-05-18 17:26:46 +0000171 Builder.CreateBr(TheOnlyDest);
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000172 BasicBlock *BB = SI->getParent();
173
174 // Remove entries from PHI nodes which we no longer branch to...
175 for (unsigned i = 0, e = SI->getNumSuccessors(); i != e; ++i) {
176 // Found case matching a constant operand?
177 BasicBlock *Succ = SI->getSuccessor(i);
178 if (Succ == TheOnlyDest)
179 TheOnlyDest = 0; // Don't modify the first branch to TheOnlyDest
180 else
181 Succ->removePredecessor(BB);
182 }
183
Chris Lattner0a4c6782009-11-01 03:40:38 +0000184 // Delete the old switch.
Frits van Bommel5649ba72011-05-22 16:24:18 +0000185 Value *Cond = SI->getCondition();
186 SI->eraseFromParent();
187 if (DeleteDeadConditions)
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000188 RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI);
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000189 return true;
Chris Lattner0a4c6782009-11-01 03:40:38 +0000190 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000191
Stepan Dyatkovskiy24473122012-02-01 07:49:51 +0000192 if (SI->getNumCases() == 1) {
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000193 // Otherwise, we can fold this switch into a conditional branch
194 // instruction if it has only one non-default destination.
Stepan Dyatkovskiy3d3abe02012-03-11 06:09:17 +0000195 SwitchInst::CaseIt FirstCase = SI->case_begin();
Stepan Dyatkovskiy47cbc4e2012-06-23 10:58:58 +0000196 IntegersSubset& Case = FirstCase.getCaseValueEx();
197 if (Case.isSingleNumber()) {
Stepan Dyatkovskiy484fc932012-05-28 12:39:09 +0000198 // FIXME: Currently work with ConstantInt based numbers.
Stepan Dyatkovskiya2067fb2012-05-23 08:18:26 +0000199 Value *Cond = Builder.CreateICmpEQ(SI->getCondition(),
Stepan Dyatkovskiy47cbc4e2012-06-23 10:58:58 +0000200 Case.getSingleNumber(0).toConstantInt(),
Stepan Dyatkovskiy484fc932012-05-28 12:39:09 +0000201 "cond");
Devang Patel62fb3552011-05-18 17:26:46 +0000202
Stepan Dyatkovskiya2067fb2012-05-23 08:18:26 +0000203 // Insert the new branch.
Manman Renad289072012-09-14 21:53:06 +0000204 BranchInst *NewBr = Builder.CreateCondBr(Cond,
205 FirstCase.getCaseSuccessor(),
206 SI->getDefaultDest());
207 MDNode* MD = SI->getMetadata(LLVMContext::MD_prof);
208 if (MD && MD->getNumOperands() == 3) {
209 ConstantInt *SICase = dyn_cast<ConstantInt>(MD->getOperand(2));
210 ConstantInt *SIDef = dyn_cast<ConstantInt>(MD->getOperand(1));
211 assert(SICase && SIDef);
212 // The TrueWeight should be the weight for the single case of SI.
213 NewBr->setMetadata(LLVMContext::MD_prof,
214 MDBuilder(BB->getContext()).
215 createBranchWeights(SICase->getValue().getZExtValue(),
216 SIDef->getValue().getZExtValue()));
217 }
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000218
Stepan Dyatkovskiya2067fb2012-05-23 08:18:26 +0000219 // Delete the old switch.
220 SI->eraseFromParent();
221 return true;
Stepan Dyatkovskiya2067fb2012-05-23 08:18:26 +0000222 }
Chris Lattner10b1f5a2003-08-17 20:21:14 +0000223 }
Chris Lattner0a4c6782009-11-01 03:40:38 +0000224 return false;
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000225 }
Chris Lattner0a4c6782009-11-01 03:40:38 +0000226
227 if (IndirectBrInst *IBI = dyn_cast<IndirectBrInst>(T)) {
228 // indirectbr blockaddress(@F, @BB) -> br label @BB
229 if (BlockAddress *BA =
230 dyn_cast<BlockAddress>(IBI->getAddress()->stripPointerCasts())) {
231 BasicBlock *TheOnlyDest = BA->getBasicBlock();
232 // Insert the new branch.
Devang Patel62fb3552011-05-18 17:26:46 +0000233 Builder.CreateBr(TheOnlyDest);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000234
Chris Lattner0a4c6782009-11-01 03:40:38 +0000235 for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) {
236 if (IBI->getDestination(i) == TheOnlyDest)
237 TheOnlyDest = 0;
238 else
239 IBI->getDestination(i)->removePredecessor(IBI->getParent());
240 }
Frits van Bommel5649ba72011-05-22 16:24:18 +0000241 Value *Address = IBI->getAddress();
Chris Lattner0a4c6782009-11-01 03:40:38 +0000242 IBI->eraseFromParent();
Frits van Bommel5649ba72011-05-22 16:24:18 +0000243 if (DeleteDeadConditions)
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000244 RecursivelyDeleteTriviallyDeadInstructions(Address, TLI);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000245
Chris Lattner0a4c6782009-11-01 03:40:38 +0000246 // If we didn't find our destination in the IBI successor list, then we
247 // have undefined behavior. Replace the unconditional branch with an
248 // 'unreachable' instruction.
249 if (TheOnlyDest) {
250 BB->getTerminator()->eraseFromParent();
251 new UnreachableInst(BB->getContext(), BB);
252 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000253
Chris Lattner0a4c6782009-11-01 03:40:38 +0000254 return true;
255 }
256 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000257
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000258 return false;
259}
260
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000261
262//===----------------------------------------------------------------------===//
Chris Lattner40d8c282009-11-10 22:26:15 +0000263// Local dead code elimination.
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000264//
265
Chris Lattner3481f242008-11-27 22:57:53 +0000266/// isInstructionTriviallyDead - Return true if the result produced by the
267/// instruction is not used, and the instruction has no side effects.
268///
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000269bool llvm::isInstructionTriviallyDead(Instruction *I,
270 const TargetLibraryInfo *TLI) {
Chris Lattnerec710c52005-05-06 05:27:34 +0000271 if (!I->use_empty() || isa<TerminatorInst>(I)) return false;
Jeff Cohen00b168892005-07-27 06:12:32 +0000272
Bill Wendling187b1922011-08-15 20:10:51 +0000273 // We don't want the landingpad instruction removed by anything this general.
274 if (isa<LandingPadInst>(I))
275 return false;
276
Devang Patel9c5822a2011-03-18 23:28:02 +0000277 // We don't want debug info removed by anything this general, unless
278 // debug info is empty.
279 if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(I)) {
Nick Lewycky3e69c132011-08-02 21:19:27 +0000280 if (DDI->getAddress())
Devang Patel9c5822a2011-03-18 23:28:02 +0000281 return false;
Devang Patelb9946212011-03-21 22:04:45 +0000282 return true;
Nick Lewycky3e69c132011-08-02 21:19:27 +0000283 }
Devang Patelb9946212011-03-21 22:04:45 +0000284 if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(I)) {
Devang Patel9c5822a2011-03-18 23:28:02 +0000285 if (DVI->getValue())
286 return false;
Devang Patelb9946212011-03-21 22:04:45 +0000287 return true;
Devang Patel9c5822a2011-03-18 23:28:02 +0000288 }
289
Duncan Sands7af1c782009-05-06 06:49:50 +0000290 if (!I->mayHaveSideEffects()) return true;
291
292 // Special case intrinsics that "may have side effects" but can be deleted
293 // when dead.
Nick Lewycky3e69c132011-08-02 21:19:27 +0000294 if (IntrinsicInst *II = dyn_cast<IntrinsicInst>(I)) {
Chris Lattner741c0ae2007-12-29 00:59:12 +0000295 // Safe to delete llvm.stacksave if dead.
296 if (II->getIntrinsicID() == Intrinsic::stacksave)
297 return true;
Nick Lewycky3e69c132011-08-02 21:19:27 +0000298
299 // Lifetime intrinsics are dead when their right-hand is undef.
300 if (II->getIntrinsicID() == Intrinsic::lifetime_start ||
301 II->getIntrinsicID() == Intrinsic::lifetime_end)
302 return isa<UndefValue>(II->getArgOperand(1));
303 }
Nick Lewycky4a3935c2011-10-24 04:35:36 +0000304
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000305 if (isAllocLikeFn(I, TLI)) return true;
Nick Lewycky4a3935c2011-10-24 04:35:36 +0000306
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000307 if (CallInst *CI = isFreeCall(I, TLI))
Nick Lewycky4a3935c2011-10-24 04:35:36 +0000308 if (Constant *C = dyn_cast<Constant>(CI->getArgOperand(0)))
309 return C->isNullValue() || isa<UndefValue>(C);
310
Chris Lattnerec710c52005-05-06 05:27:34 +0000311 return false;
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000312}
313
Chris Lattner3481f242008-11-27 22:57:53 +0000314/// RecursivelyDeleteTriviallyDeadInstructions - If the specified value is a
315/// trivially dead instruction, delete it. If that makes any of its operands
Dan Gohman90fe0bd2010-01-05 15:45:31 +0000316/// trivially dead, delete them too, recursively. Return true if any
317/// instructions were deleted.
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000318bool
319llvm::RecursivelyDeleteTriviallyDeadInstructions(Value *V,
320 const TargetLibraryInfo *TLI) {
Chris Lattner3481f242008-11-27 22:57:53 +0000321 Instruction *I = dyn_cast<Instruction>(V);
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000322 if (!I || !I->use_empty() || !isInstructionTriviallyDead(I, TLI))
Dan Gohman90fe0bd2010-01-05 15:45:31 +0000323 return false;
Jakub Staszaka18c5742013-07-22 23:16:36 +0000324
Chris Lattner76057302008-11-28 01:20:46 +0000325 SmallVector<Instruction*, 16> DeadInsts;
326 DeadInsts.push_back(I);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000327
Dan Gohman321a8132010-01-05 16:27:25 +0000328 do {
Dan Gohmane9d87f42009-05-06 17:22:41 +0000329 I = DeadInsts.pop_back_val();
Chris Lattner28721772008-11-28 00:58:15 +0000330
Chris Lattner76057302008-11-28 01:20:46 +0000331 // Null out all of the instruction's operands to see if any operand becomes
332 // dead as we go.
333 for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) {
334 Value *OpV = I->getOperand(i);
335 I->setOperand(i, 0);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000336
Chris Lattner76057302008-11-28 01:20:46 +0000337 if (!OpV->use_empty()) continue;
Jakub Staszaka18c5742013-07-22 23:16:36 +0000338
Chris Lattner76057302008-11-28 01:20:46 +0000339 // If the operand is an instruction that became dead as we nulled out the
340 // operand, and if it is 'trivially' dead, delete it in a future loop
341 // iteration.
342 if (Instruction *OpI = dyn_cast<Instruction>(OpV))
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000343 if (isInstructionTriviallyDead(OpI, TLI))
Chris Lattner76057302008-11-28 01:20:46 +0000344 DeadInsts.push_back(OpI);
345 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000346
Chris Lattner76057302008-11-28 01:20:46 +0000347 I->eraseFromParent();
Dan Gohman321a8132010-01-05 16:27:25 +0000348 } while (!DeadInsts.empty());
Dan Gohman90fe0bd2010-01-05 15:45:31 +0000349
350 return true;
Chris Lattner4d1e46e2002-05-07 18:07:59 +0000351}
Chris Lattnerb29714a2008-11-27 07:43:12 +0000352
Nick Lewycky1a4021a2011-02-20 08:38:20 +0000353/// areAllUsesEqual - Check whether the uses of a value are all the same.
354/// This is similar to Instruction::hasOneUse() except this will also return
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000355/// true when there are no uses or multiple uses that all refer to the same
356/// value.
Nick Lewycky1a4021a2011-02-20 08:38:20 +0000357static bool areAllUsesEqual(Instruction *I) {
358 Value::use_iterator UI = I->use_begin();
359 Value::use_iterator UE = I->use_end();
360 if (UI == UE)
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000361 return true;
Nick Lewycky1a4021a2011-02-20 08:38:20 +0000362
363 User *TheUse = *UI;
364 for (++UI; UI != UE; ++UI) {
365 if (*UI != TheUse)
366 return false;
367 }
368 return true;
369}
370
Dan Gohmanafc36a92009-05-02 18:29:22 +0000371/// RecursivelyDeleteDeadPHINode - If the specified value is an effectively
372/// dead PHI node, due to being a def-use chain of single-use nodes that
373/// either forms a cycle or is terminated by a trivially dead instruction,
374/// delete it. If that makes any of its operands trivially dead, delete them
Duncan Sands2cfbf012011-02-21 17:32:05 +0000375/// too, recursively. Return true if a change was made.
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000376bool llvm::RecursivelyDeleteDeadPHINode(PHINode *PN,
377 const TargetLibraryInfo *TLI) {
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000378 SmallPtrSet<Instruction*, 4> Visited;
379 for (Instruction *I = PN; areAllUsesEqual(I) && !I->mayHaveSideEffects();
380 I = cast<Instruction>(*I->use_begin())) {
381 if (I->use_empty())
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000382 return RecursivelyDeleteTriviallyDeadInstructions(I, TLI);
Nick Lewyckyeff5e692011-02-20 18:05:56 +0000383
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000384 // If we find an instruction more than once, we're on a cycle that
Dan Gohmanafc36a92009-05-02 18:29:22 +0000385 // won't prove fruitful.
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000386 if (!Visited.insert(I)) {
387 // Break the cycle and delete the instruction and its operands.
388 I->replaceAllUsesWith(UndefValue::get(I->getType()));
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000389 (void)RecursivelyDeleteTriviallyDeadInstructions(I, TLI);
Duncan Sands2cfbf012011-02-21 17:32:05 +0000390 return true;
Duncan Sandsb4098ba2011-02-21 16:27:36 +0000391 }
392 }
393 return false;
Dan Gohmanafc36a92009-05-02 18:29:22 +0000394}
Chris Lattner3481f242008-11-27 22:57:53 +0000395
Chris Lattnere234a302010-01-12 19:40:54 +0000396/// SimplifyInstructionsInBlock - Scan the specified basic block and try to
397/// simplify any instructions in it and recursively delete dead instructions.
398///
399/// This returns true if it changed the code, note that it can delete
400/// instructions in other blocks as well in this block.
Micah Villmow3574eca2012-10-08 16:38:25 +0000401bool llvm::SimplifyInstructionsInBlock(BasicBlock *BB, const DataLayout *TD,
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000402 const TargetLibraryInfo *TLI) {
Chris Lattnere234a302010-01-12 19:40:54 +0000403 bool MadeChange = false;
Chandler Carruthacdae3e2012-03-25 03:29:25 +0000404
405#ifndef NDEBUG
406 // In debug builds, ensure that the terminator of the block is never replaced
407 // or deleted by these simplifications. The idea of simplification is that it
408 // cannot introduce new instructions, and there is no way to replace the
409 // terminator of a block without introducing a new instruction.
410 AssertingVH<Instruction> TerminatorVH(--BB->end());
411#endif
412
Chandler Carruth858cd1c2012-03-24 23:03:27 +0000413 for (BasicBlock::iterator BI = BB->begin(), E = --BB->end(); BI != E; ) {
414 assert(!BI->isTerminator());
Chris Lattnere234a302010-01-12 19:40:54 +0000415 Instruction *Inst = BI++;
Chandler Carruth6b980542012-03-24 21:11:24 +0000416
417 WeakVH BIHandle(BI);
418 if (recursivelySimplifyInstruction(Inst, TD)) {
Chris Lattnere234a302010-01-12 19:40:54 +0000419 MadeChange = true;
Chris Lattner35a939b2010-07-15 06:06:04 +0000420 if (BIHandle != BI)
Chris Lattnere234a302010-01-12 19:40:54 +0000421 BI = BB->begin();
422 continue;
423 }
Eli Friedman71ad2c92011-04-02 22:45:17 +0000424
Benjamin Kramer8e0d1c02012-08-29 15:32:21 +0000425 MadeChange |= RecursivelyDeleteTriviallyDeadInstructions(Inst, TLI);
Eli Friedman71ad2c92011-04-02 22:45:17 +0000426 if (BIHandle != BI)
427 BI = BB->begin();
Chris Lattnere234a302010-01-12 19:40:54 +0000428 }
429 return MadeChange;
430}
431
Chris Lattnerb29714a2008-11-27 07:43:12 +0000432//===----------------------------------------------------------------------===//
Chris Lattner40d8c282009-11-10 22:26:15 +0000433// Control Flow Graph Restructuring.
Chris Lattnerb29714a2008-11-27 07:43:12 +0000434//
435
Chris Lattner40d8c282009-11-10 22:26:15 +0000436
437/// RemovePredecessorAndSimplify - Like BasicBlock::removePredecessor, this
438/// method is called when we're about to delete Pred as a predecessor of BB. If
439/// BB contains any PHI nodes, this drops the entries in the PHI nodes for Pred.
440///
441/// Unlike the removePredecessor method, this attempts to simplify uses of PHI
442/// nodes that collapse into identity values. For example, if we have:
443/// x = phi(1, 0, 0, 0)
444/// y = and x, z
445///
446/// .. and delete the predecessor corresponding to the '1', this will attempt to
447/// recursively fold the and to 0.
448void llvm::RemovePredecessorAndSimplify(BasicBlock *BB, BasicBlock *Pred,
Micah Villmow3574eca2012-10-08 16:38:25 +0000449 DataLayout *TD) {
Chris Lattner40d8c282009-11-10 22:26:15 +0000450 // This only adjusts blocks with PHI nodes.
451 if (!isa<PHINode>(BB->begin()))
452 return;
Jakub Staszaka18c5742013-07-22 23:16:36 +0000453
Chris Lattner40d8c282009-11-10 22:26:15 +0000454 // Remove the entries for Pred from the PHI nodes in BB, but do not simplify
455 // them down. This will leave us with single entry phi nodes and other phis
456 // that can be removed.
457 BB->removePredecessor(Pred, true);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000458
Chris Lattner40d8c282009-11-10 22:26:15 +0000459 WeakVH PhiIt = &BB->front();
460 while (PHINode *PN = dyn_cast<PHINode>(PhiIt)) {
461 PhiIt = &*++BasicBlock::iterator(cast<Instruction>(PhiIt));
Chris Lattner35a939b2010-07-15 06:06:04 +0000462 Value *OldPhiIt = PhiIt;
Chandler Carruth6b980542012-03-24 21:11:24 +0000463
464 if (!recursivelySimplifyInstruction(PN, TD))
465 continue;
466
Chris Lattner40d8c282009-11-10 22:26:15 +0000467 // If recursive simplification ended up deleting the next PHI node we would
468 // iterate to, then our iterator is invalid, restart scanning from the top
469 // of the block.
Chris Lattner35a939b2010-07-15 06:06:04 +0000470 if (PhiIt != OldPhiIt) PhiIt = &BB->front();
Chris Lattner40d8c282009-11-10 22:26:15 +0000471 }
472}
473
474
Chris Lattnerb29714a2008-11-27 07:43:12 +0000475/// MergeBasicBlockIntoOnlyPred - DestBB is a block with one predecessor and its
476/// predecessor is known to have one successor (DestBB!). Eliminate the edge
477/// between them, moving the instructions in the predecessor into DestBB and
478/// deleting the predecessor block.
479///
Andreas Neustifterad809812009-09-16 09:26:52 +0000480void llvm::MergeBasicBlockIntoOnlyPred(BasicBlock *DestBB, Pass *P) {
Chris Lattnerb29714a2008-11-27 07:43:12 +0000481 // If BB has single-entry PHI nodes, fold them.
482 while (PHINode *PN = dyn_cast<PHINode>(DestBB->begin())) {
483 Value *NewVal = PN->getIncomingValue(0);
484 // Replace self referencing PHI with undef, it must be dead.
Owen Anderson9e9a0d52009-07-30 23:03:37 +0000485 if (NewVal == PN) NewVal = UndefValue::get(PN->getType());
Chris Lattnerb29714a2008-11-27 07:43:12 +0000486 PN->replaceAllUsesWith(NewVal);
487 PN->eraseFromParent();
488 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000489
Chris Lattnerb29714a2008-11-27 07:43:12 +0000490 BasicBlock *PredBB = DestBB->getSinglePredecessor();
491 assert(PredBB && "Block doesn't have a single predecessor!");
Jakub Staszaka18c5742013-07-22 23:16:36 +0000492
Chris Lattner37914c82010-02-15 20:47:49 +0000493 // Zap anything that took the address of DestBB. Not doing this will give the
494 // address an invalid value.
495 if (DestBB->hasAddressTaken()) {
496 BlockAddress *BA = BlockAddress::get(DestBB);
497 Constant *Replacement =
498 ConstantInt::get(llvm::Type::getInt32Ty(BA->getContext()), 1);
499 BA->replaceAllUsesWith(ConstantExpr::getIntToPtr(Replacement,
500 BA->getType()));
501 BA->destroyConstant();
502 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000503
Chris Lattnerb29714a2008-11-27 07:43:12 +0000504 // Anything that branched to PredBB now branches to DestBB.
505 PredBB->replaceAllUsesWith(DestBB);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000506
Jay Foad95c3e482011-06-23 09:09:15 +0000507 // Splice all the instructions from PredBB to DestBB.
508 PredBB->getTerminator()->eraseFromParent();
509 DestBB->getInstList().splice(DestBB->begin(), PredBB->getInstList());
510
Andreas Neustifterad809812009-09-16 09:26:52 +0000511 if (P) {
Cameron Zwarich80f6a502011-01-08 17:01:52 +0000512 DominatorTree *DT = P->getAnalysisIfAvailable<DominatorTree>();
513 if (DT) {
514 BasicBlock *PredBBIDom = DT->getNode(PredBB)->getIDom()->getBlock();
515 DT->changeImmediateDominator(DestBB, PredBBIDom);
516 DT->eraseNode(PredBB);
517 }
Andreas Neustifterad809812009-09-16 09:26:52 +0000518 ProfileInfo *PI = P->getAnalysisIfAvailable<ProfileInfo>();
519 if (PI) {
520 PI->replaceAllUses(PredBB, DestBB);
521 PI->removeEdge(ProfileInfo::getEdge(PredBB, DestBB));
522 }
523 }
Chris Lattnerb29714a2008-11-27 07:43:12 +0000524 // Nuke BB.
525 PredBB->eraseFromParent();
526}
Devang Patel4afc90d2009-02-10 07:00:59 +0000527
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000528/// CanMergeValues - Return true if we can choose one of these values to use
529/// in place of the other. Note that we will always choose the non-undef
530/// value to keep.
531static bool CanMergeValues(Value *First, Value *Second) {
532 return First == Second || isa<UndefValue>(First) || isa<UndefValue>(Second);
533}
534
Chris Lattnerdce94d92009-11-10 05:59:26 +0000535/// CanPropagatePredecessorsForPHIs - Return true if we can fold BB, an
536/// almost-empty BB ending in an unconditional branch to Succ, into succ.
537///
538/// Assumption: Succ is the single successor for BB.
539///
540static bool CanPropagatePredecessorsForPHIs(BasicBlock *BB, BasicBlock *Succ) {
541 assert(*succ_begin(BB) == Succ && "Succ is not successor of BB!");
542
Jakub Staszaka18c5742013-07-22 23:16:36 +0000543 DEBUG(dbgs() << "Looking to fold " << BB->getName() << " into "
Chris Lattnerdce94d92009-11-10 05:59:26 +0000544 << Succ->getName() << "\n");
545 // Shortcut, if there is only a single predecessor it must be BB and merging
546 // is always safe
547 if (Succ->getSinglePredecessor()) return true;
548
549 // Make a list of the predecessors of BB
Benjamin Kramer88c09142011-12-06 16:14:29 +0000550 SmallPtrSet<BasicBlock*, 16> BBPreds(pred_begin(BB), pred_end(BB));
Chris Lattnerdce94d92009-11-10 05:59:26 +0000551
Chris Lattnerdce94d92009-11-10 05:59:26 +0000552 // Look at all the phi nodes in Succ, to see if they present a conflict when
553 // merging these blocks
554 for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) {
555 PHINode *PN = cast<PHINode>(I);
556
557 // If the incoming value from BB is again a PHINode in
558 // BB which has the same incoming value for *PI as PN does, we can
559 // merge the phi nodes and then the blocks can still be merged
560 PHINode *BBPN = dyn_cast<PHINode>(PN->getIncomingValueForBlock(BB));
561 if (BBPN && BBPN->getParent() == BB) {
Benjamin Kramer88c09142011-12-06 16:14:29 +0000562 for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) {
563 BasicBlock *IBB = PN->getIncomingBlock(PI);
564 if (BBPreds.count(IBB) &&
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000565 !CanMergeValues(BBPN->getIncomingValueForBlock(IBB),
566 PN->getIncomingValue(PI))) {
Jakub Staszaka18c5742013-07-22 23:16:36 +0000567 DEBUG(dbgs() << "Can't fold, phi node " << PN->getName() << " in "
568 << Succ->getName() << " is conflicting with "
Chris Lattnerdce94d92009-11-10 05:59:26 +0000569 << BBPN->getName() << " with regard to common predecessor "
Benjamin Kramer88c09142011-12-06 16:14:29 +0000570 << IBB->getName() << "\n");
Chris Lattnerdce94d92009-11-10 05:59:26 +0000571 return false;
572 }
573 }
574 } else {
575 Value* Val = PN->getIncomingValueForBlock(BB);
Benjamin Kramer88c09142011-12-06 16:14:29 +0000576 for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) {
Chris Lattnerdce94d92009-11-10 05:59:26 +0000577 // See if the incoming value for the common predecessor is equal to the
578 // one for BB, in which case this phi node will not prevent the merging
579 // of the block.
Benjamin Kramer88c09142011-12-06 16:14:29 +0000580 BasicBlock *IBB = PN->getIncomingBlock(PI);
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000581 if (BBPreds.count(IBB) &&
582 !CanMergeValues(Val, PN->getIncomingValue(PI))) {
Jakub Staszaka18c5742013-07-22 23:16:36 +0000583 DEBUG(dbgs() << "Can't fold, phi node " << PN->getName() << " in "
Chris Lattnerdce94d92009-11-10 05:59:26 +0000584 << Succ->getName() << " is conflicting with regard to common "
Benjamin Kramer88c09142011-12-06 16:14:29 +0000585 << "predecessor " << IBB->getName() << "\n");
Chris Lattnerdce94d92009-11-10 05:59:26 +0000586 return false;
587 }
588 }
589 }
590 }
591
592 return true;
593}
594
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000595typedef SmallVector<BasicBlock *, 16> PredBlockVector;
596typedef DenseMap<BasicBlock *, Value *> IncomingValueMap;
597
598/// \brief Determines the value to use as the phi node input for a block.
599///
600/// Select between \p OldVal any value that we know flows from \p BB
601/// to a particular phi on the basis of which one (if either) is not
602/// undef. Update IncomingValues based on the selected value.
603///
604/// \param OldVal The value we are considering selecting.
605/// \param BB The block that the value flows in from.
606/// \param IncomingValues A map from block-to-value for other phi inputs
607/// that we have examined.
608///
609/// \returns the selected value.
610static Value *selectIncomingValueForBlock(Value *OldVal, BasicBlock *BB,
611 IncomingValueMap &IncomingValues) {
612 if (!isa<UndefValue>(OldVal)) {
613 assert((!IncomingValues.count(BB) ||
614 IncomingValues.find(BB)->second == OldVal) &&
615 "Expected OldVal to match incoming value from BB!");
616
617 IncomingValues.insert(std::make_pair(BB, OldVal));
618 return OldVal;
619 }
620
621 IncomingValueMap::const_iterator It = IncomingValues.find(BB);
622 if (It != IncomingValues.end()) return It->second;
623
624 return OldVal;
625}
626
627/// \brief Create a map from block to value for the operands of a
628/// given phi.
629///
630/// Create a map from block to value for each non-undef value flowing
631/// into \p PN.
632///
633/// \param PN The phi we are collecting the map for.
634/// \param IncomingValues [out] The map from block to value for this phi.
635static void gatherIncomingValuesToPhi(PHINode *PN,
636 IncomingValueMap &IncomingValues) {
637 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
638 BasicBlock *BB = PN->getIncomingBlock(i);
639 Value *V = PN->getIncomingValue(i);
640
641 if (!isa<UndefValue>(V))
642 IncomingValues.insert(std::make_pair(BB, V));
643 }
644}
645
646/// \brief Replace the incoming undef values to a phi with the values
647/// from a block-to-value map.
648///
649/// \param PN The phi we are replacing the undefs in.
650/// \param IncomingValues A map from block to value.
651static void replaceUndefValuesInPhi(PHINode *PN,
652 const IncomingValueMap &IncomingValues) {
653 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
654 Value *V = PN->getIncomingValue(i);
655
656 if (!isa<UndefValue>(V)) continue;
657
658 BasicBlock *BB = PN->getIncomingBlock(i);
659 IncomingValueMap::const_iterator It = IncomingValues.find(BB);
660 if (It == IncomingValues.end()) continue;
661
662 PN->setIncomingValue(i, It->second);
663 }
664}
665
666/// \brief Replace a value flowing from a block to a phi with
667/// potentially multiple instances of that value flowing from the
668/// block's predecessors to the phi.
669///
670/// \param BB The block with the value flowing into the phi.
671/// \param BBPreds The predecessors of BB.
672/// \param PN The phi that we are updating.
673static void redirectValuesFromPredecessorsToPhi(BasicBlock *BB,
674 const PredBlockVector &BBPreds,
675 PHINode *PN) {
676 Value *OldVal = PN->removeIncomingValue(BB, false);
677 assert(OldVal && "No entry in PHI for Pred BB!");
678
679 IncomingValueMap IncomingValues;
680
681 // We are merging two blocks - BB, and the block containing PN - and
682 // as a result we need to redirect edges from the predecessors of BB
683 // to go to the block containing PN, and update PN
684 // accordingly. Since we allow merging blocks in the case where the
685 // predecessor and successor blocks both share some predecessors,
686 // and where some of those common predecessors might have undef
687 // values flowing into PN, we want to rewrite those values to be
688 // consistent with the non-undef values.
689
690 gatherIncomingValuesToPhi(PN, IncomingValues);
691
692 // If this incoming value is one of the PHI nodes in BB, the new entries
693 // in the PHI node are the entries from the old PHI.
694 if (isa<PHINode>(OldVal) && cast<PHINode>(OldVal)->getParent() == BB) {
695 PHINode *OldValPN = cast<PHINode>(OldVal);
696 for (unsigned i = 0, e = OldValPN->getNumIncomingValues(); i != e; ++i) {
697 // Note that, since we are merging phi nodes and BB and Succ might
698 // have common predecessors, we could end up with a phi node with
699 // identical incoming branches. This will be cleaned up later (and
700 // will trigger asserts if we try to clean it up now, without also
701 // simplifying the corresponding conditional branch).
702 BasicBlock *PredBB = OldValPN->getIncomingBlock(i);
703 Value *PredVal = OldValPN->getIncomingValue(i);
704 Value *Selected = selectIncomingValueForBlock(PredVal, PredBB,
705 IncomingValues);
706
707 // And add a new incoming value for this predecessor for the
708 // newly retargeted branch.
709 PN->addIncoming(Selected, PredBB);
710 }
711 } else {
712 for (unsigned i = 0, e = BBPreds.size(); i != e; ++i) {
713 // Update existing incoming values in PN for this
714 // predecessor of BB.
715 BasicBlock *PredBB = BBPreds[i];
716 Value *Selected = selectIncomingValueForBlock(OldVal, PredBB,
717 IncomingValues);
718
719 // And add a new incoming value for this predecessor for the
720 // newly retargeted branch.
721 PN->addIncoming(Selected, PredBB);
722 }
723 }
724
725 replaceUndefValuesInPhi(PN, IncomingValues);
726}
727
Chris Lattnerdce94d92009-11-10 05:59:26 +0000728/// TryToSimplifyUncondBranchFromEmptyBlock - BB is known to contain an
729/// unconditional branch, and contains no instructions other than PHI nodes,
Rafael Espindola77a2c372011-06-30 20:14:24 +0000730/// potential side-effect free intrinsics and the branch. If possible,
731/// eliminate BB by rewriting all the predecessors to branch to the successor
732/// block and return true. If we can't transform, return false.
Chris Lattnerdce94d92009-11-10 05:59:26 +0000733bool llvm::TryToSimplifyUncondBranchFromEmptyBlock(BasicBlock *BB) {
Dan Gohmane2c6d132010-08-14 00:29:42 +0000734 assert(BB != &BB->getParent()->getEntryBlock() &&
735 "TryToSimplifyUncondBranchFromEmptyBlock called on entry block!");
736
Chris Lattnerdce94d92009-11-10 05:59:26 +0000737 // We can't eliminate infinite loops.
738 BasicBlock *Succ = cast<BranchInst>(BB->getTerminator())->getSuccessor(0);
739 if (BB == Succ) return false;
Jakub Staszaka18c5742013-07-22 23:16:36 +0000740
Chris Lattnerdce94d92009-11-10 05:59:26 +0000741 // Check to see if merging these blocks would cause conflicts for any of the
742 // phi nodes in BB or Succ. If not, we can safely merge.
743 if (!CanPropagatePredecessorsForPHIs(BB, Succ)) return false;
744
745 // Check for cases where Succ has multiple predecessors and a PHI node in BB
746 // has uses which will not disappear when the PHI nodes are merged. It is
747 // possible to handle such cases, but difficult: it requires checking whether
748 // BB dominates Succ, which is non-trivial to calculate in the case where
749 // Succ has multiple predecessors. Also, it requires checking whether
Alexey Samsonov3a199992012-12-24 08:52:53 +0000750 // constructing the necessary self-referential PHI node doesn't introduce any
Chris Lattnerdce94d92009-11-10 05:59:26 +0000751 // conflicts; this isn't too difficult, but the previous code for doing this
752 // was incorrect.
753 //
754 // Note that if this check finds a live use, BB dominates Succ, so BB is
755 // something like a loop pre-header (or rarely, a part of an irreducible CFG);
756 // folding the branch isn't profitable in that case anyway.
757 if (!Succ->getSinglePredecessor()) {
758 BasicBlock::iterator BBI = BB->begin();
759 while (isa<PHINode>(*BBI)) {
760 for (Value::use_iterator UI = BBI->use_begin(), E = BBI->use_end();
761 UI != E; ++UI) {
762 if (PHINode* PN = dyn_cast<PHINode>(*UI)) {
763 if (PN->getIncomingBlock(UI) != BB)
764 return false;
765 } else {
766 return false;
767 }
768 }
769 ++BBI;
770 }
771 }
772
David Greenefae77062010-01-05 01:26:57 +0000773 DEBUG(dbgs() << "Killing Trivial BB: \n" << *BB);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000774
Chris Lattnerdce94d92009-11-10 05:59:26 +0000775 if (isa<PHINode>(Succ->begin())) {
776 // If there is more than one pred of succ, and there are PHI nodes in
777 // the successor, then we need to add incoming edges for the PHI nodes
778 //
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000779 const PredBlockVector BBPreds(pred_begin(BB), pred_end(BB));
Jakub Staszaka18c5742013-07-22 23:16:36 +0000780
Chris Lattnerdce94d92009-11-10 05:59:26 +0000781 // Loop over all of the PHI nodes in the successor of BB.
782 for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) {
783 PHINode *PN = cast<PHINode>(I);
Duncan Sandsc48b55a2013-07-11 08:28:20 +0000784
785 redirectValuesFromPredecessorsToPhi(BB, BBPreds, PN);
Chris Lattnerdce94d92009-11-10 05:59:26 +0000786 }
787 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000788
Rafael Espindola77a2c372011-06-30 20:14:24 +0000789 if (Succ->getSinglePredecessor()) {
790 // BB is the only predecessor of Succ, so Succ will end up with exactly
791 // the same predecessors BB had.
792
793 // Copy over any phi, debug or lifetime instruction.
794 BB->getTerminator()->eraseFromParent();
795 Succ->getInstList().splice(Succ->getFirstNonPHI(), BB->getInstList());
796 } else {
797 while (PHINode *PN = dyn_cast<PHINode>(&BB->front())) {
Chris Lattnerdce94d92009-11-10 05:59:26 +0000798 // We explicitly check for such uses in CanPropagatePredecessorsForPHIs.
799 assert(PN->use_empty() && "There shouldn't be any uses here!");
800 PN->eraseFromParent();
801 }
802 }
Jakub Staszaka18c5742013-07-22 23:16:36 +0000803
Chris Lattnerdce94d92009-11-10 05:59:26 +0000804 // Everything that jumped to BB now goes to Succ.
805 BB->replaceAllUsesWith(Succ);
806 if (!Succ->hasName()) Succ->takeName(BB);
807 BB->eraseFromParent(); // Delete the old basic block.
808 return true;
809}
810
Jim Grosbach43a82412009-12-02 17:06:45 +0000811/// EliminateDuplicatePHINodes - Check for and eliminate duplicate PHI
812/// nodes in this block. This doesn't try to be clever about PHI nodes
813/// which differ only in the order of the incoming values, but instcombine
814/// orders them so it usually won't matter.
815///
816bool llvm::EliminateDuplicatePHINodes(BasicBlock *BB) {
817 bool Changed = false;
818
819 // This implementation doesn't currently consider undef operands
Nick Lewycky89991d42011-06-28 03:57:31 +0000820 // specially. Theoretically, two phis which are identical except for
Jim Grosbach43a82412009-12-02 17:06:45 +0000821 // one having an undef where the other doesn't could be collapsed.
822
823 // Map from PHI hash values to PHI nodes. If multiple PHIs have
824 // the same hash value, the element is the first PHI in the
825 // linked list in CollisionMap.
826 DenseMap<uintptr_t, PHINode *> HashMap;
827
828 // Maintain linked lists of PHI nodes with common hash values.
829 DenseMap<PHINode *, PHINode *> CollisionMap;
830
831 // Examine each PHI.
832 for (BasicBlock::iterator I = BB->begin();
833 PHINode *PN = dyn_cast<PHINode>(I++); ) {
834 // Compute a hash value on the operands. Instcombine will likely have sorted
835 // them, which helps expose duplicates, but we have to check all the
836 // operands to be safe in case instcombine hasn't run.
837 uintptr_t Hash = 0;
Jay Foad95c3e482011-06-23 09:09:15 +0000838 // This hash algorithm is quite weak as hash functions go, but it seems
839 // to do a good enough job for this particular purpose, and is very quick.
Jim Grosbach43a82412009-12-02 17:06:45 +0000840 for (User::op_iterator I = PN->op_begin(), E = PN->op_end(); I != E; ++I) {
Jim Grosbach43a82412009-12-02 17:06:45 +0000841 Hash ^= reinterpret_cast<uintptr_t>(static_cast<Value *>(*I));
842 Hash = (Hash << 7) | (Hash >> (sizeof(uintptr_t) * CHAR_BIT - 7));
843 }
Jay Foad95c3e482011-06-23 09:09:15 +0000844 for (PHINode::block_iterator I = PN->block_begin(), E = PN->block_end();
845 I != E; ++I) {
846 Hash ^= reinterpret_cast<uintptr_t>(static_cast<BasicBlock *>(*I));
847 Hash = (Hash << 7) | (Hash >> (sizeof(uintptr_t) * CHAR_BIT - 7));
848 }
Jakob Stoklund Olesen2bc2a082011-03-04 02:48:56 +0000849 // Avoid colliding with the DenseMap sentinels ~0 and ~0-1.
850 Hash >>= 1;
Jim Grosbach43a82412009-12-02 17:06:45 +0000851 // If we've never seen this hash value before, it's a unique PHI.
852 std::pair<DenseMap<uintptr_t, PHINode *>::iterator, bool> Pair =
853 HashMap.insert(std::make_pair(Hash, PN));
854 if (Pair.second) continue;
855 // Otherwise it's either a duplicate or a hash collision.
856 for (PHINode *OtherPN = Pair.first->second; ; ) {
857 if (OtherPN->isIdenticalTo(PN)) {
858 // A duplicate. Replace this PHI with its duplicate.
859 PN->replaceAllUsesWith(OtherPN);
860 PN->eraseFromParent();
861 Changed = true;
862 break;
863 }
864 // A non-duplicate hash collision.
865 DenseMap<PHINode *, PHINode *>::iterator I = CollisionMap.find(OtherPN);
866 if (I == CollisionMap.end()) {
867 // Set this PHI to be the head of the linked list of colliding PHIs.
868 PHINode *Old = Pair.first->second;
869 Pair.first->second = PN;
870 CollisionMap[PN] = Old;
871 break;
872 }
Benjamin Kramerd9b0b022012-06-02 10:20:22 +0000873 // Proceed to the next PHI in the list.
Jim Grosbach43a82412009-12-02 17:06:45 +0000874 OtherPN = I->second;
875 }
876 }
877
878 return Changed;
879}
Chris Lattner687140c2010-12-25 20:37:57 +0000880
881/// enforceKnownAlignment - If the specified pointer points to an object that
882/// we control, modify the object's alignment to PrefAlign. This isn't
883/// often possible though. If alignment is important, a more reliable approach
884/// is to simply align all global variables and allocation instructions to
885/// their preferred alignment from the beginning.
886///
Benjamin Kramer19282362010-12-30 22:34:44 +0000887static unsigned enforceKnownAlignment(Value *V, unsigned Align,
Micah Villmow3574eca2012-10-08 16:38:25 +0000888 unsigned PrefAlign, const DataLayout *TD) {
Eli Friedmanb53c7932011-06-15 21:08:25 +0000889 V = V->stripPointerCasts();
Chris Lattner687140c2010-12-25 20:37:57 +0000890
Eli Friedmanb53c7932011-06-15 21:08:25 +0000891 if (AllocaInst *AI = dyn_cast<AllocaInst>(V)) {
Lang Hamesbb5b3f32011-10-10 23:42:08 +0000892 // If the preferred alignment is greater than the natural stack alignment
893 // then don't round up. This avoids dynamic stack realignment.
894 if (TD && TD->exceedsNaturalStackAlignment(PrefAlign))
895 return Align;
Chris Lattner687140c2010-12-25 20:37:57 +0000896 // If there is a requested alignment and if this is an alloca, round up.
897 if (AI->getAlignment() >= PrefAlign)
898 return AI->getAlignment();
899 AI->setAlignment(PrefAlign);
900 return PrefAlign;
901 }
Chris Lattner687140c2010-12-25 20:37:57 +0000902
903 if (GlobalValue *GV = dyn_cast<GlobalValue>(V)) {
904 // If there is a large requested alignment and we can, bump up the alignment
905 // of the global.
906 if (GV->isDeclaration()) return Align;
Duncan Sandsd3a38cc2011-11-29 18:26:38 +0000907 // If the memory we set aside for the global may not be the memory used by
908 // the final program then it is impossible for us to reliably enforce the
909 // preferred alignment.
910 if (GV->isWeakForLinker()) return Align;
Jakub Staszaka18c5742013-07-22 23:16:36 +0000911
Chris Lattner687140c2010-12-25 20:37:57 +0000912 if (GV->getAlignment() >= PrefAlign)
913 return GV->getAlignment();
914 // We can only increase the alignment of the global if it has no alignment
915 // specified or if it is not assigned a section. If it is assigned a
916 // section, the global could be densely packed with other objects in the
917 // section, increasing the alignment could cause padding issues.
918 if (!GV->hasSection() || GV->getAlignment() == 0)
919 GV->setAlignment(PrefAlign);
920 return GV->getAlignment();
921 }
922
923 return Align;
924}
925
926/// getOrEnforceKnownAlignment - If the specified pointer has an alignment that
927/// we can determine, return it, otherwise return 0. If PrefAlign is specified,
928/// and it is more than the alignment of the ultimate object, see if we can
929/// increase the alignment of the ultimate object, making this check succeed.
930unsigned llvm::getOrEnforceKnownAlignment(Value *V, unsigned PrefAlign,
Micah Villmow3574eca2012-10-08 16:38:25 +0000931 const DataLayout *TD) {
Chris Lattner687140c2010-12-25 20:37:57 +0000932 assert(V->getType()->isPointerTy() &&
933 "getOrEnforceKnownAlignment expects a pointer!");
Chandler Carruth426c2bf2012-11-01 09:14:31 +0000934 unsigned BitWidth = TD ? TD->getPointerSizeInBits() : 64;
Chris Lattner687140c2010-12-25 20:37:57 +0000935 APInt KnownZero(BitWidth, 0), KnownOne(BitWidth, 0);
Rafael Espindola26c8dcc2012-04-04 12:51:34 +0000936 ComputeMaskedBits(V, KnownZero, KnownOne, TD);
Chris Lattner687140c2010-12-25 20:37:57 +0000937 unsigned TrailZ = KnownZero.countTrailingOnes();
Jakub Staszaka18c5742013-07-22 23:16:36 +0000938
Chris Lattner687140c2010-12-25 20:37:57 +0000939 // Avoid trouble with rediculously large TrailZ values, such as
940 // those computed from a null pointer.
941 TrailZ = std::min(TrailZ, unsigned(sizeof(unsigned) * CHAR_BIT - 1));
Jakub Staszaka18c5742013-07-22 23:16:36 +0000942
Chris Lattner687140c2010-12-25 20:37:57 +0000943 unsigned Align = 1u << std::min(BitWidth - 1, TrailZ);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000944
Chris Lattner687140c2010-12-25 20:37:57 +0000945 // LLVM doesn't support alignments larger than this currently.
946 Align = std::min(Align, +Value::MaximumAlignment);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000947
Chris Lattner687140c2010-12-25 20:37:57 +0000948 if (PrefAlign > Align)
Lang Hamesbb5b3f32011-10-10 23:42:08 +0000949 Align = enforceKnownAlignment(V, Align, PrefAlign, TD);
Jakub Staszaka18c5742013-07-22 23:16:36 +0000950
Chris Lattner687140c2010-12-25 20:37:57 +0000951 // We don't need to make any adjustment.
952 return Align;
953}
954
Devang Patel5ee20682011-03-17 21:58:19 +0000955///===---------------------------------------------------------------------===//
956/// Dbg Intrinsic utilities
957///
958
Adrian Prantl163da932013-04-26 17:48:33 +0000959/// See if there is a dbg.value intrinsic for DIVar before I.
960static bool LdStHasDebugValue(DIVariable &DIVar, Instruction *I) {
961 // Since we can't guarantee that the original dbg.declare instrinsic
962 // is removed by LowerDbgDeclare(), we need to make sure that we are
963 // not inserting the same dbg.value intrinsic over and over.
964 llvm::BasicBlock::InstListType::iterator PrevI(I);
965 if (PrevI != I->getParent()->getInstList().begin()) {
966 --PrevI;
967 if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(PrevI))
968 if (DVI->getValue() == I->getOperand(0) &&
969 DVI->getOffset() == 0 &&
970 DVI->getVariable() == DIVar)
971 return true;
972 }
973 return false;
974}
975
Adrian Prantl9d5d58a2013-04-26 18:10:50 +0000976/// Inserts a llvm.dbg.value intrinsic before a store to an alloca'd value
Devang Patel5ee20682011-03-17 21:58:19 +0000977/// that has an associated llvm.dbg.decl intrinsic.
978bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI,
979 StoreInst *SI, DIBuilder &Builder) {
980 DIVariable DIVar(DDI->getVariable());
Manman Rencbafae62013-06-28 05:43:10 +0000981 assert((!DIVar || DIVar.isVariable()) &&
982 "Variable in DbgDeclareInst should be either null or a DIVariable.");
983 if (!DIVar)
Devang Patel5ee20682011-03-17 21:58:19 +0000984 return false;
985
Adrian Prantl163da932013-04-26 17:48:33 +0000986 if (LdStHasDebugValue(DIVar, SI))
987 return true;
988
Devang Patel227dfdb2011-05-16 21:24:05 +0000989 Instruction *DbgVal = NULL;
990 // If an argument is zero extended then use argument directly. The ZExt
991 // may be zapped by an optimization pass in future.
992 Argument *ExtendedArg = NULL;
993 if (ZExtInst *ZExt = dyn_cast<ZExtInst>(SI->getOperand(0)))
994 ExtendedArg = dyn_cast<Argument>(ZExt->getOperand(0));
995 if (SExtInst *SExt = dyn_cast<SExtInst>(SI->getOperand(0)))
996 ExtendedArg = dyn_cast<Argument>(SExt->getOperand(0));
997 if (ExtendedArg)
998 DbgVal = Builder.insertDbgValueIntrinsic(ExtendedArg, 0, DIVar, SI);
999 else
1000 DbgVal = Builder.insertDbgValueIntrinsic(SI->getOperand(0), 0, DIVar, SI);
1001
Devang Patel5ee20682011-03-17 21:58:19 +00001002 // Propagate any debug metadata from the store onto the dbg.value.
1003 DebugLoc SIDL = SI->getDebugLoc();
1004 if (!SIDL.isUnknown())
1005 DbgVal->setDebugLoc(SIDL);
1006 // Otherwise propagate debug metadata from dbg.declare.
1007 else
1008 DbgVal->setDebugLoc(DDI->getDebugLoc());
1009 return true;
1010}
1011
Adrian Prantl9d5d58a2013-04-26 18:10:50 +00001012/// Inserts a llvm.dbg.value intrinsic before a load of an alloca'd value
Devang Patel36fae672011-03-18 23:45:43 +00001013/// that has an associated llvm.dbg.decl intrinsic.
1014bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI,
1015 LoadInst *LI, DIBuilder &Builder) {
1016 DIVariable DIVar(DDI->getVariable());
Jakub Staszaka18c5742013-07-22 23:16:36 +00001017 assert((!DIVar || DIVar.isVariable()) &&
Manman Rencbafae62013-06-28 05:43:10 +00001018 "Variable in DbgDeclareInst should be either null or a DIVariable.");
1019 if (!DIVar)
Devang Patel36fae672011-03-18 23:45:43 +00001020 return false;
1021
Adrian Prantl163da932013-04-26 17:48:33 +00001022 if (LdStHasDebugValue(DIVar, LI))
1023 return true;
1024
Jakub Staszaka18c5742013-07-22 23:16:36 +00001025 Instruction *DbgVal =
Devang Patel36fae672011-03-18 23:45:43 +00001026 Builder.insertDbgValueIntrinsic(LI->getOperand(0), 0,
1027 DIVar, LI);
Jakub Staszaka18c5742013-07-22 23:16:36 +00001028
Devang Patel36fae672011-03-18 23:45:43 +00001029 // Propagate any debug metadata from the store onto the dbg.value.
1030 DebugLoc LIDL = LI->getDebugLoc();
1031 if (!LIDL.isUnknown())
1032 DbgVal->setDebugLoc(LIDL);
1033 // Otherwise propagate debug metadata from dbg.declare.
1034 else
1035 DbgVal->setDebugLoc(DDI->getDebugLoc());
1036 return true;
1037}
1038
Devang Patel813c9a02011-03-17 22:18:16 +00001039/// LowerDbgDeclare - Lowers llvm.dbg.declare intrinsics into appropriate set
1040/// of llvm.dbg.value intrinsics.
1041bool llvm::LowerDbgDeclare(Function &F) {
1042 DIBuilder DIB(*F.getParent());
1043 SmallVector<DbgDeclareInst *, 4> Dbgs;
1044 for (Function::iterator FI = F.begin(), FE = F.end(); FI != FE; ++FI)
1045 for (BasicBlock::iterator BI = FI->begin(), BE = FI->end(); BI != BE; ++BI) {
1046 if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(BI))
1047 Dbgs.push_back(DDI);
1048 }
1049 if (Dbgs.empty())
1050 return false;
1051
Craig Topper6227d5c2013-07-04 01:31:24 +00001052 for (SmallVectorImpl<DbgDeclareInst *>::iterator I = Dbgs.begin(),
Devang Patel813c9a02011-03-17 22:18:16 +00001053 E = Dbgs.end(); I != E; ++I) {
1054 DbgDeclareInst *DDI = *I;
1055 if (AllocaInst *AI = dyn_cast_or_null<AllocaInst>(DDI->getAddress())) {
Adrian Prantl163da932013-04-26 17:48:33 +00001056 // We only remove the dbg.declare intrinsic if all uses are
1057 // converted to dbg.value intrinsics.
Devang Patel81ad03c2011-04-28 20:32:02 +00001058 bool RemoveDDI = true;
Devang Patel813c9a02011-03-17 22:18:16 +00001059 for (Value::use_iterator UI = AI->use_begin(), E = AI->use_end();
1060 UI != E; ++UI)
1061 if (StoreInst *SI = dyn_cast<StoreInst>(*UI))
1062 ConvertDebugDeclareToDebugValue(DDI, SI, DIB);
Devang Patel36fae672011-03-18 23:45:43 +00001063 else if (LoadInst *LI = dyn_cast<LoadInst>(*UI))
1064 ConvertDebugDeclareToDebugValue(DDI, LI, DIB);
Devang Patel81ad03c2011-04-28 20:32:02 +00001065 else
1066 RemoveDDI = false;
1067 if (RemoveDDI)
1068 DDI->eraseFromParent();
Devang Patel813c9a02011-03-17 22:18:16 +00001069 }
Devang Patel813c9a02011-03-17 22:18:16 +00001070 }
1071 return true;
1072}
Cameron Zwarichc8279392011-05-24 03:10:43 +00001073
1074/// FindAllocaDbgDeclare - Finds the llvm.dbg.declare intrinsic describing the
1075/// alloca 'V', if any.
1076DbgDeclareInst *llvm::FindAllocaDbgDeclare(Value *V) {
1077 if (MDNode *DebugNode = MDNode::getIfExists(V->getContext(), V))
1078 for (Value::use_iterator UI = DebugNode->use_begin(),
1079 E = DebugNode->use_end(); UI != E; ++UI)
1080 if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(*UI))
1081 return DDI;
1082
1083 return 0;
1084}
Alexey Samsonov1afbb512012-12-12 14:31:53 +00001085
1086bool llvm::replaceDbgDeclareForAlloca(AllocaInst *AI, Value *NewAllocaAddress,
1087 DIBuilder &Builder) {
1088 DbgDeclareInst *DDI = FindAllocaDbgDeclare(AI);
1089 if (!DDI)
1090 return false;
1091 DIVariable DIVar(DDI->getVariable());
Jakub Staszaka18c5742013-07-22 23:16:36 +00001092 assert((!DIVar || DIVar.isVariable()) &&
Manman Rencbafae62013-06-28 05:43:10 +00001093 "Variable in DbgDeclareInst should be either null or a DIVariable.");
1094 if (!DIVar)
Alexey Samsonov1afbb512012-12-12 14:31:53 +00001095 return false;
1096
1097 // Create a copy of the original DIDescriptor for user variable, appending
1098 // "deref" operation to a list of address elements, as new llvm.dbg.declare
1099 // will take a value storing address of the memory for variable, not
1100 // alloca itself.
1101 Type *Int64Ty = Type::getInt64Ty(AI->getContext());
1102 SmallVector<Value*, 4> NewDIVarAddress;
1103 if (DIVar.hasComplexAddress()) {
1104 for (unsigned i = 0, n = DIVar.getNumAddrElements(); i < n; ++i) {
1105 NewDIVarAddress.push_back(
1106 ConstantInt::get(Int64Ty, DIVar.getAddrElement(i)));
1107 }
1108 }
1109 NewDIVarAddress.push_back(ConstantInt::get(Int64Ty, DIBuilder::OpDeref));
1110 DIVariable NewDIVar = Builder.createComplexVariable(
1111 DIVar.getTag(), DIVar.getContext(), DIVar.getName(),
1112 DIVar.getFile(), DIVar.getLineNumber(), DIVar.getType(),
1113 NewDIVarAddress, DIVar.getArgNumber());
1114
1115 // Insert llvm.dbg.declare in the same basic block as the original alloca,
1116 // and remove old llvm.dbg.declare.
1117 BasicBlock *BB = AI->getParent();
1118 Builder.insertDeclare(NewAllocaAddress, NewDIVar, BB);
1119 DDI->eraseFromParent();
1120 return true;
1121}
Evgeniy Stepanov3333e662012-12-21 11:18:49 +00001122
1123bool llvm::removeUnreachableBlocks(Function &F) {
1124 SmallPtrSet<BasicBlock*, 16> Reachable;
1125 SmallVector<BasicBlock*, 128> Worklist;
1126 Worklist.push_back(&F.getEntryBlock());
1127 Reachable.insert(&F.getEntryBlock());
1128 do {
1129 BasicBlock *BB = Worklist.pop_back_val();
1130 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI)
1131 if (Reachable.insert(*SI))
1132 Worklist.push_back(*SI);
1133 } while (!Worklist.empty());
1134
1135 if (Reachable.size() == F.size())
1136 return false;
1137
1138 assert(Reachable.size() < F.size());
1139 for (Function::iterator I = llvm::next(F.begin()), E = F.end(); I != E; ++I) {
1140 if (Reachable.count(I))
1141 continue;
1142
Evgeniy Stepanov3333e662012-12-21 11:18:49 +00001143 for (succ_iterator SI = succ_begin(I), SE = succ_end(I); SI != SE; ++SI)
1144 if (Reachable.count(*SI))
1145 (*SI)->removePredecessor(I);
Evgeniy Stepanov7541cd32013-03-22 08:43:04 +00001146 I->dropAllReferences();
Evgeniy Stepanov3333e662012-12-21 11:18:49 +00001147 }
Evgeniy Stepanov7541cd32013-03-22 08:43:04 +00001148
1149 for (Function::iterator I = llvm::next(F.begin()), E=F.end(); I != E;)
1150 if (!Reachable.count(I))
1151 I = F.getBasicBlockList().erase(I);
1152 else
1153 ++I;
1154
Evgeniy Stepanov3333e662012-12-21 11:18:49 +00001155 return true;
1156}