[PATCH] D110465: [llvm-profgen] On-demand symbolization

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 26 20:42:49 PDT 2021


wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.h:174
+  // Offset to instruction size map. Also used for quick offset lookup.
+  std::unordered_map<uint64_t, uint64_t> Offset2InstSizeMap;
+
----------------
wenlei wrote:
> We don't seem to use the size (value of the map)?
Oh, it's used in the next diff of preinliner optimization,  tried to make a shortcut :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110465



More information about the llvm-commits mailing list