[PATCH] D56140: [llvm-readobj] [COFF] Print the symbol index for relocations

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 28 14:22:29 PST 2018


ruiu added inline comments.


================
Comment at: include/llvm/Object/COFF.h:438
   const coff_symbol32 *CS32 = nullptr;
+  size_t Index = 0;
 };
----------------
Instead of storing an index as a member, can you compute it when you need it from the offset from the beginning of a symbol table?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56140





More information about the llvm-commits mailing list