r342581 - test: actually fix the condition properly

Saleem Abdulrasool via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 19 12:20:30 PDT 2018


Author: compnerd
Date: Wed Sep 19 12:20:30 2018
New Revision: 342581

URL: http://llvm.org/viewvc/llvm-project?rev=342581&view=rev
Log:
test: actually fix the condition properly

I had locally changed the test to add an explicit triple to figure out the issue
with the SCEI buildbots, and that hid the error.  This now works with and
without the explicit triple.

Modified:
    cfe/trunk/test/Sema/format-strings.c

Modified: cfe/trunk/test/Sema/format-strings.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/format-strings.c?rev=342581&r1=342580&r2=342581&view=diff
==============================================================================
--- cfe/trunk/test/Sema/format-strings.c (original)
+++ cfe/trunk/test/Sema/format-strings.c Wed Sep 19 12:20:30 2018
@@ -401,7 +401,7 @@ void bug7377_bad_length_mod_usage() {
 void pr7981(wint_t c, wchar_t c2) {
   printf("%lc", c); // no-warning
   printf("%lc", 1.0); // expected-warning{{the argument has type 'double'}}
-#if __WINT_TYPE__ == int
+#if __WINT_WIDTH__ == 32
   printf("%lc", (char) 1); // no-warning
 #else
   printf("%lc", (char) 1); // expected-warning{{the argument has type 'char'}}




More information about the cfe-commits mailing list