[PATCH] clang-cl: Add the /c, /W0 and /W1 options

[Rui Ueyama]

On Wed, Jul 31, 2013 at 1:19 PM, Hans Wennborg <hans at chromium.org> wrote:

> On Wed, Jul 31, 2013 at 12:58 PM, Rui Ueyama <ruiu at google.com> wrote:
> > This is not a comment for this specific patch, but I'm wondering if the
> > driver can handle options in a case-insensitive manner. Does it?
>
> No, it can't.
>
> AFAIK, the only cl.exe option that's case-insensitive is /help, and
> I'm not sure if that's important to support.
>
> Do you need case-insensitive parsing for lld?
>

Yes I do. The linker options seems to be case insensitive. I'd guess cl.exe
is exceptional in the Windows world where the option is case insensitive in
general.

> On Wed, Jul 31, 2013 at 11:30 AM, Reid Kleckner <rnk at google.com> wrote:
> >>
> >>
> >>   LGTM
> >>
> >>   I would go ahead and commit, since this doesn't have any review-worthy
> >> bits now that these are just aliases.  If someone objects to
> "CoreOption"
> >> they can bikeshed it after the commit.
> >>
> >> http://llvm-reviews.chandlerc.com/D1232
> >> _______________________________________________
> >> cfe-commits mailing list
> >> cfe-commits at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130731/258843c6/attachment.html>