[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