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

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 23:46:27 PDT 2023


wlei added inline comments.


================
Comment at: llvm/test/Transforms/SampleProfile/profile-mismatch.ll:9
 
-; CHECK: (2/3) of callsites' profile are invalid and (15/25) of samples are discarded due to callsite location mismatch.
+; CHECK: (2/3) of callsites' profile are invalid and (25/35) of samples are discarded due to callsite location mismatch.
 
----------------
This is due to I changed to use the `totalSamples` instead of `headerSamples` for the mismatched inlined samples, since we switched to use the original nested profile, using `totalSamples` should makes more sense.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158817



More information about the llvm-commits mailing list