alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.

In https://reviews.llvm.org/D20689#633889, @varjujan wrote:

> I ran the check on multiple projects and tried to categorize the warnings: 
> real errors, false positives, naming errors and coincidences. The results are 
> attached. I got no warnings on LLVM.


I didn't find any "real errors" in the files you posted, which means that 
either all of these projects are of extreme code quality or that the error is 
rather unlikely to happen.

Another concern is that the check seems to treat argument types in a rather 
primitive way, e.g. it doesn't consider any type conversions or promotions.


https://reviews.llvm.org/D20689



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

Reply via email to