On May 3, 2017, at 12:39 AM, Tom de Vries <tom_devr...@mentor.com> wrote: > diff --git a/gcc/testsuite/c-c++-common/Wshift-negative-value-1.c > b/gcc/testsuite/c-c++-common/Wshift-negative-value-1.c > index 8f14034..7df1804 100644 > --- a/gcc/testsuite/c-c++-common/Wshift-negative-value-1.c > +++ b/gcc/testsuite/c-c++-common/Wshift-negative-value-1.c > @@ -7,6 +7,7 @@ enum E { > A = 0 << 1, > B = 1 << 1, > C = -1 << 1, /* { dg-warning "left shift of negative value|not an integer > constant" } */ > + /* { dg-error "left operand of shift expression" "shift" { target c++ } > .-1 } */ > D = 0 >> 1, > E = 1 >> 1, > F = -1 >> 1 > @@ -47,5 +48,3 @@ right (int x) > r += -1U >> x; > return r; > } > - > -/* { dg-error "left operand of shift expression" "shift" { target c++ } 9 } > */
I like this type of change, thanks.