[PATCH] D63912: [ObjC] Add a warning for implicit conversions of a constant non-boolean value to BOOL

Mon Jul 8 13:41:18 PDT 2019

rjmccall added inline comments.

================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3251
+  "the only well defined values for BOOL are YES and NO">,
+  InGroup<ConstantConversion>;

----------------
The `from` seems awkward here; `%0` is the number, not the type.  (I agree that the type isn't important for the diagnostic.)

Thoughts about this being in a sub-group?  I'm not sure either way.

Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63912/new/

https://reviews.llvm.org/D63912