[llvm-commits] [llvm] r94615 - in /llvm/trunk: include/llvm/Analysis/InlineCost.h lib/Analysis/InlineCost.cpp

[Eric Christopher]

On Jan 26, 2010, at 3:21 PM, Jakob Stoklund Olesen wrote:

> Own argument bonus:       20 -> 5
> Outgoing argument penalty: 0 -> 5
> Alloca bonus:             10 -> 5
> Constant instr bonus:      7 -> 5
> Dead successor bonus:     40 -> 5*(avg instrs/block)

Aiee!

So, what'd you test and how'd you figure it out? Platforms?

> The new cost metrics are generaly 25 points higher than before, so we may need
> to move thresholds.

If this affects Os inlining then almost assuredly, we're barely inlining anything there - even in cases where it does save space.

-eric