[PATCH] D15245: Use the inlinehint-threshold for hot callees.

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 16:55:35 PST 2015


eraman updated this revision to Diff 43282.
eraman added a comment.

I am now checking for cold calls and using the cold threshold as well and added a test case.

Earlier in this thread I had commented that Cold attribute is used in other places as well, but I was wrong. Branch probability computation checks for Cold attribute in instruction and not the function. With this change, both setting of cold and inlinehint attribute can be removed from CodeGenPGO.cpp in CFE. Does this look good?


Repository:
  rL LLVM

http://reviews.llvm.org/D15245

Files:
  lib/Transforms/IPO/Inliner.cpp
  test/Transforms/Inline/inline-cold-callee.ll
  test/Transforms/Inline/inline-hot-callee.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15245.43282.patch
Type: text/x-patch
Size: 5017 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151219/768d9415/attachment.bin>


More information about the llvm-commits mailing list