[PATCH] clang-cl: Add the /c, /W0 and /W1 options
ruiu at google.com
Wed Jul 31 13:22:53 PDT 2013
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
> 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
> >> 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...
More information about the cfe-commits