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

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 13 18:23:16 PDT 2021


wenlei added a comment.

one more place needs a name change, otherwise lgtm, thanks.



================
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);
----------------
Change `BASE_ADDRESS` to `ADDRESS` or `MMAPPED_ADDRESS` too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103178



More information about the llvm-commits mailing list