[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 llvm-commits at lists.llvm.org
Fri Jan 7 12:27:52 PST 2022


oontvoo added a comment.

In D116787#3228302 <https://reviews.llvm.org/D116787#3228302>, @MaskRay wrote:

> local/external/undefined may be better as it is more likely the order matches ld64.lld order.

How strongly do you feel about this? I find it simpler to just sort by the numerical value of the `n_type` field


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116787



More information about the llvm-commits mailing list