[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