sashab added a comment.

Thanks all! :)

Comment at: test/SemaCXX/warn-msvc-enum-bitfield.cpp:12
@@ +11,3 @@
+  s.e2 = E2;
+  s.f2 = F2;
thakis wrote:
> 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

Reply via email to