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

Reid Kleckner rnk at google.com
Wed Jul 31 13:24:08 PDT 2013


Hm, no.  It looks like cl.exe is case sensitive, but link.exe is not.  I
think we can handle case sensitivity as a property of the OptTable.  I'm
not sure if we'll have to change the table generator to support this.


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?
>
>
> 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/ea6485c6/attachment.html>


More information about the cfe-commits mailing list