Make sure always-inline functions get inlined. <rdar://problem/12423986>
authorBob Wilson <bob.wilson@apple.com>
Sun, 7 Oct 2012 01:11:19 +0000 (01:11 +0000)
committerBob Wilson <bob.wilson@apple.com>
Sun, 7 Oct 2012 01:11:19 +0000 (01:11 +0000)
commitc38b636e6ee76cc55c0e99a12cc959a2b3cda789
tree2245c9d1e64ccb8c2c66d18ca2ffb406b4c3529b
parentaf63f0b6dcda0b29dddc6001451048635531f683
Make sure always-inline functions get inlined. <rdar://problem/12423986>

Without this change, when the estimated cost for inlining a function with
an "alwaysinline" attribute was lower than the inlining threshold, the
getInlineCost function was returning that estimated cost rather than the
special InlineCost::AlwaysInlineCost value. That is fine in the normal
inlining case, but it can fail when the inliner considers the opportunity
cost of inlining into an internal or linkonce-odr function. It may decide
not to inline the always-inline function in that case. The fix here is just
to make getInlineCost always return the special value for always-inline
functions. I ran into this building clang with libc++. Tablegen failed to
link because of an always-inline function that was not inlined. I have been
unable to reduce the testcase down to a reasonable size.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@165367 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Analysis/InlineCost.cpp