[PATCH] D83639: [OptTable] Support grouped short options

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/include/llvm/Option/Option.h:224-225
 private:
-  Arg *acceptInternal(const ArgList &Args, unsigned &Index,
-                      unsigned ArgSize) const;
+  Arg *acceptInternal(const ArgList &Args, StringRef CurArg,
+                      unsigned &Index) const;
 
----------------
It might be nice to keep the `Index` parameter in the same place as it was before (i.e. move `CurArg` after it). I don't feel strongly about this though if you prefer your current approach.


================
Comment at: llvm/lib/Option/OptTable.cpp:348
+
+  // Search for the option which can match Str.
+  for (; Start != End; ++Start) {
----------------
"Search for an option which matches Str."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83639