Prior to this patch, the CommandLine parser would strip an
unlimitted number of dashes from options. This patch limits it to
two.
This is the first of a set of patches that address how dashes are
handled by the CommandLine parser.
Paths
| Differential D61229
[CommandLine] Don't allow unlimitted dashes for options. Part 1 or 5 ClosedPublic Authored by hintonda on Apr 27 2019, 10:59 AM.
Details Summary Prior to this patch, the CommandLine parser would strip an This is the first of a set of patches that address how dashes are
Diff Detail
Event Timelinehintonda retitled this revision from [CommandLine] Don't allow unlimitted dashes for options. to [CommandLine] Don't allow unlimitted dashes for options. Part 1 or 5.Apr 29 2019, 10:40 AM Comment Actions 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. This revision is now accepted and ready to land.Apr 29 2019, 11:19 AM Closed by commit rL359480: [CommandLine] Don't allow unlimitted dashes for options. Part 1 or 5 (authored by dhinton). · Explain WhyApr 29 2019, 11:32 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 197152 llvm/trunk/lib/Support/CommandLine.cpp
llvm/trunk/test/DebugInfo/X86/array.ll
llvm/trunk/test/DebugInfo/dwarfdump-64-bit-dwarf.test
|