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: [3.4, C++] PR 14950

From: Mark Mitchell <mark at codesourcery dot com>
To: Jan Hubicka <jh at suse dot cz>
Cc: gcc-patches at gcc dot gnu dot org
Date: Mon, 21 Jun 2004 14:16:18 -0700
Subject: Re: [3.4, C++] PR 14950
Organization: CodeSourcery, LLC
References: <20040617203515.GW10156@kam.mff.cuni.cz>

Jan Hubicka wrote:

Hi,
after all this problem is not that dificult to fix.  C++ frontend manages to
instantiate template before emmiting it but it mistakely keeps
TI_PENDING_TEMPLATE_FLAG on that makes C++ frontend to think that it should
prohibit inlining here.

Clearing the flag slightly easier appears to do the trick.

Bootstrapped/regtested ia64-linux

OK? (I would like to install it into mainline too even tought the testcase does
not reproduce there as this interference might hit us in other scenario too)

This change is OK for 3.4.1 and mainline. However, please move the other code that clears TI_PENDING_TEMPLATE_FLAG (a couple of lines below your patch) up to the VAR_DECL case; it's confusing to clear the flag twice in a row.

Thanks,

--
Mark Mitchell
CodeSourcery, LLC
(916) 791-8304
mark@codesourcery.com

References:
- [3.4, C++] PR 14950
  - From: Jan Hubicka

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