[cfe-commits] r160324 - /cfe/trunk/test/FixIt/fixit-unicode.c

Jordan Rose jordan_rose at apple.com
Mon Jul 16 15:18:33 PDT 2012


Author: jrose
Date: Mon Jul 16 17:18:33 2012
New Revision: 160324

URL: http://llvm.org/viewvc/llvm-project?rev=160324&view=rev
Log:
Weaken Unicode fixit test to work even if the character is printed escaped.

This should fix the failure on Windows.

Modified:
    cfe/trunk/test/FixIt/fixit-unicode.c

Modified: cfe/trunk/test/FixIt/fixit-unicode.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/FixIt/fixit-unicode.c?rev=160324&r1=160323&r2=160324&view=diff
==============================================================================
--- cfe/trunk/test/FixIt/fixit-unicode.c (original)
+++ cfe/trunk/test/FixIt/fixit-unicode.c Mon Jul 16 17:18:33 2012
@@ -1,6 +1,5 @@
 // RUN: %clang_cc1 -fsyntax-only %s 2>&1 | FileCheck -strict-whitespace %s
 // RUN: %clang_cc1 -fsyntax-only -fdiagnostics-parseable-fixits %s 2>&1 | FileCheck -check-prefix=CHECK-MACHINE %s
-// XFAIL: win32
 
 struct Foo {
   int bar;
@@ -24,9 +23,11 @@
   printf("∆: %d", 1L);
 // CHECK: warning: format specifies type 'int' but the argument has type 'long'
 // Don't crash emitting a fixit after the delta.
-//     CHECK-NEXT:  printf("∆: %d", 1L);
-// CHECK-NEXT: {{^             ~~   \^~}}
-// CHECK-NEXT: {{^             %ld}}
+// CHECK:  printf("
+// CHECK: : %d", 1L);
+// Unfortunately, we can't actually check the location of the printed fixit,
+// because different systems will render the delta differently (either as a
+// character, or as <U+2206>.) The fixit should line up with the %d regardless.
 
 // CHECK-MACHINE: fix-it:"{{.*}}fixit-unicode.c":{23:16-23:18}:"%ld"
 }





More information about the cfe-commits mailing list