Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Jim Blandy <jimb@red-bean.com>
Cc: Mark Kettenis <mark.kettenis@xs4all.nl>, gdb-patches@sourceware.org
Subject: Re: [RFC/RFA] Deal with -g1 generated DWARF2 debug info
Date: Mon, 05 Dec 2005 10:32:00 -0000	[thread overview]
Message-ID: <20051204221547.GA25034@nevyn.them.org> (raw)
In-Reply-To: <8f2776cb0512041400x257a1889u8fb53eb23e62ce49@mail.gmail.com>

On Sun, Dec 04, 2005 at 02:00:21PM -0800, Jim Blandy wrote:
> On 12/4/05, Daniel Jacobowitz <drow@false.org> wrote:
> > I don't see why we care what the prototype for malloc in the debugging
> > information is.  We know the type of malloc.  A system which uses a
> > different type for malloc won't work here anyway.
> 
> You mean, just ignore malloc's type altogether, and unconditionally
> construct our own type?

Yes.

> I'd prefer that over Mark's patch, but this isn't great.  If malloc
> has a bogus type, I think we should at least complain.  On
> free-standing systems that might be real information.
> 
> The central issue here is that the compiler is emitting incorrect
> debug info.  The GCC manual says of -g1:
> 
>      Level 1 produces minimal information, enough for making backtraces
>      in parts of the program that you don't plan to debug.  This
>      includes descriptions of functions and external variables, but no
>      information about local variables and no line numbers.
> 
> Nothing there about reducing all return types to 'void'.

The compiler is buggy, but I object to any classification of this as a
BSD-specific change; it's GCC that's buggy and Debian used -g1 too for
a while.  It ICEs on some targets, and does this sort of thing.

Mark, FYI, we ended up removing all dwarf sections other than
.debug_frame.  This plus .symtab are enough to make backtraces useful;
it works quite well and avoids the undertested -g1.

-- 
Daniel Jacobowitz
CodeSourcery, LLC


  reply	other threads:[~2005-12-04 22:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-04 22:05 Mark Kettenis
2005-12-04 22:31 ` Jim Blandy
2005-12-05  9:43   ` Mark Kettenis
2005-12-04 22:40 ` Jim Blandy
2005-12-04 23:06   ` Daniel Jacobowitz
2005-12-04 23:07     ` Jim Blandy
2005-12-05 10:32       ` Daniel Jacobowitz [this message]
2005-12-05 18:29         ` Mark Kettenis
2005-12-05 19:23           ` Daniel Jacobowitz
2005-12-07 16:19             ` Mark Kettenis
2005-12-05 16:56       ` Mark Kettenis
2005-12-05 19:57         ` Daniel Jacobowitz
2005-12-05 14:40   ` Mark Kettenis

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=20051204221547.GA25034@nevyn.them.org \
    --to=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    --cc=jimb@red-bean.com \
    --cc=mark.kettenis@xs4all.nl \
    /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