[Patch] Option parsing: allow option aliases in option groups

Reid Kleckner rnk at google.com
Mon Jul 22 07:38:25 PDT 2013


LGTM


On Fri, Jul 19, 2013 at 8:34 PM, Hans Wennborg <hans at chromium.org> wrote:

> Hi,
>
> This patch makes the option parsing code allow for having option
> aliases in option groups.
>
> Aliases in groups were previously disallowed by an assert. As far as I
> know, there is no technical reason for this, and I would like to allow
> them, to be able to put cl.exe compatible options in their own group,
> but have some of them (like "/c" for example) be aliases.
>
> Please take a look.
>
> Thanks,
> Hans
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130722/ce27e0e4/attachment.html>


More information about the llvm-commits mailing list