[llvm-commits] [llvm] r153812 - in /llvm/trunk: include/llvm/Analysis/ include/llvm/Transforms/IPO/ lib/Analysis/ lib/Transforms/IPO/ test/Transforms/Inline/

[Chandler Carruth]

On Wed, Apr 11, 2012 at 9:46 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk>wrote:

>
> On Apr 11, 2012, at 11:33 AM, Eli Friedman <eli.friedman at gmail.com> wrote:
>
> > If your patch consistently leads to lower computed inlining costs,
> > perhaps we should lower the inlining threshold?
>
> The inlining thresholds haven't been calibrated for many years. It is time
> to recalibrate.
>
> Somebody should plot the code size vs speed curve and pick thresholds for
> -Oz, -Os, -O2, and -O3.
>

I actually plan to do this, but haven't had time, and would like to finish
changes to the inliner, or to things that dramatically effect the speed we
see so that the numbers are more stable / meaningful. If others have a
faster way of generating this data than I do (mine would take many days on
a benchmarking rig), by all means.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120412/b45f2861/attachment.html>