[PATCH] D60411: Filter out irrelevant llvm-nm option

[serge via Phabricator via llvm-commits]

serge-sans-paille marked 2 inline comments as done.
serge-sans-paille added inline comments.


================
Comment at: tools/llvm-nm/llvm-nm.cpp:62
 cl::list<std::string> InputFilenames(cl::Positional, cl::desc("<input files>"),
-                                     cl::ZeroOrMore);
+                                     cl::ZeroOrMore, cl::cat(NMCat));
 
----------------
jhenderson wrote:
> I've not experimented with this, but do you need to put a category for the positional arguments?
Probably not. This may explain the --s glitter, I'll check that.


================
Comment at: tools/llvm-nm/llvm-nm.cpp:101
 cl::alias Portability("portability", cl::desc("Alias for --format=posix"),
-                      cl::aliasopt(POSIXFormat), cl::NotHidden);
+                      cl::aliasopt(POSIXFormat), cl::NotHidden, cl::cat(NMCat));
 cl::opt<bool> DarwinFormat("m", cl::desc("Alias for --format=darwin"),
----------------
jhenderson wrote:
> Do aliases need a category or do they inherit their base one?
I'll check too.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60411