[PATCH] D42556: [InlineCost] Mark functions accessing varargs as not viable.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 08:31:31 PST 2018


fhahn added a comment.

In https://reviews.llvm.org/D42556#988451, @efriedma wrote:

> (The original patch didn't make it into 6.0, so we don't need to cherry-pick this, I think.)


Ah yes, the original patch went in just after the branch.

> Do we need to check for musttail calls, since they can also access varargs?

I do not think so: when inlining a function containing a `musttail` call, InlineFunction should forward the varargs appropriately.


https://reviews.llvm.org/D42556





More information about the llvm-commits mailing list