[PATCH] D115013: [llvm-profgen] Fix total samples related issues

[Lei Wang via Phabricator via llvm-commits]

wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:500
+        FunctionSamples &FunctionProfile = getLeafProfileAndAddTotalSamples(
+            FrameVec, Count * Binary->getInstSize(Offset));
         updateBodySamplesforFunctionProfile(FunctionProfile, FrameVec.back(),
----------------
hoy wrote:
> This doesn't really makes sense. Wondering how much it matters, compared to other places that affect total samples.
This place caused 0.1% regression(I confirmed it's not noise) on unicorn, no regression on ads. With others together, they caused ~0.5% regression on unicorn.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115013