[PATCH] D105330: [llvm-nm] Switch command line parsing from llvm::cl to OptTable

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 14 10:10:24 PDT 2021


MaskRay marked an inline comment as done.
MaskRay added a comment.

In D105330#2876946 <https://reviews.llvm.org/D105330#2876946>, @DavidSpickett wrote:

> @MaskRay You say in the commit msg:
>
>   One-dash long options are still supported.
>
> Does that mean that `-format=sysv` should still be supported? I'm not sure if that's a "long option" or what.
>
> It's used in the llvm-testsuite: https://github.com/llvm/llvm-test-suite/blob/main/litsupport/modules/codesize.py

This is llvm-nm.

The llvm-test-suite you fixed is llvm-size. llvm-size is simple so I just removed one-dash long options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105330



More information about the llvm-commits mailing list