On Oct 3, 2010, at 11:19 PM, Nico Weber wrote:
> Sean Hunt requested that ucns > 0xffff should be converted to utf16 in
> addition to displaying a warning, and that there should be tests for
> the converted text. The attached revised patch adds these two things.
+++ test/Lexer/wchar-warns-in-c89.c (revision 0)
@@ -0,0 +1,5 @@
+// RUN: %clang_cc1 -fsyntax-only -std=c89 -verify %s
+
+void f() {
+ (void)L"\u1234"; // expected-warning {{unicode escape sequences are only
valid in C99 or C++}}
+}
Please add this to test/Lexer/c90.c. Tests that can be checked with -verify
should follow a functional grouping instead of having one test per file.
The patch to lib/Lex/LiteralSupport.cpp is greek[1] to me, but your output
matches llvm-gcc... so... sounds great! Please commit! :-)
-Chris
[1] Hi Argiris :)
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits