[PATCH] D30651: [InlineCost, -Oz] Don't take into account the penalty of a fast call of frequently used functions

Evgeny Astigeevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 10 03:57:21 PST 2017


eastig added a comment.

In https://reviews.llvm.org/D30651#697010, @efriedma wrote:

> OptMinSizeThreshold is basically set experimentally; http://reviews.llvm.org/rL288024 is the last time it was adjusted.
>
> I don't think anyone's looked at CallPenalty in a long time; it's possible it's too large at -Oz.


Hi Eli,
What do think about an idea to add a knob for CallPenalty? Playing with it and the threshold knob in the driver can do what I want to do with the patch.


https://reviews.llvm.org/D30651





More information about the llvm-commits mailing list