[PATCH] D124632: [CSSPGO] Turn on priority inlining for probe-only profile

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 28 16:29:37 PDT 2022


wenlei added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:2035-2038
+      if (!ProfileInlineLimitMin.getNumOccurrences())
+        ProfileInlineLimitMin = std::numeric_limits<unsigned>::max();
+      if (!ProfileInlineLimitMax.getNumOccurrences())
+        ProfileInlineLimitMax = std::numeric_limits<unsigned>::max();
----------------
Do we also need to clear ProfileInlineGrowthLimit here? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124632



More information about the llvm-commits mailing list