[PATCH] D135887: [XOCFF] llvvm-readobj support display symbol table of loader section of xcoff object file.

[Digger Lin via Phabricator via llvm-commits]

DiggerLin added a comment.

In D135887#3898552 <https://reviews.llvm.org/D135887#3898552>, @jhenderson wrote:

> Thanks for the explanation, I follow what you're saying. One follow-up question: what is the motivation for having the outer "Loader Section" grouping to wrap the header/relocations/symbols? I'm not aware of any prior art for this style, at least within the llvm-readobj tooling (for example, in ELF, dynamic symbols, the dynamic section tags, and the dynamic relocations are all separate and ungrouped).
>
> (For clarity, I'm not opposed to a combined option that dumps them all, but am just wondering why the grouping is important).



  I think it is more clear that which data are belong to loader-section, and it will also display the header, symbol table , relocation ,etc in sequence of loader section. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135887