sashab added a comment.
Thanks all! :)
Comment at: test/SemaCXX/warn-msvc-enum-bitfield.cpp:12
@@ +11,3 @@
+ s.e2 = E2;
+ s.f2 = F2;
> Shouldn't this be the version that warns? The assignment with E1 assigns 0
> which is portable, but this assigns 1 which overflows, right?
e2 is not a bitfield :) So this code is fine.
And we should warn on all assignments, since any assigned value could
potentially be incorrect. Also, most assignments are not static so we don't
always have that information :)
cfe-commits mailing list