[PATCH] D56910: [llvm-readelf]Revert --dyn-symbols behaviour to make it GNU compatible, and add new --hash-symbols switch for old behaviour

Mon Jan 21 00:06:31 PST 2019

grimar accepted this revision.
grimar added a comment.

Patch looks reasonable to me too.

================
Comment at: test/tools/llvm-readobj/gnu-symbols.test:3
+RUN:  -elf-output-style=GNU | FileCheck %s -check-prefix ELF32
+RUN: llvm-readobj -symbols %p/Inputs/relocs.obj.elf-x86_64 -elf-output-style=GNU \
 RUN:   | FileCheck %s -check-prefix ELF64
----------------
nit: seems you changed `--elf-output-style` to `-elf-output-style`. That made more lines to be shown
as changes for probably no solid reason? I would commit such changes as NFC separately if you want them.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D56910