[PATCH] D32783: [PartialInlining] Add frequency based cost analysis

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 17:16:39 PDT 2017


davidxl updated this revision to Diff 98043.
davidxl added a comment.

Address feedback -- revert change in CodeExtractor.h and .cc file

Also let the cost analysis only trust PGO and user annotation data. Without PGO, add an option to control the aggressiveness.


https://reviews.llvm.org/D32783

Files:
  lib/Transforms/IPO/PartialInlining.cpp
  test/Transforms/CodeExtractor/ExtractedFnEntryCount.ll
  test/Transforms/CodeExtractor/MultipleExitBranchProb.ll
  test/Transforms/CodeExtractor/PartialInlineEntryUpdate.ll
  test/Transforms/CodeExtractor/PartialInlineHighCost.ll
  test/Transforms/CodeExtractor/PartialInlineOrAnd.ll
  test/Transforms/CodeExtractor/SingleCondition.ll
  test/Transforms/CodeExtractor/X86/InheritTargetAttributes.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32783.98043.patch
Type: text/x-patch
Size: 30610 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170506/89acdec8/attachment.bin>


More information about the llvm-commits mailing list