[PATCH] D61294: [CommandLine] Add long option flag for cl::ParseCommandLineOptions . Part 5 of 5

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 19:31:45 PDT 2019


MaskRay added inline comments.


================
Comment at: llvm/lib/Support/CommandLine.cpp:108
 
+static inline bool isGrouping(const Option *O);
+
----------------
Just move the definition here?


================
Comment at: llvm/lib/Support/CommandLine.cpp:404
+    if (Opt && LongOptionsUseDoubleDash && !HaveDoubleDash && !isGrouping(Opt))
+      Opt = nullptr;
+    return Opt;
----------------
return nullptr;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61294





More information about the llvm-commits mailing list