thakis added a comment.

It looks like bits of this landed recently. I don't know which of the commits 
it was, so I'm commenting here: The -Wint-in-bool context that clang emits are 
very useful and have found several bugs in Chromium, with just one false 
positive (which was in confusing code, so we changed that one too). Thank you 
for this useful warning :)

(https://crbug.com/1007367 lists what this founds, if you're curious.)


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

https://reviews.llvm.org/D63082



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D63082: [Diagnos... Nico Weber via Phabricator via cfe-commits

Reply via email to