[PATCH] D78730: [llvm][NFC] Add an explicit 'ComputeFullInlineCost' API

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

Why not passing the 'computeFullCost' as an additional parameter to InlineCostAnalyzer?



================
Comment at: llvm/lib/Analysis/InlineCost.cpp:716
+/// Calculate inline cost irrespective of thresholds.
+class CompleteInlineCostAnalyzer final : public InlineCostCallAnalyzer {
+public:
----------------
should it define 'shouldStop()' to return false?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78730/new/

https://reviews.llvm.org/D78730