[PATCH] D22261: [InlineCost] Set minsize inline threshold to 0

[Chandler Carruth via llvm-commits]

chandlerc added a comment.

I've always thought this was the ideal threshold to use for minsize, and something we should strive for in order to tune the inliner.

However, last time I checked, this setting actually *increased* code size quite a bit. What benchmarking have you done to evaluate this change? I wonder what other uses of minsize would be useful to get to check that we don't significantly regress code size for them?

(All of these would likely be great test cases for the inline cost analysis of course....)


Repository:
  rL LLVM

http://reviews.llvm.org/D22261