[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