[PATCH] D18622: Replace the use of MaxFunctionCount module flag

[Easwaran Raman via llvm-commits]

eraman added a comment.

In http://reviews.llvm.org/D18622#400020, @davidxl wrote:

> What is the impact to overall compile time (using a large app)? On the other hand, if caching does not increase code complexity too much, it is also fine.  Do you have a updated version of the patch?


I was looking at compilation time of a fairly large file with a higher threshold (1500) for hot callsites. In this case, the total time spent on inlining (not just inline cost analysis) was around 5.5% of the total compilation time (as reported by -time-passes). If this is representative, we are looking at increasing the compilation time by ~1.4% (25% of 5.5%). Not much, but this is a low hanging fruit and the code with ManagedStatic is not much complex.


http://reviews.llvm.org/D18622