fix ConstantFoldCompareInstOperands to take the LHS/RHS as
[oota-llvm.git] / lib / Transforms / Utils / CloneFunction.cpp
1 //===- CloneFunction.cpp - Clone a function into another function ---------===//
2 //
3 //                     The LLVM Compiler Infrastructure
4 //
5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details.
7 //
8 //===----------------------------------------------------------------------===//
9 //
10 // This file implements the CloneFunctionInto interface, which is used as the
11 // low-level function cloner.  This is used by the CloneFunction and function
12 // inliner to do the dirty work of copying the body of a function around.
13 //
14 //===----------------------------------------------------------------------===//
15
16 #include "llvm/Transforms/Utils/Cloning.h"
17 #include "llvm/Constants.h"
18 #include "llvm/DerivedTypes.h"
19 #include "llvm/Instructions.h"
20 #include "llvm/IntrinsicInst.h"
21 #include "llvm/GlobalVariable.h"
22 #include "llvm/Function.h"
23 #include "llvm/Support/CFG.h"
24 #include "llvm/Transforms/Utils/ValueMapper.h"
25 #include "llvm/Analysis/ConstantFolding.h"
26 #include "llvm/Analysis/DebugInfo.h"
27 #include "llvm/ADT/SmallVector.h"
28 #include <map>
29 using namespace llvm;
30
31 // CloneBasicBlock - See comments in Cloning.h
32 BasicBlock *llvm::CloneBasicBlock(const BasicBlock *BB,
33                                   DenseMap<const Value*, Value*> &ValueMap,
34                                   const char *NameSuffix, Function *F,
35                                   ClonedCodeInfo *CodeInfo) {
36   BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "", F);
37   if (BB->hasName()) NewBB->setName(BB->getName()+NameSuffix);
38
39   bool hasCalls = false, hasDynamicAllocas = false, hasStaticAllocas = false;
40   
41   // Loop over all instructions, and copy them over.
42   for (BasicBlock::const_iterator II = BB->begin(), IE = BB->end();
43        II != IE; ++II) {
44     Instruction *NewInst = II->clone();
45     if (II->hasName())
46       NewInst->setName(II->getName()+NameSuffix);
47     NewBB->getInstList().push_back(NewInst);
48     ValueMap[II] = NewInst;                // Add instruction map to value.
49     
50     hasCalls |= (isa<CallInst>(II) && !isa<DbgInfoIntrinsic>(II));
51     if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) {
52       if (isa<ConstantInt>(AI->getArraySize()))
53         hasStaticAllocas = true;
54       else
55         hasDynamicAllocas = true;
56     }
57   }
58   
59   if (CodeInfo) {
60     CodeInfo->ContainsCalls          |= hasCalls;
61     CodeInfo->ContainsUnwinds        |= isa<UnwindInst>(BB->getTerminator());
62     CodeInfo->ContainsDynamicAllocas |= hasDynamicAllocas;
63     CodeInfo->ContainsDynamicAllocas |= hasStaticAllocas && 
64                                         BB != &BB->getParent()->getEntryBlock();
65   }
66   return NewBB;
67 }
68
69 // Clone OldFunc into NewFunc, transforming the old arguments into references to
70 // ArgMap values.
71 //
72 void llvm::CloneFunctionInto(Function *NewFunc, const Function *OldFunc,
73                              DenseMap<const Value*, Value*> &ValueMap,
74                              SmallVectorImpl<ReturnInst*> &Returns,
75                              const char *NameSuffix, ClonedCodeInfo *CodeInfo) {
76   assert(NameSuffix && "NameSuffix cannot be null!");
77
78 #ifndef NDEBUG
79   for (Function::const_arg_iterator I = OldFunc->arg_begin(), 
80        E = OldFunc->arg_end(); I != E; ++I)
81     assert(ValueMap.count(I) && "No mapping from source argument specified!");
82 #endif
83
84   // Clone any attributes.
85   if (NewFunc->arg_size() == OldFunc->arg_size())
86     NewFunc->copyAttributesFrom(OldFunc);
87   else {
88     //Some arguments were deleted with the ValueMap. Copy arguments one by one
89     for (Function::const_arg_iterator I = OldFunc->arg_begin(), 
90            E = OldFunc->arg_end(); I != E; ++I)
91       if (Argument* Anew = dyn_cast<Argument>(ValueMap[I]))
92         Anew->addAttr( OldFunc->getAttributes()
93                        .getParamAttributes(I->getArgNo() + 1));
94     NewFunc->setAttributes(NewFunc->getAttributes()
95                            .addAttr(0, OldFunc->getAttributes()
96                                      .getRetAttributes()));
97     NewFunc->setAttributes(NewFunc->getAttributes()
98                            .addAttr(~0, OldFunc->getAttributes()
99                                      .getFnAttributes()));
100
101   }
102
103   // Loop over all of the basic blocks in the function, cloning them as
104   // appropriate.  Note that we save BE this way in order to handle cloning of
105   // recursive functions into themselves.
106   //
107   for (Function::const_iterator BI = OldFunc->begin(), BE = OldFunc->end();
108        BI != BE; ++BI) {
109     const BasicBlock &BB = *BI;
110
111     // Create a new basic block and copy instructions into it!
112     BasicBlock *CBB = CloneBasicBlock(&BB, ValueMap, NameSuffix, NewFunc,
113                                       CodeInfo);
114     ValueMap[&BB] = CBB;                       // Add basic block mapping.
115
116     if (ReturnInst *RI = dyn_cast<ReturnInst>(CBB->getTerminator()))
117       Returns.push_back(RI);
118   }
119
120   // Loop over all of the instructions in the function, fixing up operand
121   // references as we go.  This uses ValueMap to do all the hard work.
122   //
123   for (Function::iterator BB = cast<BasicBlock>(ValueMap[OldFunc->begin()]),
124          BE = NewFunc->end(); BB != BE; ++BB)
125     // Loop over all instructions, fixing each one as we find it...
126     for (BasicBlock::iterator II = BB->begin(); II != BB->end(); ++II)
127       RemapInstruction(II, ValueMap);
128 }
129
130 /// CloneFunction - Return a copy of the specified function, but without
131 /// embedding the function into another module.  Also, any references specified
132 /// in the ValueMap are changed to refer to their mapped value instead of the
133 /// original one.  If any of the arguments to the function are in the ValueMap,
134 /// the arguments are deleted from the resultant function.  The ValueMap is
135 /// updated to include mappings from all of the instructions and basicblocks in
136 /// the function from their old to new values.
137 ///
138 Function *llvm::CloneFunction(const Function *F,
139                               DenseMap<const Value*, Value*> &ValueMap,
140                               ClonedCodeInfo *CodeInfo) {
141   std::vector<const Type*> ArgTypes;
142
143   // The user might be deleting arguments to the function by specifying them in
144   // the ValueMap.  If so, we need to not add the arguments to the arg ty vector
145   //
146   for (Function::const_arg_iterator I = F->arg_begin(), E = F->arg_end();
147        I != E; ++I)
148     if (ValueMap.count(I) == 0)  // Haven't mapped the argument to anything yet?
149       ArgTypes.push_back(I->getType());
150
151   // Create a new function type...
152   FunctionType *FTy = FunctionType::get(F->getFunctionType()->getReturnType(),
153                                     ArgTypes, F->getFunctionType()->isVarArg());
154
155   // Create the new function...
156   Function *NewF = Function::Create(FTy, F->getLinkage(), F->getName());
157
158   // Loop over the arguments, copying the names of the mapped arguments over...
159   Function::arg_iterator DestI = NewF->arg_begin();
160   for (Function::const_arg_iterator I = F->arg_begin(), E = F->arg_end();
161        I != E; ++I)
162     if (ValueMap.count(I) == 0) {   // Is this argument preserved?
163       DestI->setName(I->getName()); // Copy the name over...
164       ValueMap[I] = DestI++;        // Add mapping to ValueMap
165     }
166
167   SmallVector<ReturnInst*, 8> Returns;  // Ignore returns cloned.
168   CloneFunctionInto(NewF, F, ValueMap, Returns, "", CodeInfo);
169   return NewF;
170 }
171
172
173
174 namespace {
175   /// PruningFunctionCloner - This class is a private class used to implement
176   /// the CloneAndPruneFunctionInto method.
177   struct PruningFunctionCloner {
178     Function *NewFunc;
179     const Function *OldFunc;
180     DenseMap<const Value*, Value*> &ValueMap;
181     SmallVectorImpl<ReturnInst*> &Returns;
182     const char *NameSuffix;
183     ClonedCodeInfo *CodeInfo;
184     const TargetData *TD;
185     Value *DbgFnStart;
186   public:
187     PruningFunctionCloner(Function *newFunc, const Function *oldFunc,
188                           DenseMap<const Value*, Value*> &valueMap,
189                           SmallVectorImpl<ReturnInst*> &returns,
190                           const char *nameSuffix, 
191                           ClonedCodeInfo *codeInfo,
192                           const TargetData *td)
193     : NewFunc(newFunc), OldFunc(oldFunc), ValueMap(valueMap), Returns(returns),
194       NameSuffix(nameSuffix), CodeInfo(codeInfo), TD(td), DbgFnStart(NULL) {
195     }
196
197     /// CloneBlock - The specified block is found to be reachable, clone it and
198     /// anything that it can reach.
199     void CloneBlock(const BasicBlock *BB,
200                     std::vector<const BasicBlock*> &ToClone);
201     
202   public:
203     /// ConstantFoldMappedInstruction - Constant fold the specified instruction,
204     /// mapping its operands through ValueMap if they are available.
205     Constant *ConstantFoldMappedInstruction(const Instruction *I);
206   };
207 }
208
209 /// CloneBlock - The specified block is found to be reachable, clone it and
210 /// anything that it can reach.
211 void PruningFunctionCloner::CloneBlock(const BasicBlock *BB,
212                                        std::vector<const BasicBlock*> &ToClone){
213   Value *&BBEntry = ValueMap[BB];
214
215   // Have we already cloned this block?
216   if (BBEntry) return;
217   
218   // Nope, clone it now.
219   BasicBlock *NewBB;
220   BBEntry = NewBB = BasicBlock::Create(BB->getContext());
221   if (BB->hasName()) NewBB->setName(BB->getName()+NameSuffix);
222
223   bool hasCalls = false, hasDynamicAllocas = false, hasStaticAllocas = false;
224   
225   // Loop over all instructions, and copy them over, DCE'ing as we go.  This
226   // loop doesn't include the terminator.
227   for (BasicBlock::const_iterator II = BB->begin(), IE = --BB->end();
228        II != IE; ++II) {
229     // If this instruction constant folds, don't bother cloning the instruction,
230     // instead, just add the constant to the value map.
231     if (Constant *C = ConstantFoldMappedInstruction(II)) {
232       ValueMap[II] = C;
233       continue;
234     }
235
236     // Do not clone llvm.dbg.region.end. It will be adjusted by the inliner.
237     if (const DbgFuncStartInst *DFSI = dyn_cast<DbgFuncStartInst>(II)) {
238       if (DbgFnStart == NULL) {
239         DISubprogram SP(DFSI->getSubprogram());
240         if (SP.describes(BB->getParent()))
241           DbgFnStart = DFSI->getSubprogram();
242       }
243     } 
244     if (const DbgRegionEndInst *DREIS = dyn_cast<DbgRegionEndInst>(II)) {
245       if (DREIS->getContext() == DbgFnStart)
246         continue;
247     }
248       
249     Instruction *NewInst = II->clone();
250     if (II->hasName())
251       NewInst->setName(II->getName()+NameSuffix);
252     NewBB->getInstList().push_back(NewInst);
253     ValueMap[II] = NewInst;                // Add instruction map to value.
254     
255     hasCalls |= (isa<CallInst>(II) && !isa<DbgInfoIntrinsic>(II));
256     if (const AllocaInst *AI = dyn_cast<AllocaInst>(II)) {
257       if (isa<ConstantInt>(AI->getArraySize()))
258         hasStaticAllocas = true;
259       else
260         hasDynamicAllocas = true;
261     }
262   }
263   
264   // Finally, clone over the terminator.
265   const TerminatorInst *OldTI = BB->getTerminator();
266   bool TerminatorDone = false;
267   if (const BranchInst *BI = dyn_cast<BranchInst>(OldTI)) {
268     if (BI->isConditional()) {
269       // If the condition was a known constant in the callee...
270       ConstantInt *Cond = dyn_cast<ConstantInt>(BI->getCondition());
271       // Or is a known constant in the caller...
272       if (Cond == 0)  
273         Cond = dyn_cast_or_null<ConstantInt>(ValueMap[BI->getCondition()]);
274
275       // Constant fold to uncond branch!
276       if (Cond) {
277         BasicBlock *Dest = BI->getSuccessor(!Cond->getZExtValue());
278         ValueMap[OldTI] = BranchInst::Create(Dest, NewBB);
279         ToClone.push_back(Dest);
280         TerminatorDone = true;
281       }
282     }
283   } else if (const SwitchInst *SI = dyn_cast<SwitchInst>(OldTI)) {
284     // If switching on a value known constant in the caller.
285     ConstantInt *Cond = dyn_cast<ConstantInt>(SI->getCondition());
286     if (Cond == 0)  // Or known constant after constant prop in the callee...
287       Cond = dyn_cast_or_null<ConstantInt>(ValueMap[SI->getCondition()]);
288     if (Cond) {     // Constant fold to uncond branch!
289       BasicBlock *Dest = SI->getSuccessor(SI->findCaseValue(Cond));
290       ValueMap[OldTI] = BranchInst::Create(Dest, NewBB);
291       ToClone.push_back(Dest);
292       TerminatorDone = true;
293     }
294   }
295   
296   if (!TerminatorDone) {
297     Instruction *NewInst = OldTI->clone();
298     if (OldTI->hasName())
299       NewInst->setName(OldTI->getName()+NameSuffix);
300     NewBB->getInstList().push_back(NewInst);
301     ValueMap[OldTI] = NewInst;             // Add instruction map to value.
302     
303     // Recursively clone any reachable successor blocks.
304     const TerminatorInst *TI = BB->getTerminator();
305     for (unsigned i = 0, e = TI->getNumSuccessors(); i != e; ++i)
306       ToClone.push_back(TI->getSuccessor(i));
307   }
308   
309   if (CodeInfo) {
310     CodeInfo->ContainsCalls          |= hasCalls;
311     CodeInfo->ContainsUnwinds        |= isa<UnwindInst>(OldTI);
312     CodeInfo->ContainsDynamicAllocas |= hasDynamicAllocas;
313     CodeInfo->ContainsDynamicAllocas |= hasStaticAllocas && 
314       BB != &BB->getParent()->front();
315   }
316   
317   if (ReturnInst *RI = dyn_cast<ReturnInst>(NewBB->getTerminator()))
318     Returns.push_back(RI);
319 }
320
321 /// ConstantFoldMappedInstruction - Constant fold the specified instruction,
322 /// mapping its operands through ValueMap if they are available.
323 Constant *PruningFunctionCloner::
324 ConstantFoldMappedInstruction(const Instruction *I) {
325   SmallVector<Constant*, 8> Ops;
326   for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
327     if (Constant *Op = dyn_cast_or_null<Constant>(MapValue(I->getOperand(i),
328                                                            ValueMap)))
329       Ops.push_back(Op);
330     else
331       return 0;  // All operands not constant!
332
333   if (const CmpInst *CI = dyn_cast<CmpInst>(I))
334     return ConstantFoldCompareInstOperands(CI->getPredicate(), Ops[0], Ops[1],
335                                            TD);
336
337   if (const LoadInst *LI = dyn_cast<LoadInst>(I))
338     if (ConstantExpr *CE = dyn_cast<ConstantExpr>(Ops[0]))
339       if (!LI->isVolatile() && CE->getOpcode() == Instruction::GetElementPtr)
340         if (GlobalVariable *GV = dyn_cast<GlobalVariable>(CE->getOperand(0)))
341           if (GV->isConstant() && GV->hasDefinitiveInitializer())
342             return ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(),
343                                                           CE);
344
345   return ConstantFoldInstOperands(I->getOpcode(), I->getType(), &Ops[0],
346                                   Ops.size(), TD);
347 }
348
349 /// CloneAndPruneFunctionInto - This works exactly like CloneFunctionInto,
350 /// except that it does some simple constant prop and DCE on the fly.  The
351 /// effect of this is to copy significantly less code in cases where (for
352 /// example) a function call with constant arguments is inlined, and those
353 /// constant arguments cause a significant amount of code in the callee to be
354 /// dead.  Since this doesn't produce an exact copy of the input, it can't be
355 /// used for things like CloneFunction or CloneModule.
356 void llvm::CloneAndPruneFunctionInto(Function *NewFunc, const Function *OldFunc,
357                                      DenseMap<const Value*, Value*> &ValueMap,
358                                      SmallVectorImpl<ReturnInst*> &Returns,
359                                      const char *NameSuffix, 
360                                      ClonedCodeInfo *CodeInfo,
361                                      const TargetData *TD) {
362   assert(NameSuffix && "NameSuffix cannot be null!");
363   
364 #ifndef NDEBUG
365   for (Function::const_arg_iterator II = OldFunc->arg_begin(), 
366        E = OldFunc->arg_end(); II != E; ++II)
367     assert(ValueMap.count(II) && "No mapping from source argument specified!");
368 #endif
369
370   PruningFunctionCloner PFC(NewFunc, OldFunc, ValueMap, Returns,
371                             NameSuffix, CodeInfo, TD);
372
373   // Clone the entry block, and anything recursively reachable from it.
374   std::vector<const BasicBlock*> CloneWorklist;
375   CloneWorklist.push_back(&OldFunc->getEntryBlock());
376   while (!CloneWorklist.empty()) {
377     const BasicBlock *BB = CloneWorklist.back();
378     CloneWorklist.pop_back();
379     PFC.CloneBlock(BB, CloneWorklist);
380   }
381   
382   // Loop over all of the basic blocks in the old function.  If the block was
383   // reachable, we have cloned it and the old block is now in the value map:
384   // insert it into the new function in the right order.  If not, ignore it.
385   //
386   // Defer PHI resolution until rest of function is resolved.
387   SmallVector<const PHINode*, 16> PHIToResolve;
388   for (Function::const_iterator BI = OldFunc->begin(), BE = OldFunc->end();
389        BI != BE; ++BI) {
390     BasicBlock *NewBB = cast_or_null<BasicBlock>(ValueMap[BI]);
391     if (NewBB == 0) continue;  // Dead block.
392
393     // Add the new block to the new function.
394     NewFunc->getBasicBlockList().push_back(NewBB);
395     
396     // Loop over all of the instructions in the block, fixing up operand
397     // references as we go.  This uses ValueMap to do all the hard work.
398     //
399     BasicBlock::iterator I = NewBB->begin();
400     
401     // Handle PHI nodes specially, as we have to remove references to dead
402     // blocks.
403     if (PHINode *PN = dyn_cast<PHINode>(I)) {
404       // Skip over all PHI nodes, remembering them for later.
405       BasicBlock::const_iterator OldI = BI->begin();
406       for (; (PN = dyn_cast<PHINode>(I)); ++I, ++OldI)
407         PHIToResolve.push_back(cast<PHINode>(OldI));
408     }
409     
410     // Otherwise, remap the rest of the instructions normally.
411     for (; I != NewBB->end(); ++I)
412       RemapInstruction(I, ValueMap);
413   }
414   
415   // Defer PHI resolution until rest of function is resolved, PHI resolution
416   // requires the CFG to be up-to-date.
417   for (unsigned phino = 0, e = PHIToResolve.size(); phino != e; ) {
418     const PHINode *OPN = PHIToResolve[phino];
419     unsigned NumPreds = OPN->getNumIncomingValues();
420     const BasicBlock *OldBB = OPN->getParent();
421     BasicBlock *NewBB = cast<BasicBlock>(ValueMap[OldBB]);
422
423     // Map operands for blocks that are live and remove operands for blocks
424     // that are dead.
425     for (; phino != PHIToResolve.size() &&
426          PHIToResolve[phino]->getParent() == OldBB; ++phino) {
427       OPN = PHIToResolve[phino];
428       PHINode *PN = cast<PHINode>(ValueMap[OPN]);
429       for (unsigned pred = 0, e = NumPreds; pred != e; ++pred) {
430         if (BasicBlock *MappedBlock = 
431             cast_or_null<BasicBlock>(ValueMap[PN->getIncomingBlock(pred)])) {
432           Value *InVal = MapValue(PN->getIncomingValue(pred),
433                                   ValueMap);
434           assert(InVal && "Unknown input value?");
435           PN->setIncomingValue(pred, InVal);
436           PN->setIncomingBlock(pred, MappedBlock);
437         } else {
438           PN->removeIncomingValue(pred, false);
439           --pred, --e;  // Revisit the next entry.
440         }
441       } 
442     }
443     
444     // The loop above has removed PHI entries for those blocks that are dead
445     // and has updated others.  However, if a block is live (i.e. copied over)
446     // but its terminator has been changed to not go to this block, then our
447     // phi nodes will have invalid entries.  Update the PHI nodes in this
448     // case.
449     PHINode *PN = cast<PHINode>(NewBB->begin());
450     NumPreds = std::distance(pred_begin(NewBB), pred_end(NewBB));
451     if (NumPreds != PN->getNumIncomingValues()) {
452       assert(NumPreds < PN->getNumIncomingValues());
453       // Count how many times each predecessor comes to this block.
454       std::map<BasicBlock*, unsigned> PredCount;
455       for (pred_iterator PI = pred_begin(NewBB), E = pred_end(NewBB);
456            PI != E; ++PI)
457         --PredCount[*PI];
458       
459       // Figure out how many entries to remove from each PHI.
460       for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
461         ++PredCount[PN->getIncomingBlock(i)];
462       
463       // At this point, the excess predecessor entries are positive in the
464       // map.  Loop over all of the PHIs and remove excess predecessor
465       // entries.
466       BasicBlock::iterator I = NewBB->begin();
467       for (; (PN = dyn_cast<PHINode>(I)); ++I) {
468         for (std::map<BasicBlock*, unsigned>::iterator PCI =PredCount.begin(),
469              E = PredCount.end(); PCI != E; ++PCI) {
470           BasicBlock *Pred     = PCI->first;
471           for (unsigned NumToRemove = PCI->second; NumToRemove; --NumToRemove)
472             PN->removeIncomingValue(Pred, false);
473         }
474       }
475     }
476     
477     // If the loops above have made these phi nodes have 0 or 1 operand,
478     // replace them with undef or the input value.  We must do this for
479     // correctness, because 0-operand phis are not valid.
480     PN = cast<PHINode>(NewBB->begin());
481     if (PN->getNumIncomingValues() == 0) {
482       BasicBlock::iterator I = NewBB->begin();
483       BasicBlock::const_iterator OldI = OldBB->begin();
484       while ((PN = dyn_cast<PHINode>(I++))) {
485         Value *NV = UndefValue::get(PN->getType());
486         PN->replaceAllUsesWith(NV);
487         assert(ValueMap[OldI] == PN && "ValueMap mismatch");
488         ValueMap[OldI] = NV;
489         PN->eraseFromParent();
490         ++OldI;
491       }
492     }
493     // NOTE: We cannot eliminate single entry phi nodes here, because of
494     // ValueMap.  Single entry phi nodes can have multiple ValueMap entries
495     // pointing at them.  Thus, deleting one would require scanning the ValueMap
496     // to update any entries in it that would require that.  This would be
497     // really slow.
498   }
499   
500   // Now that the inlined function body has been fully constructed, go through
501   // and zap unconditional fall-through branches.  This happen all the time when
502   // specializing code: code specialization turns conditional branches into
503   // uncond branches, and this code folds them.
504   Function::iterator I = cast<BasicBlock>(ValueMap[&OldFunc->getEntryBlock()]);
505   while (I != NewFunc->end()) {
506     BranchInst *BI = dyn_cast<BranchInst>(I->getTerminator());
507     if (!BI || BI->isConditional()) { ++I; continue; }
508     
509     // Note that we can't eliminate uncond branches if the destination has
510     // single-entry PHI nodes.  Eliminating the single-entry phi nodes would
511     // require scanning the ValueMap to update any entries that point to the phi
512     // node.
513     BasicBlock *Dest = BI->getSuccessor(0);
514     if (!Dest->getSinglePredecessor() || isa<PHINode>(Dest->begin())) {
515       ++I; continue;
516     }
517     
518     // We know all single-entry PHI nodes in the inlined function have been
519     // removed, so we just need to splice the blocks.
520     BI->eraseFromParent();
521     
522     // Move all the instructions in the succ to the pred.
523     I->getInstList().splice(I->end(), Dest->getInstList());
524     
525     // Make all PHI nodes that referred to Dest now refer to I as their source.
526     Dest->replaceAllUsesWith(I);
527
528     // Remove the dest block.
529     Dest->eraseFromParent();
530     
531     // Do not increment I, iteratively merge all things this block branches to.
532   }
533 }