[PATCH] D73994: [InlineCost] Relax bonus restrictions on uninlinable functions

[George Burgess IV via Phabricator via llvm-commits]

george.burgess.iv updated this revision to Diff 243060.
george.burgess.iv added a comment.

Updated to plumb through the "are we doing ThinLTO || prelink LTO" bit in a way that makes sense to me. Suggestions for potentially better approaches is welcome :)

> Among the 3 bonuses disabled in DisallowAllBonuses, only the LastCallToStaticBonus is useful to allow for code size reduction [...].

Fixed; thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73994/new/

https://reviews.llvm.org/D73994

Files:
  clang/lib/CodeGen/BackendUtil.cpp
  llvm/include/llvm/Analysis/InlineCost.h
  llvm/include/llvm/Passes/PassBuilder.h
  llvm/include/llvm/Transforms/IPO.h
  llvm/include/llvm/Transforms/IPO/Inliner.h
  llvm/lib/Analysis/InlineCost.cpp
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Transforms/IPO/InlineSimple.cpp
  llvm/lib/Transforms/IPO/Inliner.cpp
  llvm/lib/Transforms/IPO/PassManagerBuilder.cpp
  llvm/test/Transforms/Inline/bpi-cold-inlining.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73994.243060.patch
Type: text/x-patch
Size: 29599 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200207/91e5aa01/attachment.bin>