[PATCH] D116787: [llvm-readobj][MachO] Add option to sort the symbol table before dumping (MachO only, for now).

[Vy Nguyen via Phabricator via llvm-commits]

oontvoo updated this revision to Diff 399423.
oontvoo marked 5 inline comments as done.
oontvoo added a comment.

Addressed reviews comment:

- expect strict white-spaces in test
- add docs
- keep the option in alphabetical order (also fix the text s/outputting/displaying)
- removed unneeded assert and restore indent


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116787

Files:
  llvm/docs/CommandGuide/llvm-readobj.rst
  llvm/test/tools/llvm-readobj/MachO/stabs-sorted.yaml
  llvm/tools/llvm-readobj/MachODumper.cpp
  llvm/tools/llvm-readobj/ObjDumper.h
  llvm/tools/llvm-readobj/Opts.td
  llvm/tools/llvm-readobj/llvm-readobj.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116787.399423.patch
Type: text/x-patch
Size: 11550 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220112/8dd69acd/attachment.bin>