commit c9a392b9dd7791420aa6475f8a2a81bf5731dfa6
author Reid Kleckner <rnk@google.com> Thu Mar 16 22:58:56 2017 +0000
committer Reid Kleckner <rnk@google.com> Thu Mar 16 22:58:56 2017 +0000
tree c7010d058a82ebc1e0e23822ae3859b0415d3910
parent 720883520d0dbe652baac17771c21764b8603879

Remove dead F parameter from Argument constructor

When Function creates its argument list, it does the ilist push_back
itself. No other caller passes in a parent function, so this is dead,
and it uses the soon-to-be-deleted getArgumentList accessor.

llvm-svn: 298009

llvm/lib/IR/Function.cpp

diff --git a/llvm/lib/IR/Function.cpp b/llvm/lib/IR/Function.cpp
index 228152e..6e1b476 100644
--- a/llvm/lib/IR/Function.cpp
+++ b/llvm/lib/IR/Function.cpp
@@ -39,12 +39,8 @@
 
 void Argument::anchor() { }
 
-Argument::Argument(Type *Ty, const Twine &Name, Function *Par, unsigned ArgNo)
-  : Value(Ty, Value::ArgumentVal), ArgNo(ArgNo) {
-  Parent = nullptr;
-
-  if (Par)
-    Par->getArgumentList().push_back(this);
+Argument::Argument(Type *Ty, const Twine &Name, unsigned ArgNo)
+    : Value(Ty, Value::ArgumentVal), Parent(nullptr), ArgNo(ArgNo) {
   setName(Name);
 }
 
@@ -233,7 +229,7 @@
     assert(!FT->getParamType(i)->isVoidTy() &&
            "Cannot have void typed arguments!");
     ArgumentList.push_back(
-        new Argument(FT->getParamType(i), "", nullptr, i));
+        new Argument(FT->getParamType(i), "", i));
   }
 
   // Clear the lazy arguments bit.