[PATCH] D39539: [Support] Don't error out on multiple occurrances of an Optional flag
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 2 13:30:17 PDT 2017
mstorsjo added a comment.
In https://reviews.llvm.org/D39539#914121, @zturner wrote:
> Agreed. `Optional` should basically be called `OneOrZero`. And if you think about it in those terms, it obviously doesn't make sense to allow it to be specified more than once.
>
> Just change this specific option to be `ZeroOrMore`
Right - I hadn't realized how one could change that in `cl::opt` - now I see.
https://reviews.llvm.org/D39539
More information about the llvm-commits
mailing list