[PATCH] D23339: Don't import variadic functions

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 10 14:38:24 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D23339#511890, @eraman wrote:

> In https://reviews.llvm.org/D23339#511869, @mehdi_amini wrote:
>
> > LGTM, thanks.
> >  We probably need to review other properties of function that prevents them from being inlined.
>
>
> This check should be eventually moved to InlineCost so that importer and inliner are out of sync. For now, a FIXME should be useful.


Very good point indeed.


https://reviews.llvm.org/D23339





More information about the llvm-commits mailing list