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] Fix -fno-for-scope (PR c++/47388)

From: Jason Merrill <jason at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Fri, 21 Jan 2011 16:28:39 -0500
Subject: Re: [C++ PATCH] Fix -fno-for-scope (PR c++/47388)
References: <20110121194204.GL2724@tyan-ft48-01.lab.bos.redhat.com>

On 01/21/2011 02:42 PM, Jakub Jelinek wrote:

The following patch fixes it by not calling begin_for_scope
in that case again.

OK.

BTW, I was kind of surprised pt.c calls begin_for_stmt
for both FOR_STMT and RANGE_FOR_STMT, I'd kind of expect
it should be begin_range_for_stmt in the latter case, but
haven't investigated in detail...

Calling begin_for_stmt is correct; RANGE_FOR_STMT only exists in templates, at tsubst time we convert them into the equivalent C-style for.

Jason

References:
- [C++ PATCH] Fix -fno-for-scope (PR c++/47388)
  - From: Jakub Jelinek

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