[PATCH] D104363: [llvm] [tools] Hide unrelated options from all tools

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 2 04:48:15 PDT 2021


andreadb added inline comments.


================
Comment at: llvm/tools/llvm-mca/llvm-mca.cpp:71-76
+static cl::OptionCategory MCAOptions("MCA Options");
 static cl::OptionCategory ViewOptions("View Options");
 
 static cl::opt<std::string> InputFilename(cl::Positional,
                                           cl::desc("<input file>"),
+                                          cl::cat(MCAOptions), cl::init("-"));
----------------
Is this change (and the other changes to this file) really needed?

If not, then I'd like those to be removed from this patch.

Thanks
-Andrea


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

https://reviews.llvm.org/D104363



More information about the llvm-commits mailing list