[llvm] 5efffac - [llvm-symbolizer] Move setGroupedShortOptions and don't ignore case

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 1 19:43:59 PDT 2021


Author: Fangrui Song
Date: 2021-07-01T19:43:49-07:00
New Revision: 5efffac71a1c640b0cba8e34f5e2374d397f6eb3

URL: https://github.com/llvm/llvm-project/commit/5efffac71a1c640b0cba8e34f5e2374d397f6eb3
DIFF: https://github.com/llvm/llvm-project/commit/5efffac71a1c640b0cba8e34f5e2374d397f6eb3.diff

LOG: [llvm-symbolizer] Move setGroupedShortOptions and don't ignore case

setGroupedShortOptions in the ctor seems more popular.

Added: 
    

Modified: 
    llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp b/llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp
index 2359e5d2b4587..227ce12a6d9af 100644
--- a/llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp
+++ b/llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp
@@ -66,7 +66,9 @@ static const opt::OptTable::Info InfoTable[] = {
 
 class SymbolizerOptTable : public opt::OptTable {
 public:
-  SymbolizerOptTable() : OptTable(InfoTable, true) {}
+  SymbolizerOptTable() : OptTable(InfoTable) {
+    setGroupedShortOptions(true);
+  }
 };
 } // namespace
 
@@ -206,7 +208,6 @@ static opt::InputArgList parseOptions(int Argc, char *Argv[], bool IsAddr2Line,
                                       StringSaver &Saver,
                                       SymbolizerOptTable &Tbl) {
   StringRef ToolName = IsAddr2Line ? "llvm-addr2line" : "llvm-symbolizer";
-  Tbl.setGroupedShortOptions(true);
   // The environment variable specifies initial options which can be overridden
   // by commnad line options.
   Tbl.setInitialOptionsFromEnvironment(IsAddr2Line ? "LLVM_ADDR2LINE_OPTS"


        


More information about the llvm-commits mailing list