blob: e7bc1357ffe641ef125c67908072449d082d3b13 [file] [log] [blame]
Chris Lattnerea54ab92002-05-07 22:11:39 +00001//===- ADCE.cpp - Code to perform aggressive dead code elimination --------===//
Chris Lattner02e90d52001-06-30 06:39:11 +00002//
Chris Lattnerea54ab92002-05-07 22:11:39 +00003// This file implements "aggressive" dead code elimination. ADCE is DCe where
Chris Lattner02e90d52001-06-30 06:39:11 +00004// values are assumed to be dead until proven otherwise. This is similar to
5// SCCP, except applied to the liveness of values.
6//
7//===----------------------------------------------------------------------===//
8
Chris Lattner022103b2002-05-07 20:03:00 +00009#include "llvm/Transforms/Scalar.h"
Chris Lattnerea54ab92002-05-07 22:11:39 +000010#include "llvm/Transforms/Utils/Local.h"
Chris Lattner011de072002-07-29 22:31:39 +000011#include "llvm/Transforms/Utils/BasicBlockUtils.h"
Chris Lattner02e90d52001-06-30 06:39:11 +000012#include "llvm/Type.h"
Chris Lattnera69fd902002-08-21 23:43:50 +000013#include "llvm/Analysis/PostDominators.h"
Chris Lattnerb8259dd2001-09-09 22:26:47 +000014#include "llvm/iTerminators.h"
Chris Lattner7061dc52001-12-03 18:02:31 +000015#include "llvm/iPHINode.h"
Chris Lattnerd9036a12002-05-22 21:32:16 +000016#include "llvm/Constant.h"
Chris Lattner221d6882002-02-12 21:07:25 +000017#include "llvm/Support/CFG.h"
Chris Lattnercee8f9a2001-11-27 00:03:19 +000018#include "Support/STLExtras.h"
19#include "Support/DepthFirstIterator.h"
Chris Lattnera92f6962002-10-01 22:38:41 +000020#include "Support/Statistic.h"
Chris Lattner72f1e992001-07-08 18:38:36 +000021#include <algorithm>
Chris Lattner697954c2002-01-20 22:54:45 +000022using std::cerr;
Anand Shukla5ba99bd2002-06-25 21:07:58 +000023using std::vector;
Chris Lattner02e90d52001-06-30 06:39:11 +000024
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000025namespace {
Chris Lattnera92f6962002-10-01 22:38:41 +000026 Statistic<> NumBlockRemoved("adce", "Number of basic blocks removed");
27 Statistic<> NumInstRemoved ("adce", "Number of instructions removed");
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000028
Chris Lattner02e90d52001-06-30 06:39:11 +000029//===----------------------------------------------------------------------===//
30// ADCE Class
31//
Chris Lattnerea54ab92002-05-07 22:11:39 +000032// This class does all of the work of Aggressive Dead Code Elimination.
Chris Lattner02e90d52001-06-30 06:39:11 +000033// It's public interface consists of a constructor and a doADCE() method.
34//
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000035class ADCE : public FunctionPass {
36 Function *Func; // The function that we are working on
Chris Lattner697954c2002-01-20 22:54:45 +000037 std::vector<Instruction*> WorkList; // Instructions that just became live
38 std::set<Instruction*> LiveSet; // The set of live instructions
Chris Lattner02e90d52001-06-30 06:39:11 +000039
40 //===--------------------------------------------------------------------===//
41 // The public interface for this class
42 //
43public:
Chris Lattnerd9036a12002-05-22 21:32:16 +000044 // Execute the Aggressive Dead Code Elimination Algorithm
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000045 //
Chris Lattner7e708292002-06-25 16:13:24 +000046 virtual bool runOnFunction(Function &F) {
47 Func = &F;
Chris Lattnerd9036a12002-05-22 21:32:16 +000048 bool Changed = doADCE();
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000049 assert(WorkList.empty());
50 LiveSet.clear();
Chris Lattnerd9036a12002-05-22 21:32:16 +000051 return Changed;
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000052 }
53 // getAnalysisUsage - We require post dominance frontiers (aka Control
54 // Dependence Graph)
55 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
Chris Lattner5f0eb8d2002-08-08 19:01:30 +000056 AU.addRequired<PostDominatorTree>();
57 AU.addRequired<PostDominanceFrontier>();
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000058 }
Chris Lattner02e90d52001-06-30 06:39:11 +000059
Chris Lattner02e90d52001-06-30 06:39:11 +000060
61 //===--------------------------------------------------------------------===//
62 // The implementation of this class
63 //
64private:
Chris Lattnerea54ab92002-05-07 22:11:39 +000065 // doADCE() - Run the Aggressive Dead Code Elimination algorithm, returning
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000066 // true if the function was modified.
67 //
Chris Lattnerd9036a12002-05-22 21:32:16 +000068 bool doADCE();
69
70 void markBlockAlive(BasicBlock *BB);
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000071
Chris Lattner446698b2002-07-30 00:22:34 +000072
73 // dropReferencesOfDeadInstructionsInLiveBlock - Loop over all of the
74 // instructions in the specified basic block, dropping references on
75 // instructions that are dead according to LiveSet.
76 bool dropReferencesOfDeadInstructionsInLiveBlock(BasicBlock *BB);
77
Chris Lattner02e90d52001-06-30 06:39:11 +000078 inline void markInstructionLive(Instruction *I) {
79 if (LiveSet.count(I)) return;
Chris Lattnerf016ea42002-05-22 17:17:27 +000080 DEBUG(cerr << "Insn Live: " << I);
Chris Lattner02e90d52001-06-30 06:39:11 +000081 LiveSet.insert(I);
82 WorkList.push_back(I);
83 }
84
Chris Lattner72f1e992001-07-08 18:38:36 +000085 inline void markTerminatorLive(const BasicBlock *BB) {
Chris Lattnerf016ea42002-05-22 17:17:27 +000086 DEBUG(cerr << "Terminat Live: " << BB->getTerminator());
Chris Lattnerb8259dd2001-09-09 22:26:47 +000087 markInstructionLive((Instruction*)BB->getTerminator());
Chris Lattner72f1e992001-07-08 18:38:36 +000088 }
Chris Lattner02e90d52001-06-30 06:39:11 +000089};
90
Chris Lattnera6275cc2002-07-26 21:12:46 +000091 RegisterOpt<ADCE> X("adce", "Aggressive Dead Code Elimination");
Chris Lattnerdfe81ab2002-05-06 17:27:57 +000092} // End of anonymous namespace
93
Chris Lattnerd9036a12002-05-22 21:32:16 +000094Pass *createAggressiveDCEPass() { return new ADCE(); }
95
Chris Lattnerd9036a12002-05-22 21:32:16 +000096void ADCE::markBlockAlive(BasicBlock *BB) {
97 // Mark the basic block as being newly ALIVE... and mark all branches that
98 // this block is control dependant on as being alive also...
99 //
Chris Lattnerce6ef112002-07-26 18:40:14 +0000100 PostDominanceFrontier &CDG = getAnalysis<PostDominanceFrontier>();
Chris Lattnerd9036a12002-05-22 21:32:16 +0000101
Chris Lattnerce6ef112002-07-26 18:40:14 +0000102 PostDominanceFrontier::const_iterator It = CDG.find(BB);
Chris Lattnerd9036a12002-05-22 21:32:16 +0000103 if (It != CDG.end()) {
104 // Get the blocks that this node is control dependant on...
Chris Lattnerce6ef112002-07-26 18:40:14 +0000105 const PostDominanceFrontier::DomSetType &CDB = It->second;
Chris Lattnerd9036a12002-05-22 21:32:16 +0000106 for_each(CDB.begin(), CDB.end(), // Mark all their terminators as live
107 bind_obj(this, &ADCE::markTerminatorLive));
108 }
109
110 // If this basic block is live, then the terminator must be as well!
111 markTerminatorLive(BB);
Chris Lattnerdfe81ab2002-05-06 17:27:57 +0000112}
Chris Lattner02e90d52001-06-30 06:39:11 +0000113
Chris Lattner446698b2002-07-30 00:22:34 +0000114// dropReferencesOfDeadInstructionsInLiveBlock - Loop over all of the
115// instructions in the specified basic block, dropping references on
116// instructions that are dead according to LiveSet.
117bool ADCE::dropReferencesOfDeadInstructionsInLiveBlock(BasicBlock *BB) {
118 bool Changed = false;
119 for (BasicBlock::iterator I = BB->begin(), E = --BB->end(); I != E; )
120 if (!LiveSet.count(I)) { // Is this instruction alive?
121 I->dropAllReferences(); // Nope, drop references...
122 if (PHINode *PN = dyn_cast<PHINode>(&*I)) {
123 // We don't want to leave PHI nodes in the program that have
124 // #arguments != #predecessors, so we remove them now.
125 //
126 PN->replaceAllUsesWith(Constant::getNullValue(PN->getType()));
127
128 // Delete the instruction...
129 I = BB->getInstList().erase(I);
130 Changed = true;
131 } else {
132 ++I;
133 }
134 } else {
135 ++I;
136 }
137 return Changed;
138}
139
Chris Lattner02e90d52001-06-30 06:39:11 +0000140
Chris Lattnerea54ab92002-05-07 22:11:39 +0000141// doADCE() - Run the Aggressive Dead Code Elimination algorithm, returning
Chris Lattnerf57b8452002-04-27 06:56:12 +0000142// true if the function was modified.
Chris Lattner02e90d52001-06-30 06:39:11 +0000143//
Chris Lattnerd9036a12002-05-22 21:32:16 +0000144bool ADCE::doADCE() {
145 bool MadeChanges = false;
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000146
Chris Lattnerf57b8452002-04-27 06:56:12 +0000147 // Iterate over all of the instructions in the function, eliminating trivially
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000148 // dead instructions, and marking instructions live that are known to be
149 // needed. Perform the walk in depth first order so that we avoid marking any
150 // instructions live in basic blocks that are unreachable. These blocks will
151 // be eliminated later, along with the instructions inside.
152 //
Chris Lattnerdfe81ab2002-05-06 17:27:57 +0000153 for (df_iterator<Function*> BBI = df_begin(Func), BBE = df_end(Func);
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000154 BBI != BBE; ++BBI) {
155 BasicBlock *BB = *BBI;
156 for (BasicBlock::iterator II = BB->begin(), EI = BB->end(); II != EI; ) {
Chris Lattnerf0a93ed2003-02-24 20:48:32 +0000157 if (II->mayWriteToMemory() || II->getOpcode() == Instruction::Ret) {
Chris Lattner7e708292002-06-25 16:13:24 +0000158 markInstructionLive(II);
Chris Lattnerea54ab92002-05-07 22:11:39 +0000159 ++II; // Increment the inst iterator if the inst wasn't deleted
Chris Lattner7e708292002-06-25 16:13:24 +0000160 } else if (isInstructionTriviallyDead(II)) {
Chris Lattnerea54ab92002-05-07 22:11:39 +0000161 // Remove the instruction from it's basic block...
Chris Lattner7e708292002-06-25 16:13:24 +0000162 II = BB->getInstList().erase(II);
Chris Lattnerd9036a12002-05-22 21:32:16 +0000163 ++NumInstRemoved;
Chris Lattnerea54ab92002-05-07 22:11:39 +0000164 MadeChanges = true;
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000165 } else {
Chris Lattnerea54ab92002-05-07 22:11:39 +0000166 ++II; // Increment the inst iterator if the inst wasn't deleted
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000167 }
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000168 }
169 }
170
Chris Lattnerf016ea42002-05-22 17:17:27 +0000171 DEBUG(cerr << "Processing work list\n");
Chris Lattner02e90d52001-06-30 06:39:11 +0000172
Chris Lattner72f1e992001-07-08 18:38:36 +0000173 // AliveBlocks - Set of basic blocks that we know have instructions that are
174 // alive in them...
175 //
Chris Lattner697954c2002-01-20 22:54:45 +0000176 std::set<BasicBlock*> AliveBlocks;
Chris Lattner72f1e992001-07-08 18:38:36 +0000177
Chris Lattner02e90d52001-06-30 06:39:11 +0000178 // Process the work list of instructions that just became live... if they
179 // became live, then that means that all of their operands are neccesary as
180 // well... make them live as well.
181 //
182 while (!WorkList.empty()) {
Chris Lattner72f1e992001-07-08 18:38:36 +0000183 Instruction *I = WorkList.back(); // Get an instruction that became live...
Chris Lattner02e90d52001-06-30 06:39:11 +0000184 WorkList.pop_back();
185
Chris Lattner72f1e992001-07-08 18:38:36 +0000186 BasicBlock *BB = I->getParent();
Chris Lattnerd9036a12002-05-22 21:32:16 +0000187 if (!AliveBlocks.count(BB)) { // Basic block not alive yet...
188 AliveBlocks.insert(BB); // Block is now ALIVE!
189 markBlockAlive(BB); // Make it so now!
Chris Lattner72f1e992001-07-08 18:38:36 +0000190 }
191
Chris Lattnerd9036a12002-05-22 21:32:16 +0000192 // PHI nodes are a special case, because the incoming values are actually
193 // defined in the predecessor nodes of this block, meaning that the PHI
194 // makes the predecessors alive.
195 //
196 if (PHINode *PN = dyn_cast<PHINode>(I))
197 for (pred_iterator PI = pred_begin(BB), PE = pred_end(BB); PI != PE; ++PI)
198 if (!AliveBlocks.count(*PI)) {
199 AliveBlocks.insert(BB); // Block is now ALIVE!
200 markBlockAlive(*PI);
201 }
202
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000203 // Loop over all of the operands of the live instruction, making sure that
204 // they are known to be alive as well...
205 //
Chris Lattnerea54ab92002-05-07 22:11:39 +0000206 for (unsigned op = 0, End = I->getNumOperands(); op != End; ++op)
Chris Lattner9636a912001-10-01 16:18:37 +0000207 if (Instruction *Operand = dyn_cast<Instruction>(I->getOperand(op)))
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000208 markInstructionLive(Operand);
Chris Lattner02e90d52001-06-30 06:39:11 +0000209 }
210
Chris Lattnerf016ea42002-05-22 17:17:27 +0000211 if (DebugFlag) {
212 cerr << "Current Function: X = Live\n";
213 for (Function::iterator I = Func->begin(), E = Func->end(); I != E; ++I)
Chris Lattner7e708292002-06-25 16:13:24 +0000214 for (BasicBlock::iterator BI = I->begin(), BE = I->end(); BI != BE; ++BI){
215 if (LiveSet.count(BI)) cerr << "X ";
Chris Lattnerf016ea42002-05-22 17:17:27 +0000216 cerr << *BI;
217 }
218 }
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000219
Chris Lattnerd9036a12002-05-22 21:32:16 +0000220 // Find the first postdominator of the entry node that is alive. Make it the
221 // new entry node...
Chris Lattner02e90d52001-06-30 06:39:11 +0000222 //
Chris Lattnerce6ef112002-07-26 18:40:14 +0000223 PostDominatorTree &DT = getAnalysis<PostDominatorTree>();
Chris Lattner02e90d52001-06-30 06:39:11 +0000224
Chris Lattner446698b2002-07-30 00:22:34 +0000225
226 if (AliveBlocks.size() == Func->size()) { // No dead blocks?
227 for (Function::iterator I = Func->begin(), E = Func->end(); I != E; ++I)
228 // Loop over all of the instructions in the function, telling dead
229 // instructions to drop their references. This is so that the next sweep
230 // over the program can safely delete dead instructions without other dead
231 // instructions still refering to them.
232 //
233 dropReferencesOfDeadInstructionsInLiveBlock(I);
234
235 } else { // If there are some blocks dead...
Chris Lattnerdb6e4d62002-08-14 21:35:02 +0000236 // If the entry node is dead, insert a new entry node to eliminate the entry
237 // node as a special case.
238 //
239 if (!AliveBlocks.count(&Func->front())) {
240 BasicBlock *NewEntry = new BasicBlock();
Chris Lattner9e6161c2002-09-10 23:46:10 +0000241 NewEntry->getInstList().push_back(new BranchInst(&Func->front()));
Chris Lattnerdb6e4d62002-08-14 21:35:02 +0000242 Func->getBasicBlockList().push_front(NewEntry);
243 AliveBlocks.insert(NewEntry); // This block is always alive!
244 }
Chris Lattnerd9036a12002-05-22 21:32:16 +0000245
246 // Loop over all of the alive blocks in the function. If any successor
247 // blocks are not alive, we adjust the outgoing branches to branch to the
248 // first live postdominator of the live block, adjusting any PHI nodes in
249 // the block to reflect this.
250 //
251 for (Function::iterator I = Func->begin(), E = Func->end(); I != E; ++I)
Chris Lattner7e708292002-06-25 16:13:24 +0000252 if (AliveBlocks.count(I)) {
253 BasicBlock *BB = I;
Chris Lattnerd9036a12002-05-22 21:32:16 +0000254 TerminatorInst *TI = BB->getTerminator();
255
Chris Lattner011de072002-07-29 22:31:39 +0000256 // Loop over all of the successors, looking for ones that are not alive.
257 // We cannot save the number of successors in the terminator instruction
258 // here because we may remove them if we don't have a postdominator...
259 //
260 for (unsigned i = 0; i != TI->getNumSuccessors(); ++i)
Chris Lattnerd9036a12002-05-22 21:32:16 +0000261 if (!AliveBlocks.count(TI->getSuccessor(i))) {
262 // Scan up the postdominator tree, looking for the first
263 // postdominator that is alive, and the last postdominator that is
264 // dead...
265 //
Chris Lattnerce6ef112002-07-26 18:40:14 +0000266 PostDominatorTree::Node *LastNode = DT[TI->getSuccessor(i)];
Chris Lattner011de072002-07-29 22:31:39 +0000267
268 // There is a special case here... if there IS no post-dominator for
269 // the block we have no owhere to point our branch to. Instead,
270 // convert it to a return. This can only happen if the code
271 // branched into an infinite loop. Note that this may not be
272 // desirable, because we _are_ altering the behavior of the code.
273 // This is a well known drawback of ADCE, so in the future if we
274 // choose to revisit the decision, this is where it should be.
Chris Lattnerd9036a12002-05-22 21:32:16 +0000275 //
Chris Lattner011de072002-07-29 22:31:39 +0000276 if (LastNode == 0) { // No postdominator!
277 // Call RemoveSuccessor to transmogrify the terminator instruction
278 // to not contain the outgoing branch, or to create a new
279 // terminator if the form fundementally changes (ie unconditional
280 // branch to return). Note that this will change a branch into an
281 // infinite loop into a return instruction!
282 //
283 RemoveSuccessor(TI, i);
284
285 // RemoveSuccessor may replace TI... make sure we have a fresh
286 // pointer... and e variable.
287 //
288 TI = BB->getTerminator();
289
290 // Rescan this successor...
291 --i;
292 } else {
293 PostDominatorTree::Node *NextNode = LastNode->getIDom();
294
295 while (!AliveBlocks.count(NextNode->getNode())) {
296 LastNode = NextNode;
297 NextNode = NextNode->getIDom();
298 }
299
300 // Get the basic blocks that we need...
301 BasicBlock *LastDead = LastNode->getNode();
302 BasicBlock *NextAlive = NextNode->getNode();
303
304 // Make the conditional branch now go to the next alive block...
305 TI->getSuccessor(i)->removePredecessor(BB);
306 TI->setSuccessor(i, NextAlive);
307
308 // If there are PHI nodes in NextAlive, we need to add entries to
309 // the PHI nodes for the new incoming edge. The incoming values
310 // should be identical to the incoming values for LastDead.
311 //
312 for (BasicBlock::iterator II = NextAlive->begin();
313 PHINode *PN = dyn_cast<PHINode>(&*II); ++II) {
314 // Get the incoming value for LastDead...
315 int OldIdx = PN->getBasicBlockIndex(LastDead);
316 assert(OldIdx != -1 && "LastDead is not a pred of NextAlive!");
317 Value *InVal = PN->getIncomingValue(OldIdx);
318
319 // Add an incoming value for BB now...
320 PN->addIncoming(InVal, BB);
321 }
Chris Lattnerd9036a12002-05-22 21:32:16 +0000322 }
323 }
Chris Lattner55547272002-05-10 15:37:35 +0000324
Chris Lattnerd9036a12002-05-22 21:32:16 +0000325 // Now loop over all of the instructions in the basic block, telling
326 // dead instructions to drop their references. This is so that the next
327 // sweep over the program can safely delete dead instructions without
328 // other dead instructions still refering to them.
329 //
Chris Lattner446698b2002-07-30 00:22:34 +0000330 dropReferencesOfDeadInstructionsInLiveBlock(BB);
Chris Lattnerd9036a12002-05-22 21:32:16 +0000331 }
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000332 }
333
Chris Lattnerd7f268d2003-01-23 02:12:18 +0000334 // We make changes if there are any dead blocks in the function...
335 if (unsigned NumDeadBlocks = Func->size() - AliveBlocks.size()) {
336 MadeChanges = true;
337 NumBlockRemoved += NumDeadBlocks;
338 }
339
340 // Loop over all of the basic blocks in the function, removing control flow
341 // edges to live blocks (also eliminating any entries in PHI functions in
342 // referenced blocks).
Chris Lattnerd9036a12002-05-22 21:32:16 +0000343 //
Chris Lattnerd7f268d2003-01-23 02:12:18 +0000344 for (Function::iterator BB = Func->begin(), E = Func->end(); BB != E; ++BB)
Chris Lattner84369b32002-05-28 21:38:16 +0000345 if (!AliveBlocks.count(BB)) {
Chris Lattnerd9036a12002-05-22 21:32:16 +0000346 // Remove all outgoing edges from this basic block and convert the
347 // terminator into a return instruction.
348 vector<BasicBlock*> Succs(succ_begin(BB), succ_end(BB));
349
350 if (!Succs.empty()) {
351 // Loop over all of the successors, removing this block from PHI node
352 // entries that might be in the block...
353 while (!Succs.empty()) {
354 Succs.back()->removePredecessor(BB);
355 Succs.pop_back();
356 }
357
358 // Delete the old terminator instruction...
Chris Lattner7e708292002-06-25 16:13:24 +0000359 BB->getInstList().pop_back();
Chris Lattnerd9036a12002-05-22 21:32:16 +0000360 const Type *RetTy = Func->getReturnType();
Chris Lattner9e6161c2002-09-10 23:46:10 +0000361 BB->getInstList().push_back(new ReturnInst(RetTy != Type::VoidTy ?
362 Constant::getNullValue(RetTy) : 0));
Chris Lattnerd9036a12002-05-22 21:32:16 +0000363 }
Chris Lattnerd9036a12002-05-22 21:32:16 +0000364 }
Chris Lattnerd7f268d2003-01-23 02:12:18 +0000365
366
367 // Loop over all of the basic blocks in the function, dropping references of
368 // the dead basic blocks. We must do this after the previous step to avoid
369 // dropping references to PHIs which still have entries...
370 //
371 for (Function::iterator BB = Func->begin(), E = Func->end(); BB != E; ++BB)
372 if (!AliveBlocks.count(BB))
373 BB->dropAllReferences();
Chris Lattner55547272002-05-10 15:37:35 +0000374
Chris Lattner84369b32002-05-28 21:38:16 +0000375 // Now loop through all of the blocks and delete the dead ones. We can safely
376 // do this now because we know that there are no references to dead blocks
377 // (because they have dropped all of their references... we also remove dead
378 // instructions from alive blocks.
Chris Lattnerb8259dd2001-09-09 22:26:47 +0000379 //
Chris Lattnerd9036a12002-05-22 21:32:16 +0000380 for (Function::iterator BI = Func->begin(); BI != Func->end(); )
Chris Lattner446698b2002-07-30 00:22:34 +0000381 if (!AliveBlocks.count(BI)) { // Delete dead blocks...
Chris Lattner7e708292002-06-25 16:13:24 +0000382 BI = Func->getBasicBlockList().erase(BI);
Chris Lattner446698b2002-07-30 00:22:34 +0000383 } else { // Scan alive blocks...
Chris Lattner7e708292002-06-25 16:13:24 +0000384 for (BasicBlock::iterator II = BI->begin(); II != --BI->end(); )
385 if (!LiveSet.count(II)) { // Is this instruction alive?
Chris Lattner84369b32002-05-28 21:38:16 +0000386 // Nope... remove the instruction from it's basic block...
Chris Lattner7e708292002-06-25 16:13:24 +0000387 II = BI->getInstList().erase(II);
Chris Lattner84369b32002-05-28 21:38:16 +0000388 ++NumInstRemoved;
389 MadeChanges = true;
390 } else {
391 ++II;
392 }
393
Chris Lattnerd9036a12002-05-22 21:32:16 +0000394 ++BI; // Increment iterator...
Chris Lattner84369b32002-05-28 21:38:16 +0000395 }
Chris Lattner55547272002-05-10 15:37:35 +0000396
Chris Lattnerd9036a12002-05-22 21:32:16 +0000397 return MadeChanges;
Chris Lattner02e90d52001-06-30 06:39:11 +0000398}