[PATCH] D15814: Implicit conversion from float->bool

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 29 13:56:51 PST 2015


aaron.ballman added a comment.

In http://reviews.llvm.org/D15814#317879, @Eugene.Zelenko wrote:

> This seems to me as specific case of Clang-tidy readability-implicit-bool-cast. May be this check should be entirely moved to Clang?


I think the reason why that is in clang-tidy is because it would be highly chatty and based on user's style preferences. This particular diagnostic has to do with implicit conversions that involve literals requiring conversion to the destination type, which are far less frequent than implicit conversions in general.


http://reviews.llvm.org/D15814





More information about the cfe-commits mailing list