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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to