[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