[PATCH] D41335: [InlineFunction] Inline vararg functions that do not access varargs.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 12:39:10 PST 2017


efriedma added reviewers: rnk, hfinkel.
efriedma added a comment.

We should probably clarify LangRef for musttail to make it clear what transforms are legal.  (Does the "thunk" attribute matter?)



================
Comment at: lib/Transforms/Utils/InlineFunction.cpp:1516
+            return false;
+          }))
+        return false;
----------------
Does the comment from https://reviews.llvm.org/D22529#488888, about doing this check during the inline cost scan, still apply here?


https://reviews.llvm.org/D41335





More information about the llvm-commits mailing list