[PATCH] D23339: Don't import variadic functions

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 17:23:40 PDT 2016


eraman added a comment.

In https://reviews.llvm.org/D23339#510677, @mehdi_amini wrote:

> What is the benefit of knowing that a function is variadic compared to just knowing that is can't be inlined?
>  We should just have a bit that is "can't be inlined"


InlineCost.cpp has an isInlineViable function. It doesn't check for vararg now, so that should be first fixed and then can be used to set a bit in the summary.


https://reviews.llvm.org/D23339





More information about the llvm-commits mailing list