Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mark Kettenis <kettenis@chello.nl>
To: mec.gnu@mindspring.com
Cc: eliz@gnu.org, gdb-patches@sources.redhat.com, cagney@gnu.org
Subject: Re: [PATCH] Improve i386 prologue analyzer
Date: Sun, 08 Aug 2004 11:08:00 -0000	[thread overview]
Message-ID: <200408081108.i78B8Cpk009362@elgar.kettenis.dyndns.org> (raw)
In-Reply-To: <4115FF44.nail59F11C8I0@mindspring.com> (message from Michael Chastain on Sun, 08 Aug 2004 06:24:04 -0400)

   Date: Sun, 08 Aug 2004 06:24:04 -0400
   Sender: mec.gnu@mindspring.com

   eliz> If so, could you please explain why you think it needs any more
   eliz> ``marinating'' in HEAD?

   It's a new design.

Yeah, it's a pretty invasive change.  It not only affects the
unwinder, but also prologue skipping in order to determine the first
line of real code for a function.

   And this feature interacts with many different external variations.
   For example, I haven't seen anyone test it with gcc 2.

I see no changes in the testsuite on i386-unknown-openbsd3.5 and
i386-unknown-freebsd4.7 which both use GCC 2.95.3/4 as their system
compiler.  I think these are fine since the code is never used.  I'm
more worried about GCC 3.1/3.2.  Those compilers are in pretty wide use.

   Or with a program that uses a lot of floating point.

That shouldn't be too relevant.  I mean, if they were broken before
the patch, they're probably still broken.  But that doesn't really
matter for the branch, does it?

Personally I'd like to wait for another two weeks, before deciding
it's ok.  I'd be willing to check it in on the branch *now* if we'd
agree we'd release 6.2.1 after that period.  That gives me the
oportunity to pull it out if any problems arise.

Mark


  reply	other threads:[~2004-08-08 11:08 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-01 21:58 Mark Kettenis
2004-08-02  4:04 ` Eli Zaretskii
2004-08-02 21:19   ` Andrew Cagney
2004-08-03  3:55     ` Eli Zaretskii
2004-08-06 19:33       ` Mark Kettenis
2004-08-06 20:29         ` Andrew Cagney
2004-08-07 15:37           ` Eli Zaretskii
2004-08-07 16:20             ` Andrew Cagney
2004-08-07 17:11               ` Eli Zaretskii
2004-08-07 17:38                 ` Andrew Cagney
2004-08-07 18:30               ` Joel Brobecker
2004-08-07 18:52                 ` Andrew Cagney
2004-08-07 22:41               ` Michael Chastain
2004-08-08  3:57                 ` Eli Zaretskii
2004-08-08 10:24                   ` Michael Chastain
2004-08-08 11:08                     ` Mark Kettenis [this message]
2004-08-08 14:08                       ` Andrew Cagney
2004-08-08 15:04                         ` Mark Kettenis
2004-08-08 19:32                         ` Eli Zaretskii
2004-08-09 13:59                           ` Andrew Cagney
2004-08-09 15:07                             ` Mark Kettenis
2004-08-09 16:46                             ` Michael Chastain
2004-08-09 19:09                             ` Eli Zaretskii
2004-08-11  0:02                               ` Andrew Cagney
2004-08-11  3:53                                 ` Eli Zaretskii
2004-08-11 17:13                                   ` Andrew Cagney
2004-08-11 17:55                                     ` Eli Zaretskii
2004-08-12 12:43                                       ` Andrew Cagney
2004-08-12 19:00                                         ` Eli Zaretskii
2004-08-12 21:41                                           ` Andrew Cagney
2004-08-08 19:29                       ` Eli Zaretskii
2004-08-08 19:28                     ` Eli Zaretskii
2004-08-07 15:31         ` Eli Zaretskii
  -- strict thread matches above, loose matches on Subject: below --
2003-08-18  9:42 Michael Elizabeth Chastain
2003-08-18 17:17 ` Mark Kettenis
2003-08-17 23:34 Michael Elizabeth Chastain
2003-08-17 23:16 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=200408081108.i78B8Cpk009362@elgar.kettenis.dyndns.org \
    --to=kettenis@chello.nl \
    --cc=cagney@gnu.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mec.gnu@mindspring.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