Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 1 | //===- DCE.cpp - Code to perform dead code elimination --------------------===// |
| 2 | // |
| 3 | // This file implements dead code elimination and basic block merging. |
| 4 | // |
| 5 | // Specifically, this: |
| 6 | // * removes definitions with no uses (including unused constants) |
| 7 | // * removes basic blocks with no predecessors |
| 8 | // * merges a basic block into its predecessor if there is only one and the |
| 9 | // predecessor only has one successor. |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 10 | // * Eliminates PHI nodes for basic blocks with a single predecessor |
| 11 | // * Eliminates a basic block that only contains an unconditional branch |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 12 | // |
| 13 | // TODO: This should REALLY be recursive instead of iterative. Right now, we |
| 14 | // scan linearly through values, removing unused ones as we go. The problem is |
| 15 | // that this may cause other earlier values to become unused. To make sure that |
| 16 | // we get them all, we iterate until things stop changing. Instead, when |
| 17 | // removing a value, recheck all of its operands to see if they are now unused. |
Chris Lattner | d36c91c | 2001-06-13 19:55:22 +0000 | [diff] [blame] | 18 | // Piece of cake, and more efficient as well. |
| 19 | // |
| 20 | // Note, this is not trivial, because we have to worry about invalidating |
| 21 | // iterators. :( |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 22 | // |
| 23 | //===----------------------------------------------------------------------===// |
| 24 | |
| 25 | #include "llvm/Module.h" |
| 26 | #include "llvm/Method.h" |
| 27 | #include "llvm/BasicBlock.h" |
| 28 | #include "llvm/iTerminators.h" |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 29 | #include "llvm/iOther.h" |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 30 | #include "llvm/Opt/AllOpts.h" |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 31 | #include "llvm/Assembly/Writer.h" |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 32 | #include "llvm/CFG.h" |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 33 | |
Chris Lattner | d36c91c | 2001-06-13 19:55:22 +0000 | [diff] [blame] | 34 | using namespace cfg; |
| 35 | |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 36 | struct ConstPoolDCE { |
| 37 | enum { EndOffs = 0 }; |
| 38 | static bool isDCEable(const Value *) { return true; } |
| 39 | }; |
| 40 | |
| 41 | struct BasicBlockDCE { |
| 42 | enum { EndOffs = 1 }; |
| 43 | static bool isDCEable(const Instruction *I) { |
| 44 | return !I->hasSideEffects(); |
| 45 | } |
| 46 | }; |
| 47 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 48 | |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 49 | template<class ValueSubclass, class ItemParentType, class DCEController> |
| 50 | static bool RemoveUnusedDefs(ValueHolder<ValueSubclass, ItemParentType> &Vals, |
| 51 | DCEController DCEControl) { |
| 52 | bool Changed = false; |
| 53 | typedef ValueHolder<ValueSubclass, ItemParentType> Container; |
| 54 | |
| 55 | int Offset = DCEController::EndOffs; |
| 56 | for (Container::iterator DI = Vals.begin(); DI != Vals.end()-Offset; ) { |
| 57 | // Look for un"used" definitions... |
| 58 | if ((*DI)->use_empty() && DCEController::isDCEable(*DI)) { |
| 59 | // Bye bye |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 60 | //cerr << "Removing: " << *DI; |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 61 | delete Vals.remove(DI); |
| 62 | Changed = true; |
| 63 | } else { |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 64 | ++DI; |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 65 | } |
| 66 | } |
| 67 | return Changed; |
| 68 | } |
| 69 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 70 | // RemoveSingularPHIs - This removes PHI nodes from basic blocks that have only |
| 71 | // a single predecessor. This means that the PHI node must only have a single |
| 72 | // RHS value and can be eliminated. |
| 73 | // |
| 74 | // This routine is very simple because we know that PHI nodes must be the first |
| 75 | // things in a basic block, if they are present. |
| 76 | // |
| 77 | static bool RemoveSingularPHIs(BasicBlock *BB) { |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 78 | pred_iterator PI(pred_begin(BB)); |
| 79 | if (PI == pred_end(BB) || ++PI != pred_end(BB)) |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 80 | return false; // More than one predecessor... |
| 81 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 82 | Instruction *I = BB->front(); |
| 83 | if (!I->isPHINode()) return false; // No PHI nodes |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 84 | |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 85 | //cerr << "Killing PHIs from " << BB; |
| 86 | //cerr << "Pred #0 = " << *pred_begin(BB); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 87 | |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 88 | //cerr << "Method == " << BB->getParent(); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 89 | |
| 90 | do { |
| 91 | PHINode *PN = (PHINode*)I; |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 92 | assert(PN->getOperand(2) == 0 && "PHI node should only have one value!"); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 93 | Value *V = PN->getOperand(0); |
| 94 | |
| 95 | PN->replaceAllUsesWith(V); // Replace PHI node with its single value. |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 96 | delete BB->getInstList().remove(BB->begin()); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 97 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 98 | I = BB->front(); |
| 99 | } while (I->isPHINode()); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 100 | |
| 101 | return true; // Yes, we nuked at least one phi node |
| 102 | } |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 103 | |
| 104 | bool DoRemoveUnusedConstants(SymTabValue *S) { |
| 105 | bool Changed = false; |
| 106 | ConstantPool &CP = S->getConstantPool(); |
| 107 | for (ConstantPool::plane_iterator PI = CP.begin(); PI != CP.end(); ++PI) |
| 108 | Changed |= RemoveUnusedDefs(**PI, ConstPoolDCE()); |
| 109 | return Changed; |
| 110 | } |
| 111 | |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 112 | static void ReplaceUsesWithConstant(Instruction *I) { |
| 113 | // Get the method level constant pool |
| 114 | ConstantPool &CP = I->getParent()->getParent()->getConstantPool(); |
| 115 | |
| 116 | ConstPoolVal *CPV = 0; |
| 117 | ConstantPool::PlaneType *P; |
| 118 | if (!CP.getPlane(I->getType(), P)) { // Does plane exist? |
| 119 | // Yes, is it empty? |
| 120 | if (!P->empty()) CPV = P->front(); |
| 121 | } |
| 122 | |
| 123 | if (CPV == 0) { // We don't have an existing constant to reuse. Just add one. |
| 124 | CPV = ConstPoolVal::getNullConstant(I->getType()); // Create a new constant |
| 125 | |
| 126 | // Add the new value to the constant pool... |
| 127 | CP.insert(CPV); |
| 128 | } |
| 129 | |
| 130 | // Make all users of this instruction reference the constant instead |
| 131 | I->replaceAllUsesWith(CPV); |
| 132 | } |
| 133 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 134 | // RemovePredecessorFromBlock - This function is called when we are about |
| 135 | // to remove a predecessor from a basic block. This function takes care of |
| 136 | // removing the predecessor from the PHI nodes in BB so that after the pred |
| 137 | // is removed, the number of PHI slots per bb is equal to the number of |
| 138 | // predecessors. |
| 139 | // |
| 140 | static void RemovePredecessorFromBlock(BasicBlock *BB, BasicBlock *Pred) { |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 141 | pred_iterator PI(pred_begin(BB)), EI(pred_end(BB)); |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 142 | unsigned max_idx; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 143 | |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 144 | //cerr << "RPFB: " << Pred << "From Block: " << BB; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 145 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 146 | // Loop over the rest of the predecssors until we run out, or until we find |
| 147 | // out that there are more than 2 predecessors. |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 148 | for (max_idx = 0; PI != EI && max_idx < 3; ++PI, ++max_idx) /*empty*/; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 149 | |
| 150 | // If there are exactly two predecessors, then we want to nuke the PHI nodes |
| 151 | // altogether. |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 152 | bool NukePHIs = max_idx == 2; |
| 153 | assert(max_idx != 0 && "PHI Node in block with 0 predecessors!?!?!"); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 154 | |
| 155 | // Okay, now we know that we need to remove predecessor #pred_idx from all |
| 156 | // PHI nodes. Iterate over each PHI node fixing them up |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 157 | BasicBlock::iterator II(BB->begin()); |
| 158 | for (; (*II)->isPHINode(); ++II) { |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 159 | PHINode *PN = (PHINode*)*II; |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 160 | PN->removeIncomingValue(BB); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 161 | |
| 162 | if (NukePHIs) { // Destroy the PHI altogether?? |
| 163 | assert(PN->getOperand(1) == 0 && "PHI node should only have one value!"); |
| 164 | Value *V = PN->getOperand(0); |
| 165 | |
| 166 | PN->replaceAllUsesWith(V); // Replace PHI node with its single value. |
| 167 | delete BB->getInstList().remove(II); |
| 168 | } |
| 169 | } |
| 170 | } |
| 171 | |
| 172 | // PropogatePredecessors - This gets "Succ" ready to have the predecessors from |
| 173 | // "BB". This is a little tricky because "Succ" has PHI nodes, which need to |
| 174 | // have extra slots added to them to hold the merge edges from BB's |
| 175 | // predecessors. |
| 176 | // |
| 177 | // Assumption: BB is the single predecessor of Succ. |
| 178 | // |
| 179 | static void PropogatePredecessorsForPHIs(BasicBlock *BB, BasicBlock *Succ) { |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 180 | assert(Succ->front()->isPHINode() && "Only works on PHId BBs!"); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 181 | |
| 182 | // If there is more than one predecessor, and there are PHI nodes in |
| 183 | // the successor, then we need to add incoming edges for the PHI nodes |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 184 | // |
| 185 | const vector<BasicBlock*> BBPreds(pred_begin(BB), pred_end(BB)); |
| 186 | |
| 187 | BasicBlock::iterator I = Succ->begin(); |
| 188 | do { // Loop over all of the PHI nodes in the successor BB |
| 189 | PHINode *PN = (PHINode*)*I; |
| 190 | Value *OldVal = PN->removeIncomingValue(BB); |
| 191 | assert(OldVal && "No entry in PHI for Pred BB!"); |
| 192 | |
| 193 | for (vector<BasicBlock*>::const_iterator PredI = BBPreds.begin(), |
| 194 | End = BBPreds.end(); PredI != End; ++PredI) { |
| 195 | // Add an incoming value for each of the new incoming values... |
| 196 | PN->addIncoming(OldVal, *PredI); |
| 197 | } |
| 198 | |
| 199 | ++I; |
| 200 | } while ((*I)->isPHINode()); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 201 | } |
| 202 | |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 203 | static bool DoDCEPass(Method *M) { |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 204 | Method::iterator BBIt, BBEnd = M->end(); |
| 205 | if (M->begin() == BBEnd) return false; // Nothing to do |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 206 | bool Changed = false; |
| 207 | |
| 208 | // Loop through now and remove instructions that have no uses... |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 209 | for (BBIt = M->begin(); BBIt != BBEnd; ++BBIt) { |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 210 | Changed |= RemoveUnusedDefs((*BBIt)->getInstList(), BasicBlockDCE()); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 211 | Changed |= RemoveSingularPHIs(*BBIt); |
| 212 | } |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 213 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 214 | // Loop over all of the basic blocks (except the first one) and remove them |
| 215 | // if they are unneeded... |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 216 | // |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 217 | for (BBIt = M->begin(), ++BBIt; BBIt != M->end(); ++BBIt) { |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 218 | BasicBlock *BB = *BBIt; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 219 | assert(BB->getTerminator() && "Degenerate basic block encountered!"); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 220 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 221 | #if 0 // This is know to basically work? |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 222 | // Remove basic blocks that have no predecessors... which are unreachable. |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 223 | if (pred_begin(BB) == pred_end(BB) && |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 224 | !BB->hasConstantPoolReferences() && 0) { |
| 225 | cerr << "Removing BB: \n" << BB; |
| 226 | |
| 227 | // Loop through all of our successors and make sure they know that one |
| 228 | // of their predecessors is going away. |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 229 | for_each(succ_begin(BB), succ_end(BB), |
| 230 | bind_2nd(RemovePredecessorFromBlock, BB)); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 231 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 232 | while (!BB->empty()) { |
| 233 | Instruction *I = BB->front(); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 234 | // If this instruction is used, replace uses with an arbitrary |
| 235 | // constant value. Because control flow can't get here, we don't care |
| 236 | // what we replace the value with. |
| 237 | if (!I->use_empty()) ReplaceUsesWithConstant(I); |
| 238 | |
| 239 | // Remove the instruction from the basic block |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 240 | delete BB->getInstList().remove(BB->begin()); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 241 | } |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 242 | delete M->getBasicBlocks().remove(BBIt); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 243 | --BBIt; // remove puts use on the next block, we want the previous one |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 244 | Changed = true; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 245 | continue; |
| 246 | } |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 247 | #endif |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 248 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 249 | #if 0 // This has problems |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 250 | // Check to see if this block has no instructions and only a single |
| 251 | // successor. If so, replace block references with successor. |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 252 | succ_iterator SI(succ_begin(BB)); |
| 253 | if (SI != succ_end(BB) && ++SI == succ_end(BB)) { // One succ? |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 254 | Instruction *I = BB->front(); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 255 | if (I->isTerminator()) { // Terminator is the only instruction! |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 256 | BasicBlock *Succ = *succ_begin(BB); // There is exactly one successor |
| 257 | cerr << "Killing Trivial BB: \n" << BB; |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 258 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 259 | if (Succ->front()->isPHINode()) { |
| 260 | // If our successor has PHI nodes, then we need to update them to |
| 261 | // include entries for BB's predecessors, not for BB itself. |
| 262 | // |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 263 | PropogatePredecessorsForPHIs(BB, Succ); |
| 264 | } |
| 265 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 266 | BB->replaceAllUsesWith(Succ); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 267 | |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 268 | BB = M->getBasicBlocks().remove(BBIt); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 269 | --BBIt; // remove puts use on the next block, we want the previous one |
| 270 | |
| 271 | if (BB->hasName() && !Succ->hasName()) // Transfer name if we can |
| 272 | Succ->setName(BB->getName()); |
| 273 | delete BB; // Delete basic block |
| 274 | |
| 275 | cerr << "Method after removal: \n" << M; |
| 276 | Changed = true; |
| 277 | continue; |
| 278 | } |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 279 | } |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 280 | #endif |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 281 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 282 | // Merge basic blocks into their predecessor if there is only one pred, |
| 283 | // and if there is only one successor of the predecessor. |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 284 | pred_iterator PI(pred_begin(BB)); |
| 285 | if (PI != pred_end(BB) && *PI != BB && // Not empty? Not same BB? |
| 286 | ++PI == pred_end(BB) && !BB->hasConstantPoolReferences()) { |
| 287 | BasicBlock *Pred = *pred_begin(BB); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 288 | TerminatorInst *Term = Pred->getTerminator(); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 289 | assert(Term != 0 && "malformed basic block without terminator!"); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 290 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 291 | // Does the predecessor block only have a single successor? |
Chris Lattner | dac6dda | 2001-06-07 21:18:45 +0000 | [diff] [blame] | 292 | succ_iterator SI(succ_begin(Pred)); |
| 293 | if (++SI == succ_end(Pred)) { |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 294 | //cerr << "Merging: " << BB << "into: " << Pred; |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 295 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 296 | // Delete the unconditianal branch from the predecessor... |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 297 | BasicBlock::iterator DI = Pred->end(); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 298 | assert(Pred->getTerminator() && |
| 299 | "Degenerate basic block encountered!"); // Empty bb??? |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 300 | delete Pred->getInstList().remove(--DI); // Destroy uncond branch |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 301 | |
| 302 | // Move all definitions in the succecessor to the predecessor... |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 303 | while (!BB->empty()) { |
| 304 | DI = BB->begin(); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 305 | Instruction *Def = BB->getInstList().remove(DI); // Remove from front |
| 306 | Pred->getInstList().push_back(Def); // Add to end... |
| 307 | } |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 308 | |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 309 | // Remove basic block from the method... and advance iterator to the |
| 310 | // next valid block... |
Chris Lattner | 7fc9fe3 | 2001-06-27 23:41:11 +0000 | [diff] [blame^] | 311 | BB = M->getBasicBlocks().remove(BBIt); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 312 | --BBIt; // remove puts us on the NEXT bb. We want the prev BB |
| 313 | Changed = true; |
Chris Lattner | ee976f3 | 2001-06-11 15:04:40 +0000 | [diff] [blame] | 314 | |
| 315 | // Make all PHI nodes that refered to BB now refer to Pred as their |
| 316 | // source... |
| 317 | BB->replaceAllUsesWith(Pred); |
Chris Lattner | f155e13 | 2001-06-07 16:59:26 +0000 | [diff] [blame] | 318 | |
| 319 | // Inherit predecessors name if it exists... |
| 320 | if (BB->hasName() && !Pred->hasName()) Pred->setName(BB->getName()); |
| 321 | |
| 322 | // You ARE the weakest link... goodbye |
| 323 | delete BB; |
Chris Lattner | d36c91c | 2001-06-13 19:55:22 +0000 | [diff] [blame] | 324 | |
Chris Lattner | 5ef7afb | 2001-06-20 23:09:39 +0000 | [diff] [blame] | 325 | //WriteToVCG(M, "MergedInto"); |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 326 | } |
Chris Lattner | 0095054 | 2001-06-06 20:29:01 +0000 | [diff] [blame] | 327 | } |
| 328 | } |
| 329 | |
| 330 | // Remove unused constants |
| 331 | Changed |= DoRemoveUnusedConstants(M); |
| 332 | return Changed; |
| 333 | } |
| 334 | |
| 335 | |
| 336 | // It is possible that we may require multiple passes over the code to fully |
| 337 | // eliminate dead code. Iterate until we are done. |
| 338 | // |
| 339 | bool DoDeadCodeElimination(Method *M) { |
| 340 | bool Changed = false; |
| 341 | while (DoDCEPass(M)) Changed = true; |
| 342 | return Changed; |
| 343 | } |
| 344 | |
| 345 | bool DoDeadCodeElimination(Module *C) { |
| 346 | bool Val = ApplyOptToAllMethods(C, DoDeadCodeElimination); |
| 347 | while (DoRemoveUnusedConstants(C)) Val = true; |
| 348 | return Val; |
| 349 | } |