[PATCH] D109713: [AutoFDO][llvm-profgen] Report zero count for unexecuted part of function code

[Wenlei He via Phabricator via llvm-commits]

wenlei accepted this revision.
wenlei added a comment.

lgtm, thanks.

> For the implementation, it leverages the range counter, initializes all the executed function range with the zero-value.

This should be quite efficient.



================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:153
+
+    BoundaryPoint() = default;
+
----------------
curious why we need this? 


================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:226
   uint64_t BeginAddress = UINT64_MAX;
+  int ZeroRangeDepth = 0;
   int Count = 0;
----------------
Since we only add zero ranges for functions, there won't be any overlapping zero range and the Depth is going to be either 0 or 1, right? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109713