[PATCH] D108002: [llvm-profgen] Clean up code dealing with multiple binaries

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 13 11:56:06 PDT 2021


wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PerfReader.h:535
 // Address to binary map for fast look-up
 using AddressBinaryMap = std::map<uint64_t, ProfiledBinary *>;
 
----------------
hoy wrote:
> Do we still need this? Looks like this can be just a single field of `PerfReaderBase`.
Good catch! We shouldn't need this, removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108002



More information about the llvm-commits mailing list