[PATCH] don't diagnose -Wchar-subscript for safe char literal

Nico Weber thakis at chromium.org
Thu Apr 17 11:02:19 PDT 2014


On Thu, Apr 17, 2014 at 7:48 AM, Daniel Marjamäki <
Daniel.Marjamaki at evidente.se> wrote:

> Thanks!
>
> > Looks like your new test will fail on platforms where plain char is
> unsigned. Please make sure the test has an explicit triple to avoid that,
> then LGTM.
>
> Interesting. I thought it was signed always when compiling with Clang.
>
> I am not sure what you mean by explicit triple. Sorry. Could you show me?
>

$ ack triple test/
test/Analysis/bitwise-ops.c
1:// RUN: %clang_cc1 -analyze -analyzer-checker=core,debug.ExprInspection
-triple x86_64-apple-darwin13 -Wno-shift-count-overflow -verify %s
[...]


>
> Best regards,
> Daniel Marjamäki
>
> _______________________________________________
> 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/20140417/bba3c744/attachment.html>


More information about the cfe-commits mailing list