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: [PATCH, cpp] Fix pr61817 and 69391

From: Manuel LÃpez-IbÃÃez <lopezibanez at gmail dot com>
To: Richard Henderson <rth at redhat dot com>, gcc-patches at gcc dot gnu dot org
Cc: David Malcolm <dmalcolm at redhat dot com>, "Joseph S. Myers" <joseph at codesourcery dot com>
Date: Tue, 5 Apr 2016 19:03:49 +0100
Subject: Re: [PATCH, cpp] Fix pr61817 and 69391
Authentication-results: sourceware.org; auth=none
References: <5703E64D dot 3090503 at redhat dot com>

On 05/04/16 17:22, Richard Henderson wrote:

These two related PRs are all about remembering where a macro is expanded.
Worse, we've got two competing goals -- the real location of the expansion, for
__LINE__, and the virtual location of the expansion, for diagnostics.

There seems to be no way to unify the two competing goals.  If we simply "fix"
the first, we break the second.  Therefore, I resort to passing down both
locations.

+++ b/gcc/testsuite/gcc.dg/pr61817.c
@@ -0,0 +1,19 @@
+/* { dg-do compile } */
+/* { dg-options "-std=c11 -ftrack-macro-expansion=0" } */
+

Why use -ftrack-macro-expansion=0? This should work with =1, which is also thedefault, no?


Cheers,

	Manuel.

Follow-Ups:
- Re: [PATCH, cpp] Fix pr61817 and 69391
  - From: Richard Henderson

References:
- [PATCH, cpp] Fix pr61817 and 69391
  - From: Richard Henderson

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