Use eraseFromParent() instead of doing that manually in two places.
authorMatthijs Kooijman <matthijs@stdin.nl>
Fri, 30 May 2008 12:35:46 +0000 (12:35 +0000)
committerMatthijs Kooijman <matthijs@stdin.nl>
Fri, 30 May 2008 12:35:46 +0000 (12:35 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51770 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/IPO/DeadArgumentElimination.cpp

index 2dab6952247d35e7bc7281ba35f56fded7e07fab..1ca5a3982ae14dbaf088ac52d34a9fd794cf072a 100644 (file)
@@ -626,7 +626,7 @@ void DAE::RemoveDeadArgumentsFromFunction(Function *F) {
 
     // Finally, remove the old call from the program, reducing the use-count of
     // F.
-    Call->getParent()->getInstList().erase(Call);
+    Call->eraseFromParent();
   }
 
   // Since we have now created the new function, splice the body of the old
@@ -665,7 +665,7 @@ void DAE::RemoveDeadArgumentsFromFunction(Function *F) {
       }
 
   // Now that the old function is dead, delete it.
-  F->getParent()->getFunctionList().erase(F);
+  F->eraseFromParent();
 }
 
 bool DAE::runOnModule(Module &M) {