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

[Wenlei He via Phabricator via llvm-commits]

wenlei accepted this revision.
wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:297
 
-ProfiledBinary &PerfReaderBase::loadBinary(const StringRef BinaryPath,
-                                           bool AllowNameConflict) {
-  // The binary table is currently indexed by the binary name not the full
-  // binary path. This is because the user-given path may not match the one
-  // that was actually executed.
-  StringRef BinaryName = llvm::sys::path::filename(BinaryPath);
-
+void PerfReaderBase::loadBinary(StringRef BinaryPath) {
   // Call to load the binary in the ctor of ProfiledBinary.
----------------
is the removal of `const` intentional?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108002