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

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

Sorry



================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:668
   Fields[PID].getAsInteger(10, Event.PID);
-  Fields[BASE_ADDRESS].getAsInteger(0, Event.BaseAddress);
+  Fields[BASE_ADDRESS].getAsInteger(0, Event.Address);
   Fields[MMAPPED_SIZE].getAsInteger(0, Event.Size);
----------------
wenlei wrote:
> Change `BASE_ADDRESS` to `ADDRESS` or `MMAPPED_ADDRESS` too?
Sorry, landed this too fast. Meant to land the other patch. Good catch, will make a follow-up commit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103178