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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Special-casing calls marked "fast" doesn't really make sense in general.  For example, on ARM v6-M, "fast" is exactly the same as the C calling convention.

That said, adding an inliner special-case to inline internal functions with two or three callers more aggressively might make sense, similar to the existing bonus for inlining internal functions with exactly one caller.  (This might have a similar effect to your patch because calls marked "fast" generally are calling functions with internal linkage.)


https://reviews.llvm.org/D30651