[PATCH] D156722: [CSSPGO] Support stale profile matching for LTO

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 10:59:53 PDT 2023


wenlei added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:2117
 
+void SampleProfileMatcher::populateIRLocations(
+    const Function &F, const FunctionSamples &FS,
----------------
wenlei wrote:
> nit on function naming: it not only populates `IRLocations`, but also `MatchedCallsiteLocs`.
More like `findAnchorLocations`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156722



More information about the llvm-commits mailing list