[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