[compiler-rt] [llvm] [MemProf] Change histogram storage from uint64_t to uint16_t (PR #147854)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 26 07:50:58 PDT 2025


================
@@ -166,6 +166,39 @@ readMemInfoBlocksV4(const char *Ptr) {
   return Items;
 }
 
+llvm::SmallVector<std::pair<uint64_t, MemInfoBlock>>
----------------
teresajohnson wrote:

It seems this is mostly identical to the v4 reader above, with the exception of a few comments that differ (unnecessarily? makes it more difficult to see the functionality diffs), and the histogram reading loop body. Maybe refactor to make a common v4/v5 reader with just the inner part of the histogram reading loop doing different things based on version?

https://github.com/llvm/llvm-project/pull/147854


More information about the llvm-commits mailing list