[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