[PATCH] D63902: [InlineCost] separate stats from CallAnalyzer into InliningStats

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 17 10:36:56 PDT 2019


greened added inline comments.


================
Comment at: llvm/lib/Analysis/InlineCost.cpp:2105
+    (*Params.PassInlineCostStats)(CA.getInlineCostStats());
+
   LLVM_DEBUG(CA.dump());
----------------
Is there a particular reason you chose to do this via callback rather than an optional passed result argument?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63902





More information about the llvm-commits mailing list