diff --git a/lib/Transforms/IPO/LowerSetJmp.cpp b/lib/Transforms/IPO/LowerSetJmp.cpp index 20385f8ba6c..80158b43a5c 100644 --- a/lib/Transforms/IPO/LowerSetJmp.cpp +++ b/lib/Transforms/IPO/LowerSetJmp.cpp @@ -440,7 +440,7 @@ void LowerSetJmp::TransformSetJmpCall(CallInst* Inst) // Replace all uses of this instruction with the PHI node created by // the eradication of setjmp. Inst->replaceAllUsesWith(PHI); - Inst->getParent()->getInstList().erase(Inst); + Inst->eraseFromParent(); ++SetJmpsTransformed; } @@ -478,10 +478,10 @@ void LowerSetJmp::visitCallInst(CallInst& CI) // Replace the old call inst with the invoke inst and remove the call. CI.replaceAllUsesWith(II); - CI.getParent()->getInstList().erase(&CI); + CI.eraseFromParent(); // The old terminator is useless now that we have the invoke inst. - Term->getParent()->getInstList().erase(Term); + Term->eraseFromParent(); ++CallsTransformed; } diff --git a/lib/Transforms/IPO/RaiseAllocations.cpp b/lib/Transforms/IPO/RaiseAllocations.cpp index 58fd682eb7a..daf8ef00053 100644 --- a/lib/Transforms/IPO/RaiseAllocations.cpp +++ b/lib/Transforms/IPO/RaiseAllocations.cpp @@ -177,7 +177,7 @@ bool RaiseAllocations::runOnModule(Module &M) { BranchInst::Create(II->getNormalDest(), I); // Delete the old call site - MI->getParent()->getInstList().erase(I); + I->eraseFromParent(); Changed = true; ++NumRaised; } diff --git a/lib/Transforms/Scalar/ConstantProp.cpp b/lib/Transforms/Scalar/ConstantProp.cpp index 90e8dad4f8a..dbbc86142cf 100644 --- a/lib/Transforms/Scalar/ConstantProp.cpp +++ b/lib/Transforms/Scalar/ConstantProp.cpp @@ -79,7 +79,7 @@ bool ConstantPropagation::runOnFunction(Function &F) { // Remove the dead instruction. WorkList.erase(I); - I->getParent()->getInstList().erase(I); + I->eraseFromParent(); // We made a change to the function... Changed = true; diff --git a/lib/Transforms/Scalar/GCSE.cpp b/lib/Transforms/Scalar/GCSE.cpp index 94cdcb1400f..9d3f1a87506 100644 --- a/lib/Transforms/Scalar/GCSE.cpp +++ b/lib/Transforms/Scalar/GCSE.cpp @@ -201,5 +201,5 @@ void GCSE::ReplaceInstructionWith(Instruction *I, Value *V) { } // Erase the instruction from the program. - I->getParent()->getInstList().erase(I); + I->eraseFromParent(); } diff --git a/lib/Transforms/Utils/CodeExtractor.cpp b/lib/Transforms/Utils/CodeExtractor.cpp index cb7a5293b7f..1c937fbd0d0 100644 --- a/lib/Transforms/Utils/CodeExtractor.cpp +++ b/lib/Transforms/Utils/CodeExtractor.cpp @@ -550,18 +550,18 @@ emitCallAndSwitchStatement(Function *newFunction, BasicBlock *codeReplacer, ReturnInst::Create(Constant::getNullValue(OldFnRetTy), TheSwitch); } - TheSwitch->getParent()->getInstList().erase(TheSwitch); + TheSwitch->eraseFromParent(); break; case 1: // Only a single destination, change the switch into an unconditional // branch. BranchInst::Create(TheSwitch->getSuccessor(1), TheSwitch); - TheSwitch->getParent()->getInstList().erase(TheSwitch); + TheSwitch->eraseFromParent(); break; case 2: BranchInst::Create(TheSwitch->getSuccessor(1), TheSwitch->getSuccessor(2), call, TheSwitch); - TheSwitch->getParent()->getInstList().erase(TheSwitch); + TheSwitch->eraseFromParent(); break; default: // Otherwise, make the default destination of the switch instruction be one diff --git a/lib/Transforms/Utils/InlineFunction.cpp b/lib/Transforms/Utils/InlineFunction.cpp index a6ef1b78dc6..22dab698465 100644 --- a/lib/Transforms/Utils/InlineFunction.cpp +++ b/lib/Transforms/Utils/InlineFunction.cpp @@ -119,7 +119,7 @@ static void HandleInlinedInvoke(InvokeInst *II, BasicBlock *FirstNewBlock, BranchInst::Create(InvokeDest, UI); // Delete the unwind instruction! - UI->getParent()->getInstList().pop_back(); + UI->eraseFromParent(); // Update any PHI nodes in the exceptional block to indicate that // there is now a new entry in them. @@ -455,10 +455,10 @@ bool llvm::InlineFunction(CallSite CS, CallGraph *CG, const TargetData *TD) { TheCall->replaceAllUsesWith(R->getReturnValue()); } // Since we are now done with the Call/Invoke, we can delete it. - TheCall->getParent()->getInstList().erase(TheCall); + TheCall->eraseFromParent(); // Since we are now done with the return instruction, delete it also. - Returns[0]->getParent()->getInstList().erase(Returns[0]); + Returns[0]->eraseFromParent(); // We are now done with the inlining. return true; diff --git a/lib/Transforms/Utils/Local.cpp b/lib/Transforms/Utils/Local.cpp index 0f7d02c8886..ed67c5eaaae 100644 --- a/lib/Transforms/Utils/Local.cpp +++ b/lib/Transforms/Utils/Local.cpp @@ -159,7 +159,7 @@ bool llvm::ConstantFoldTerminator(BasicBlock *BB) { BranchInst::Create(SI->getSuccessor(1), SI->getSuccessor(0), Cond, SI); // Delete the old switch... - SI->getParent()->getInstList().erase(SI); + SI->eraseFromParent(); return true; } }