[llvm] 7664508 - [llvm/OptTable] Add named param comment for GroupedShortOption

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 08:33:41 PDT 2021


Author: Nico Weber
Date: 2021-09-27T11:33:29-04:00
New Revision: 76645089100f344cbe77538ae83dd58d0caf18bc

URL: https://github.com/llvm/llvm-project/commit/76645089100f344cbe77538ae83dd58d0caf18bc
DIFF: https://github.com/llvm/llvm-project/commit/76645089100f344cbe77538ae83dd58d0caf18bc.diff

LOG: [llvm/OptTable] Add named param comment for GroupedShortOption

Added: 
    

Modified: 
    llvm/lib/Option/OptTable.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Option/OptTable.cpp b/llvm/lib/Option/OptTable.cpp
index deaea9b60f29..37c2fcbab181 100644
--- a/llvm/lib/Option/OptTable.cpp
+++ b/llvm/lib/Option/OptTable.cpp
@@ -360,8 +360,9 @@ std::unique_ptr<Arg> OptTable::parseOneArgGrouped(InputArgList &Args,
       continue;
 
     Option Opt(Start, this);
-    if (std::unique_ptr<Arg> A = Opt.accept(
-            Args, StringRef(Args.getArgString(Index), ArgSize), false, Index))
+    if (std::unique_ptr<Arg> A =
+            Opt.accept(Args, StringRef(Args.getArgString(Index), ArgSize),
+                       /*GroupedShortOption=*/false, Index))
       return A;
 
     // If Opt is a Flag of length 2 (e.g. "-a"), we know it is a prefix of
@@ -381,8 +382,8 @@ std::unique_ptr<Arg> OptTable::parseOneArgGrouped(InputArgList &Args,
       return std::make_unique<Arg>(getOption(UnknownOptionID), Str, Index++,
                                    CStr);
 
-    if (std::unique_ptr<Arg> A =
-            Opt.accept(Args, Str.substr(0, 2), true, Index)) {
+    if (std::unique_ptr<Arg> A = Opt.accept(
+            Args, Str.substr(0, 2), /*GroupedShortOption=*/true, Index)) {
       Args.replaceArgString(Index, Twine('-') + Str.substr(2));
       return A;
     }
@@ -442,8 +443,9 @@ std::unique_ptr<Arg> OptTable::ParseOneArg(const ArgList &Args, unsigned &Index,
       continue;
 
     // See if this option matches.
-    if (std::unique_ptr<Arg> A = Opt.accept(
-            Args, StringRef(Args.getArgString(Index), ArgSize), false, Index))
+    if (std::unique_ptr<Arg> A =
+            Opt.accept(Args, StringRef(Args.getArgString(Index), ArgSize),
+                       /*GroupedShortOption=*/false, Index))
       return A;
 
     // Otherwise, see if this argument was missing values.


        


More information about the llvm-commits mailing list