[PATCH] D103178: [CSSPGO][llvm-profgen] Allow multiple executable load segments.

[Lei Wang via Phabricator via llvm-commits]

wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.cpp:149
+        // Segments will always be loaded at a page boundary.
+        PreferredBaseAddresses.push_back(Phdr.p_vaddr & ~(Phdr.p_align - 1U));
+        ExeSegmentOffsets.push_back(Phdr.p_offset & ~(Phdr.p_align - 1U));
----------------
Could you teach me on the difference of `Phdr.p_vaddr` and `Phdr.p_offset`? why the MMP event's offset is supposed to be equal to the first `Phdr.p_offset`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103178