[PATCH] D15935: Improve diagnostics for literal conversion to Boolean

[Aaron Ballman via cfe-commits]

aaron.ballman added inline comments.

================
Comment at: lib/Sema/SemaChecking.cpp:6985
@@ -6984,6 +6984,3 @@
   SmallString<16> PrettyTargetValue;
-  if (T->isSpecificBuiltinType(BuiltinType::Bool))
-    PrettyTargetValue = Value.isZero() ? "false" : "true";
-  else
-    IntegerValue.toString(PrettyTargetValue);
+  IntegerValue.toString(PrettyTargetValue);
 
----------------
manmanren wrote:
> Why do we remove the pretty printing for boolean here? Providing context here will be nice.
Because float->Boolean conversion has moved elsewhere (to line 7207 in the new file).


http://reviews.llvm.org/D15935