[PATCH] D22529: Inlining of empty/small variadic functions.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 13:59:30 PDT 2016


chandlerc requested changes to this revision.
chandlerc added a comment.
This revision now requires changes to proceed.

I think we should do this during the inline cost scan, which has a natural thresholding effect. That will also allow us to inline variadic functions where there is a use of 'va_start', but only behind a condition that is known to be false at the callsite we're inlining through.

There are a few other places where we bail out of the cost analysis if we see a construct that simply can't be handled, and it feels like this would fit well there.


https://reviews.llvm.org/D22529





More information about the llvm-commits mailing list