Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Kevin Buettner <kevinb@redhat.com>
To: "David S. Miller" <davem@redhat.com>, kevinb@redhat.com
Cc: gdb@sources.redhat.com, gdb-patches@sources.redhat.com
Subject: Re: [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING (was Re: multi-arch TODO)
Date: Tue, 23 Apr 2002 09:43:00 -0000	[thread overview]
Message-ID: <1020423164342.ZM9027@localhost.localdomain> (raw)
In-Reply-To: "David S. Miller" <davem@redhat.com> "[RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING (was Re: multi-arch TODO)" (Apr 23,  3:00am)

On Apr 23,  3:00am, David S. Miller wrote:

>    >   Why don't all Linux targets define this?  Do some binutils ports
>    >   perform this optimization and others not?  Or was there some bug
>    >   in N_FUN/N_SO stabs in binutils and/or gcc that this is papering
>    >   around?  kevinb@cyghat.com is the one who added this to powerpc
>    >   and i386 Linux.
>    
>    I haven't given it a lot of thought recently, but my opinion is that
>    the SOFUN_ADDRESS_MAYBE_MISSING code ought to be enabled everywhere.
>    The only downside that I can think of is that we lose the ability
>    to put a symbol at address 0.
> 
> Sounds find to me, how about this patch?
> 
> 2002-04-23  David S. Miller  <davem@redhat.com>
> 
> 	* config/i386/tm-i386sol2.h, config/i386/tm-linux.h,
> 	config/powerpc/tm-linux.h, config/powerpc/tm-ppc-eabi.h,
> 	config/sparc/tm-sun4sol2.h (SOFUN_ADDRESS_MAYBE_MISSING): Kill.
> 	* dbxread.c, elfread.c, minsyms.c, mdebugread.c, symmisc.c,
> 	symtab.h (whole file): Act as if SOFUN_ADDRESS_MAYBE_MISSING was
> 	always defined, kill ifdefs.

Yes, this is exactly what I had in mind.  It looks okay to me, but it
needs to be carefully considered and approved (or not) by the symtab
maintainers.

As noted earlier, the downside is that we lose the ability to put a
symbol at address 0.  This doesn't pose a problem for operating
systems which'll never map part of the program at address 0, but there
may be some embedded environments for which this is a concern. 

Kevin


  parent reply	other threads:[~2002-04-23 16:43 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20020422.040949.16307644.davem@redhat.com>
     [not found] ` <1020422152449.ZM1373@localhost.localdomain>
2002-04-23  3:10   ` David S. Miller
2002-04-23  6:52     ` Eli Zaretskii
2002-04-23  7:01       ` [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING David S. Miller
2002-04-23  9:05         ` Eli Zaretskii
2002-04-23  9:43     ` Kevin Buettner [this message]
2002-04-23 10:58       ` [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING (was Re: multi-arch TODO) Michael Snyder
2002-04-23 12:10         ` Kevin Buettner
2002-04-23 23:11           ` [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING David S. Miller
2002-04-23 12:05     ` [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING (was Re: multi-arch TODO) Elena Zannoni
2002-04-23 23:13       ` [RFA] Kill SOFUN_ADDRESS_MAYBE_MISSING David S. Miller
2002-04-24  1:03       ` David S. Miller
2002-04-24 17:40         ` Elena Zannoni
2002-04-24 17:53           ` David S. Miller
2002-04-24  1:13       ` David S. Miller

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=1020423164342.ZM9027@localhost.localdomain \
    --to=kevinb@redhat.com \
    --cc=davem@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=gdb@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