public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Pedro Alves <palves@redhat.com>
To: Kevin Buettner <kevinb@redhat.com>, gdb-patches@sourceware.org
Cc: Simon Marchi <simon.marchi@polymtl.ca>,
	Joel Brobecker <brobecker@adacore.com>
Subject: Re: GDB 8.2 release 2018-08-21 status update
Date: Fri, 24 Aug 2018 18:35:00 -0000	[thread overview]
Message-ID: <a93d316e-4d0f-bf42-e5e6-48342a380dc1@redhat.com> (raw)
In-Reply-To: <20180823154138.66be5572@pinnacle.lan>

On 08/23/2018 11:41 PM, Kevin Buettner wrote:
> Simon Marchi <simon.marchi@polymtl.ca> wrote:

>> So from my point of view, it would be fine to include it in 8.2.  I'm 
>> just wondering though why this was considered as a blocker for 8.2 in 
>> the first place.  It's not really a regression, it's more like a new 
>> feature.  Was it to make sure we get the feature to users faster, before 
>> the new gcc that emits code like this by default starts to spread too 
>> much?

Yeah, the main issue here is that nowadays -freorder-blocks-and-partition 
is on by default in GCC (I believe the switch was flipped in GCC 8?), which
means users will run into this problem more frequently going forward.  While
technically it's not a GDB regression (you can always run into this if you
build your program with -freorder-blocks-and-partition explicitly), from
the perspective of end users looking at the toolchain as whole black box,
it's a toolchain regression (defaults no longer work).
I'd support putting the series in GDB 8.2.  Unless I'm wrong that
-freorder-blocks-and-partition is on by default in GCC 8 already?

> 
> According to Thomas Koenig, from GCC bug 84550:
> 
>     With gdb 8.0.1, stepping through functions after breakpoints is
>     often broken.  This makes it hard to debug gcc itself.
> 
> The non-contiguous address ranges patches _might_ make it easier for
> gcc developers to debug gcc.
While I'm all for helping our GCC hacker friends, I'm more concerned
with the general users, since presumably GCC developers would have no
big trouble building a new GDB from sources.

Thanks,
Pedro Alves

  reply	other threads:[~2018-08-24 18:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-21 17:51 Joel Brobecker
2018-08-21 18:33 ` Kevin Buettner
2018-08-21 19:02 ` Simon Marchi
2018-08-21 20:17   ` Simon Marchi
2018-08-21 20:54     ` Joel Brobecker
2018-08-22 16:07       ` Simon Marchi
2018-08-22 17:38         ` Simon Marchi
2018-08-22 18:03         ` Stan Cox
2018-08-22 18:09           ` Simon Marchi
2018-08-21 20:55 ` Joel Brobecker
2018-08-21 21:29   ` Simon Marchi
2018-08-23 17:56     ` Pedro Alves
2018-08-24 18:01       ` Pedro Alves
2018-08-23 18:37 ` Simon Marchi
2018-08-23 22:41   ` Kevin Buettner
2018-08-24 18:35     ` Pedro Alves [this message]
2018-08-24 19:15       ` Simon Marchi
2018-08-24 19:52         ` Pedro Alves
2018-08-24  3:57 ` Kevin Buettner
2018-08-25  5:41   ` Kevin Buettner

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=a93d316e-4d0f-bf42-e5e6-48342a380dc1@redhat.com \
    --to=palves@redhat.com \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=kevinb@redhat.com \
    --cc=simon.marchi@polymtl.ca \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).