[PATCH] D54124: [llvm-readelf] Make llvm-readelf more compatible with GNU readelf.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 5 16:48:00 PST 2018


MaskRay added a comment.

Add two subscribers:

@compnerd who added the `-a` alias to `ARMAttributesShort` in https://reviews.llvm.org/rL200450
@echristo who added `--relocations` in https://reviews.llvm.org/rL178679 and could share some background of the two tools llvm-readobj llvm-readelf and the naming of these options...

I guess there was once only `llvm-readobj` and it had better option names (but incompatible with GNU readelf, e.g. `--dynamic-table` instead of GNU `--dynamic`) and later llvm-readelf was born. Now for compatibility reasons we have to add these GNU aliases and the option parsing looks very messy now.


Repository:
  rL LLVM

https://reviews.llvm.org/D54124





More information about the llvm-commits mailing list