public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Jakub Jelinek <jakub@redhat.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: BoF DWARF5 patches (25% .debug section size reduction)
Date: Tue, 29 Sep 2020 15:56:04 +0200	[thread overview]
Message-ID: <617961ca9e9552071f821aec72b0f12eb2e5c15f.camel@klomp.org> (raw)
In-Reply-To: <20200910111657.GY18149@tucnak>

[-- Attachment #1: Type: text/plain, Size: 739 bytes --]

On Thu, 2020-09-10 at 13:16 +0200, Jakub Jelinek wrote:
> On Wed, Sep 09, 2020 at 09:57:54PM +0200, Mark Wielaard wrote:
> > --- a/gcc/doc/invoke.texi
> > +++ b/gcc/doc/invoke.texi
> > @@ -9057,13 +9057,14 @@ possible.
> >  @opindex gdwarf
> >  Produce debugging information in DWARF format (if that is supported).
> >  The value of @var{version} may be either 2, 3, 4 or 5; the default version
> > -for most targets is 4.  DWARF Version 5 is only experimental.
> > +for most targets is 5 (with the exception of vxworks and darwin which
> > +default to version 2).
> 
> I think in documentation we should spell these VxWorks and Darwin/Mac OS X

OK. As attached.

Are we ready to flip the default to 5?

Thanks,

Mark

[-- Attachment #2: 0001-Default-to-DWARF5.patch --]
[-- Type: text/x-patch, Size: 1940 bytes --]

From 409bd1b2c60905b0f96c94fface12154d3be4d32 Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Tue, 29 Sep 2020 15:52:44 +0200
Subject: [PATCH] Default to DWARF5

gcc/ChangeLog:

	* common.opt (gdwarf-): Init(5).
	* doc/invoke.texi (-gdwarf): Document default to 5.
---
 gcc/common.opt      | 2 +-
 gcc/doc/invoke.texi | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/gcc/common.opt b/gcc/common.opt
index 292c2de694ef..d1722de80bf0 100644
--- a/gcc/common.opt
+++ b/gcc/common.opt
@@ -3148,7 +3148,7 @@ Common Driver JoinedOrMissing Negative(gdwarf-)
 Generate debug information in default version of DWARF format.
 
 gdwarf-
-Common Driver Joined UInteger Var(dwarf_version) Init(4) Negative(gstabs)
+Common Driver Joined UInteger Var(dwarf_version) Init(5) Negative(gstabs)
 Generate debug information in DWARF v2 (or later) format.
 
 ggdb
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 2091e0cd23b9..e6453374bcd4 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -9210,14 +9210,15 @@ possible.
 @itemx -gdwarf-@var{version}
 @opindex gdwarf
 Produce debugging information in DWARF format (if that is supported).
-The value of @var{version} may be either 2, 3, 4 or 5; the default version
-for most targets is 4.  DWARF Version 5 is only experimental.
+The value of @var{version} may be either 2, 3, 4 or 5; the default
+version for most targets is 5 (with the exception of VxWorks and
+Darwin/Mac OS X which default to version 2).
 
 Note that with DWARF Version 2, some ports require and always
 use some non-conflicting DWARF 3 extensions in the unwind tables.
 
 Version 4 may require GDB 7.0 and @option{-fvar-tracking-assignments}
-for maximum benefit.
+for maximum benefit. Version 5 requires GDB 8.0 or higher.
 
 GCC no longer supports DWARF Version 1, which is substantially
 different than Version 2 and later.  For historical reasons, some
-- 
2.18.4


  parent reply	other threads:[~2020-09-29 13:56 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-24 12:56 BoF DWARF5 patches Mark Wielaard
2020-08-24 12:56 ` [PATCH 1/5] Don't enable -gvariable-location-views by default for DWARF5 Mark Wielaard
2020-08-24 17:38   ` Jakub Jelinek
2020-08-24 20:12     ` Mark Wielaard
2020-08-25  4:05     ` Alexandre Oliva
2020-08-25  7:27       ` Richard Biener
2020-08-25  9:24       ` Mark Wielaard
2020-08-26 12:22         ` Alexandre Oliva
2020-09-29 11:15     ` Mark Wielaard
2020-08-24 12:56 ` [PATCH 2/5] Make sure that static data member constexpr isn't optimized away in test Mark Wielaard
2020-08-24 17:40   ` Jakub Jelinek
2020-08-24 20:17     ` Mark Wielaard
2020-08-24 20:59       ` Tom Tromey
2020-08-24 21:38     ` Jason Merrill
2020-08-25  9:19       ` Mark Wielaard
2020-09-01 18:46         ` Jason Merrill
2021-02-09 19:40           ` Jason Merrill
2021-02-09 19:55             ` Jakub Jelinek
2021-02-09 23:15               ` Jakub Jelinek
2020-08-25 12:41       ` Jakub Jelinek
2020-08-24 12:56 ` [PATCH 3/5] Add DWARF5 variants of assembly scan tests that use DW_FORM_implicit_const Mark Wielaard
2020-08-24 17:44   ` Jakub Jelinek
2020-08-24 20:26     ` Mark Wielaard
2020-09-17 16:03       ` Mark Wielaard
2020-09-17 16:45         ` Jakub Jelinek
2020-08-24 12:56 ` [PATCH 4/5] Default to DWARF5 Mark Wielaard
2020-08-24 12:56 ` [PATCH 5/5] Add --gdwarf-5 to ASM_SPEC Mark Wielaard
2020-08-26 21:37   ` Duplicate .debug_lines (Was: [PATCH 5/5] Add --gdwarf-5 to ASM_SPEC) Mark Wielaard
2020-08-26 23:38     ` H.J. Lu
2020-08-29 12:23       ` Mark Wielaard
2020-08-29 14:34         ` H.J. Lu
2020-08-29 15:23           ` Mark Wielaard
2020-08-29 15:43             ` H.J. Lu
2020-08-29 16:32               ` Mark Wielaard
2020-08-29 16:44                 ` H.J. Lu
2020-08-29 17:32                   ` Mark Wielaard
2020-09-07 12:37     ` [PATCH] gas: Don't error when .debug_line already exists, unless .loc was used Mark Wielaard
2020-08-25  9:32 ` BoF DWARF5 patches Mark Wielaard
2020-09-09 19:57   ` BoF DWARF5 patches (25% .debug section size reduction) Mark Wielaard
2020-09-10 11:16     ` Jakub Jelinek
2020-09-10 11:45       ` Jakub Jelinek
2020-09-15 18:40         ` [PATCH] debug: Pass --gdwarf-N to assembler if fixed gas is detected during configure Jakub Jelinek
2020-09-16 12:33           ` Mark Wielaard
2020-09-16 13:31             ` Jakub Jelinek
2020-09-18 15:21           ` Mark Wielaard
2020-10-06 15:54             ` Mark Wielaard
2020-10-06 20:57               ` Jason Merrill
2020-10-07 11:29                 ` Mark Wielaard
2020-09-29 13:56       ` Mark Wielaard [this message]
2020-11-15 22:41         ` BoF DWARF5 patches (25% .debug section size reduction) Mark Wielaard
2021-01-15 16:16           ` Jakub Jelinek
2021-01-18 10:19             ` Sebastian Huber
2021-01-18 11:18             ` Mark Wielaard
2020-11-17  0:19         ` Jeff Law

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=617961ca9e9552071f821aec72b0f12eb2e5c15f.camel@klomp.org \
    --to=mark@klomp.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).