This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: C PATCH for c/64637 (better location for -Wunused-value)

From: Jeff Law <law at redhat dot com>
To: Marek Polacek <polacek at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Joseph Myers <joseph at codesourcery dot com>
Date: Wed, 16 Dec 2015 09:42:29 -0700
Subject: Re: C PATCH for c/64637 (better location for -Wunused-value)
Authentication-results: sourceware.org; auth=none
References: <20151216145831 dot GF3296 at redhat dot com>

On 12/16/2015 07:58 AM, Marek Polacek wrote:

The following improves the location for "statement with no effect" warning by
using the location of the expression if available.  Can't use EXPR_LOCATION as
*_DECLs still don't carry a location.

Bootstrapped/regtested on x86_64-linux, ok for trunk?

2015-12-16  Marek Polacek  <polacek@redhat.com>

	PR c/64637
	* c-typeck.c (c_process_expr_stmt): Use location of the expression if
	available.

	* gcc.dg/pr64637.c: New test.

OK.
jeff

References:
- C PATCH for c/64637 (better location for -Wunused-value)
  - From: Marek Polacek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]