[PATCH] D62070: Update llvm-nm -s to use a multi-var option

Thu Apr 23 15:47:45 PDT 2020

MaskRay added a comment.

It seems that we have some parsing difficulty here:/ I tried to use `llvm-nm -s` today for `--print-armap` (GNU/FreeBSD nm) and I noticed that `-s` is reserved for Mach-O. https://www.unix.com/man-page/osx/1/nm/ says

> -s segname sectname
>  List only those symbols in the section (segname,sectname).

This also appears to be the last use case of `cl::multi_val`:/ Does `-s` have a longer option for the `SegSect` functionality? After all, it has been broken so many years... (the only order which was supported was `llvm-nm /bin/ls -s __TEXT __text` before this patch).

Repository:
  rL LLVM

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

https://reviews.llvm.org/D62070