[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