[PATCH] D17082: FunctionImport: add a progressive heuristic to limit importing too deep in the callgraph

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 10:24:39 PST 2016


joker.eph created this revision.
joker.eph added a reviewer: tejohnson.
joker.eph added a subscriber: llvm-commits.

The current function importer will walk the callgraph, importing
transitively any callee that is below the threshold. This can
lead to import very deep which is costly in compile time and not
necessarily beneficial as most of the inline would happen in
imported function and not necessarilly in user code.

The actual factor has been carefully chosen by flipping a coin ;)
Some tuning need to be done (just at the existing limiting threshold).

http://reviews.llvm.org/D17082

Files:
  lib/Transforms/IPO/FunctionImport.cpp
  test/Transforms/FunctionImport/adjustable_threshold.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17082.47482.patch
Type: text/x-patch
Size: 6352 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160210/a025103e/attachment.bin>


More information about the llvm-commits mailing list