[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