This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C++ PATCH] Remove nested function support remnants
- From: Mark Mitchell <mark at codesourcery dot com>
- To: Steven Bosscher <s dot bosscher at student dot tudelft dot nl>
- Cc: gcc-patches at gcc dot gnu dot org, Gabriel Dos Reis <gdr at integrable-solutions dot net>
- Date: 15 Sep 2003 10:24:24 -0700
- Subject: Re: [C++ PATCH] Remove nested function support remnants
- References: <1063632819.3989.2.camel@steven.lr-s.tudelft.nl><m34qzexioa.fsf@uniton.integrable-solutions.net><1063640418.10388.3.camel@minax.codesourcery.com> <1063645375.3988.74.camel@steven.lr-s.tudelft.nl>
On Mon, 2003-09-15 at 10:02, Steven Bosscher wrote:
> Op ma 15-09-2003, om 17:40 schreef Mark Mitchell:
> > > I would say that it is a leftover of the era when cc1plus started as
> > > copy-n-paste of cc1. Moreover, C++ provides clean and standard ways
> > > to implement inline nested-functions in terms of local classes.
> > > I would suggest to get rid of that piece of code.
> >
> > Yes, this is probably dead code.
>
> Very well. This patch removes it all.
>
> Bootstrapped and regtested on i686-pc-linux-gnu. OK?
I suspect not, actually. The function_depth stuff is necessary not for
the case of lexical nesting, but for things like a function
instantiating a template and then emitting the template.
I think you should stick with just removing the block you originally
pointed out.
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com