[PATCH] D158817: [CSSPGO] Refactoring populateIRLocations and countProfileMismatches

[Wenlei He via Phabricator via llvm-commits]

wenlei added a comment.

looks cleaner and more structured now, thanks for update!



================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:2383-2384
+      !ProbeManager->profileIsValid(F, FS)) {
     LLVM_DEBUG(dbgs() << "Run stale profile matching for " << F.getName()
                       << "\n");
     // The matching result will be saved to IRToProfileLocationMap, create a new
----------------
nit: move this into `runStaleProfileMatching` as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158817