[PATCH] D61229: [CommandLine] Don't allow unlimitted dashes for options. Part 1 or 5

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 29 11:19:42 PDT 2019


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm

Since we already have CommandLineTest.cpp, I think it would be reasonable to check that three dashes are rejected, but it's not necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61229





More information about the llvm-commits mailing list