[PATCH] D103212: [WIP] Changing CG Profile to relocations based.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 27 00:49:36 PDT 2021


jhenderson added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:1777
+  uint32_t DotCGProfileSecIndex = 0;
+  for (unsigned i = 0, Size = Sections.size(); i < Size; ++i) {
+    const Elf_Shdr &Sec = Sections[i];
----------------
i -> I


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:6740
+
+    const uint32_t sizeOfEntry = sizeof(Elf_CGProfile_Impl<ELFT>);
+    for (uint32_t i = 0, size = CGProfileRelaOrError->size(); i < size;
----------------
Fix clang-tidy warnings around here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103212



More information about the llvm-commits mailing list