[PATCH] D78730: [llvm][NFC] Add an explicit 'ComputeFullInlineCost' API
Mircea Trofin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 24 12:26:45 PDT 2020
mtrofin added a comment.
In D78730#2002357 <https://reviews.llvm.org/D78730#2002357>, @davidxl wrote:
> Why not passing the 'computeFullCost' as an additional parameter to InlineCostAnalyzer?
Because InlineCostAnalyzer's finalizeAnalysis looks at threshold (see lines 633-635).
Clarified documentation of the new API.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78730/new/
https://reviews.llvm.org/D78730
More information about the llvm-commits
mailing list