[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