This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [rfa] generate symbols associated to namespaces
- From: Daniel Jacobowitz <drow at mvista dot com>
- To: gdb-patches at sources dot redhat dot com
- Cc: carlton at kealia dot com, ezannoni at redhat dot com
- Date: Sun, 31 Aug 2003 15:29:50 -0400
- Subject: Re: [rfa] generate symbols associated to namespaces
- References: <m3k7blg4dr.fsf@kealia.com> <20030622173547.GA22603@nevyn.them.org> <ro1brwntjhz.fsf@jackfruit.Stanford.EDU> <20030624185019.GA24662@nevyn.them.org> <yf28yrn1p8i.fsf@hawaii.kealia.com> <yf2k7b7yyg5.fsf@hawaii.kealia.com> <20030805175429.GA29704@nevyn.them.org>
On Tue, Aug 05, 2003 at 01:54:29PM -0400, Daniel Jacobowitz wrote:
> On Fri, Jun 27, 2003 at 02:58:50PM -0700, David Carlton wrote:
> > On Fri, 27 Jun 2003 09:04:13 -0700, David Carlton <carlton@kealia.com> said:
> >
> > > I've looked at your code comments, and they all seem reasonable. I
> > > hope I'll have a revised version of the patch today; job transition
> > > issues will take up some of my time (I start full-time work at
> > > Kealia today, so I have to figure out what they want me to do other
> > > than hack GDB), but it's near the top of my priority list.
> >
> > Okay, here's the new version. Following Daniel's suggestion, it puts
> > symbols associated to DW_TAG_namespace in the symtab where they are
> > found, instead of in a special block, and creates one possible
> > namespace block for each objfile. This fixes the problem Daniel found
> > with the last version of my patch (which, incidentally, turns out also
> > to show up under DWARF 2 with pr-1210.exp; the more tests the
> > merrier).
> >
> > Tested with GCC 3.2, i686-pc-linux-gnu with both DWARF 2 and stabs+,
> > and with a version of GCC 3.2.3 that generates DW_TAG_namespace DIEs.
> > No new regressions (though the new tests in gdb.c++/namespace.exp fail
> > under stabs+, as expected).
> >
> > How does this one look to y'all?
>
> Sorry 'bout the delay. This looks good to me; Elena?
This patch needs a trivial update for the gdb.c++ -> gdb.cp rename, and
IIRC one other merge update for the testsuite; and the comments we
discussed downthread. Mind reposting it?
Elena, could you look at this? I have some other C++ work which is
blocked waiting for this patch.
--
Daniel Jacobowitz
MontaVista Software Debian GNU/Linux Developer