[memprof][NFC] Update a stale comment about symbolization.
authorSnehasish Kumar <snehasishk@google.com>
Thu, 27 Apr 2023 18:53:37 +0000 (18:53 +0000)
committerSnehasish Kumar <snehasishk@google.com>
Thu, 27 Apr 2023 18:59:33 +0000 (18:59 +0000)
Symbolization of position independent code was added in D146181. Update
the comment to note that the checks below are to sanity check the
assumptions we make to simplify symbolization.

Differential Revision: https://reviews.llvm.org/D149370

llvm/lib/ProfileData/RawMemProfReader.cpp

index 7c519ed..dbfb472 100644 (file)
@@ -282,8 +282,8 @@ Error RawMemProfReader::initialize(std::unique_ptr<MemoryBuffer> DataBuffer) {
   }
 
   // Check whether the profiled binary was built with position independent code
-  // (PIC). For now we provide a error message until symbolization support
-  // is added for pic.
+  // (PIC). Perform sanity checks for assumptions we rely on to simplify
+  // symbolization.
   auto* Elf64LEObject = llvm::cast<llvm::object::ELF64LEObjectFile>(ElfObject);
   const llvm::object::ELF64LEFile& ElfFile = Elf64LEObject->getELFFile();
   auto PHdrsOr = ElfFile.program_headers();