[PATCH] D31052: Remove getArgumentList() in favor of arg_begin(), args(), etc

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 15:44:42 PDT 2017


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM once split.



================
Comment at: include/llvm/IR/Argument.h:40-41
 public:
-  /// If \p F is specified, the argument is inserted at the end of the argument
-  /// list for \p F.
-  explicit Argument(Type *Ty, const Twine &Name = "", Function *F = nullptr);
+  /// Argument constructor.
+  explicit Argument(Type *Ty, const Twine &Name = "");
 
----------------
This change seems unrelated, land separately? (Likely simple enough to be obvious.)


https://reviews.llvm.org/D31052





More information about the llvm-commits mailing list