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

Reply via email to