[PATCH] D135929: [profile] Add binary ids into indexed profiles

Mingming Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 27 16:42:20 PDT 2023


mingmingl added a comment.
Herald added subscribers: hoy, wlei.

Hi, I noticed that the binary id section is padded so its section size is a power of 2. So the actual bytes written could be ~2x of binary id content size (e.g, 17 is padded to 32). May I know what's the motivation? If the section is not padded to a power of 2, instr prof reader could be changed accordingly to read binary id correctly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135929



More information about the llvm-commits mailing list