[cfe-commits] [patch] Add support for UCNs in character literals

Nico Weber thakis at chromium.org
Thu Oct 7 22:50:59 PDT 2010


Now with -triple line in the new file, so that this hopefully works on
windows, too.

For what it's worth, with this patch clang can build all of
chromium/mac in debug mode.

Nico

On Wed, Oct 6, 2010 at 9:57 PM, Nico Weber <thakis at chromium.org> wrote:
> Hi,
>
> the attached patch adds support for UCN notation to character
> literals, e.g. L'\u1234'. It splits EncodeUCNEscape() into two pieces
> and calls only the first piece for character literals.
>
> Note that overflow (e.g. L'\U0010fffe' with -fshort-wchar) is handled
> differently than in gcc, because the gcc behavior makes no sense to
> me, and I guess this is implementation-defined anyhow and not used in
> practice.
>
> Please review.
>
> Thanks,
> Nico
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-4-byte-ucn.patch
Type: application/octet-stream
Size: 5378 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20101007/06cfcf0a/attachment.obj>


More information about the cfe-commits mailing list