[PATCH] D21485: [cl] Teach cl to support the notion of sub commands (e.g. "git checkout <foo>")

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 11:57:55 PDT 2016


@beanz, are you ok with this change now given that there is a unit test
verifying that `removeOption` works with `AllSubCommands` and
`TopLevelSubCommand`?

On Mon, Jun 27, 2016 at 1:00 PM Zachary Turner <zturner at google.com> wrote:

> zturner updated this revision to Diff 62006.
> zturner added a comment.
>
> Sorry for the delay, I was out the second half of last week.  This takes
> all the previous suggestions into account, and adds unit tests to make sure
> that removing from `AllSubCommands` and `TopLevelSubCommand` works as
> expected.
>
>
> http://reviews.llvm.org/D21485
>
> Files:
>   include/llvm/Support/CommandLine.h
>   lib/Support/CommandLine.cpp
>   unittests/Support/CommandLineTest.cpp
>   unittests/Support/ProgramTest.cpp
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160628/24395f2c/attachment.html>


More information about the llvm-commits mailing list