Hey Richard, One more issue that I missed in the initial review (sorry about that)
On Wed, Jul 20, 2011 at 7:46 PM, Richard Trieu <[email protected]> wrote: > - test0 = test0 ? EVal : (int) test0; // okay: EVal is an int > - test0 = test0 ? // expected-warning {{operands of ? are integers of > different signs}} > + test1 = test0 ? EVal : (int) test0; > + test0 = test0 ? > (unsigned) EVal > - : (int) test0; > + : (int) test0; // expected-warning {{operand of ? changes > signedness: 'int' to 'unsigned long'}} > Do we want to teach -Wsign-conversion to not warn about the conversion issue when both LHS and RHS have explicit casts? This seems like the only spot that's picking up a warning we didn't produce before...
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
