[PATCH] D38190: Partial Inlining with multi-region outlining based on PGO information

Graham Yiu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 12:44:57 PDT 2017


gyiu updated this revision to Diff 118670.
gyiu added a comment.

- Updated code based on review comments.
- Also added check for candidate region outputs.  If cold region has an output (ie. a live exit variable) then we could be blocking some code motion in the caller after it has been outlined.
- Also added code to mark outlined functions/callsites with coldCC.


Repository:
  rL LLVM

https://reviews.llvm.org/D38190

Files:
  include/llvm/Analysis/ProfileSummaryInfo.h
  lib/Transforms/IPO/PartialInlining.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38190.118670.patch
Type: text/x-patch
Size: 33160 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171011/96191ca9/attachment.bin>


More information about the llvm-commits mailing list