[PATCH] D109764: [NFC] [PSI] explain encoding of PercentileCutoff.

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 07:40:54 PDT 2021


mtrofin accepted this revision.
mtrofin added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/Analysis/ProfileSummaryInfo.h:138
+  /// PercentileCutoff is encoded as a 6 digit decimal fixed point number, where
+  /// the first two digits are the whole part. E.g. 995000 for 99.5 percentile.
   bool isHotCountNthPercentile(int PercentileCutoff, uint64_t C) const;
----------------
I don't think I'd understand what 'whole part' means, but the example is perfectly sufficient. I'd just leave it to the example.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109764



More information about the llvm-commits mailing list