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

[Don Hinton via Phabricator via llvm-commits]

hintonda marked an inline comment as done.
hintonda added inline comments.


================
Comment at: llvm/lib/Support/CommandLine.cpp:108
 
+static inline bool isGrouping(const Option *O);
+
----------------
MaskRay wrote:
> Just move the definition here?
Was trying to minimize the diff as much as possible, but there's no reason not to move these two here.

I could also move the others, but then it would be more difficult to see what I changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61294