Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: David Carlton <carlton@math.stanford.edu>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [drow-cplus-branch] lookup_symbol_aux refactoring
Date: Fri, 11 Oct 2002 09:31:00 -0000	[thread overview]
Message-ID: <ro1lm55hr92.fsf@jackfruit.Stanford.EDU> (raw)
In-Reply-To: <20021011161643.GA2862@nevyn.them.org>

On Fri, 11 Oct 2002 12:16:43 -0400, Daniel Jacobowitz <drow@mvista.com> said:
> On Fri, Oct 11, 2002 at 09:07:51AM -0700, David Carlton wrote:

>> Is it okay if I commit the lookup_symbol_aux refactoring from 
>> <http://sources.redhat.com/ml/gdb-patches/2002-10/msg00080.html> to
>> drow-cplus-branch?  I've got some patches to symbol table lookup to
>> handle namespace stuff that are almost ready (I just want to add some
>> testcases), and they'll be a lot cleaner if I get this part of them
>> out of the way first.

> Hmm... yes, this will be useful for me, too.  The only bit I'm nervous
> about is the changed search order, but we can update this to match
> whatever goes in mainline when your patch is approved.  Please do.

Done.  Yeah, I'm not convinced that the changed search order is
ultimately the correct thing to do either.  What is important for me
right now is to have an easy way to look up a symbol in the global
environment.  Unfortunately, the way minimal symbol lookup is
currently handled screws that up, because it will use static symbols as
a fallback measure.

So ultimately that will have to get sorted out, and we can go with
whatever solution is adopted on the mainline.  And I honestly have no
idea what that solution will be: it seems to me that trying to use all
three of symtabs, psymtabs, and minimal symbols is probably redundant,
but maybe I'm missing something and even if they are I don't have a
solid feeling for the best way to remove some of the redundancy.

For now, I'm adopting the tactic of making sure that things work well
if there's proper debugging info but also making the code modular
enough that that aspect of the search strategy will be easy to
change.

David Carlton
carlton@math.stanford.edu


      reply	other threads:[~2002-10-11 16:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-11  9:07 David Carlton
2002-10-11  9:16 ` Daniel Jacobowitz
2002-10-11  9:31   ` David Carlton [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ro1lm55hr92.fsf@jackfruit.Stanford.EDU \
    --to=carlton@math.stanford.edu \
    --cc=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox