erichkeane added a comment. I believe that this is a good patch that solves an open Bug. See : https://llvm.org/bugs/show_bug.cgi?id=25965
I think the SPEC case is much less compelling, but I think that using a float explicitly in a conditional is something I really wouldn't expect a warning on. @rsmith You had commented on the bug initially suggesting suppressing the diagnostic entirely in this case. Do you still feel that way? https://reviews.llvm.org/D26636 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits