diff options
| author | Bob Wilson <bob.wilson@apple.com> | 2012-10-07 01:11:19 +0000 |
|---|---|---|
| committer | Bob Wilson <bob.wilson@apple.com> | 2012-10-07 01:11:19 +0000 |
| commit | c38b636e6ee76cc55c0e99a12cc959a2b3cda789 (patch) | |
| tree | 2245c9d1e64ccb8c2c66d18ca2ffb406b4c3529b /docs/CompilerWriterInfo.rst | |
| parent | af63f0b6dcda0b29dddc6001451048635531f683 (diff) | |
| download | external_llvm-c38b636e6ee76cc55c0e99a12cc959a2b3cda789.tar.gz external_llvm-c38b636e6ee76cc55c0e99a12cc959a2b3cda789.tar.bz2 external_llvm-c38b636e6ee76cc55c0e99a12cc959a2b3cda789.zip | |
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
Diffstat (limited to 'docs/CompilerWriterInfo.rst')
0 files changed, 0 insertions, 0 deletions