This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [testsuite, committed] Replace absolute line numbers in c-c++-common
- From: Mike Stump <mikestump at comcast dot net>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 3 May 2017 18:17:31 -0700
- Subject: Re: [testsuite, committed] Replace absolute line numbers in c-c++-common
- Authentication-results: sourceware.org; auth=none
- References: <4980bc1a-058d-9b49-2232-af5289eb7c29@mentor.com>
On May 3, 2017, at 12:39 AM, Tom de Vries <Tom_deVries@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.