thakis added a comment.

The usual process for adding / tweaking warnings is to build some large 
open-source project with the new / tweaked warning and note true and false 
positive rate. IIRC I did this on Chromium when I added the warning, and the 
out-of-bounds check prevented false positives without removing true positives. 
(...or maybe Richard asked for it during review? Did you try and find the 
review where this got added?)

That is, can you try building Chromium with this tweak and see what it does to 
true and false positive rate?


Repository:
  rC Clang

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

https://reviews.llvm.org/D55382



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

Reply via email to