[PATCH] D112672: [llvm-profgen] Infer total samples by accumulating all its body samples

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 27 17:53:49 PDT 2021


wenlei accepted this revision.
wenlei added a comment.
This revision is now accepted and ready to land.

lgtm, except a nit. thanks.



================
Comment at: llvm/include/llvm/ProfileData/SampleProf.h:735
+  // Accumulate all body samples to set total samples.
+  void inferTotalSamples() {
+    setTotalSamples(0);
----------------
nit: inferTotalSamples->updateTotalSamples, same for ProfileGenerator.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112672



More information about the llvm-commits mailing list