[PATCH] D42728: Add more warnings for implict conversions (e.g. double truncation to float).

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 14 10:44:23 PDT 2018


rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D42728#1036743, @avt77 wrote:

> I removed cheking of bit-field case.
>  And I was wrong about increasing of number of failed tests: it's starnge but I was sure I saw it. OK, now we have one modified test only.


Yeah, the previous behavior is just a simple recursion without any assignment-specific knowledge, so that makes sense.

LGTM.


https://reviews.llvm.org/D42728





More information about the llvm-commits mailing list