* [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8}
@ 2016-12-20 18:59 Jakub Jelinek
2016-12-21 23:47 ` Jason Merrill
2017-01-01 22:26 ` Jeff Law
0 siblings, 2 replies; 3+ messages in thread
From: Jakub Jelinek @ 2016-12-20 18:59 UTC (permalink / raw)
To: Jason Merrill; +Cc: gcc-patches, Mark Wielaard
Hi!
Recently DW_FORM_ref_sup (which is meant e.g. for dwz, gcc doesn't emit it)
has been renamed to DW_FORM_ref_sup4 (and changed so that it is always 4
byte) and DW_FORM_ref_sup8 (always 8 byte) has been added.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2016-12-20 Jakub Jelinek <jakub@redhat.com>
* dwarf2.def (DW_FORM_ref_sup): Renamed to ...
(DW_FORM_ref_sup4): ... this. New form.
(DW_FORM_ref_sup8): New form.
--- include/dwarf2.def.jj 2016-10-31 13:28:05.000000000 +0100
+++ include/dwarf2.def 2016-12-19 11:40:07.303525953 +0100
@@ -212,13 +212,14 @@ DW_FORM (DW_FORM_ref_sig8, 0x20)
/* DWARF 5. */
DW_FORM (DW_FORM_strx, 0x1a)
DW_FORM (DW_FORM_addrx, 0x1b)
-DW_FORM (DW_FORM_ref_sup, 0x1c)
+DW_FORM (DW_FORM_ref_sup4, 0x1c)
DW_FORM (DW_FORM_strp_sup, 0x1d)
DW_FORM (DW_FORM_data16, 0x1e)
DW_FORM (DW_FORM_line_strp, 0x1f)
DW_FORM (DW_FORM_implicit_const, 0x21)
DW_FORM (DW_FORM_loclistx, 0x22)
DW_FORM (DW_FORM_rnglistx, 0x23)
+DW_FORM (DW_FORM_ref_sup8, 0x24)
/* Extensions for Fission. See http://gcc.gnu.org/wiki/DebugFission. */
DW_FORM (DW_FORM_GNU_addr_index, 0x1f01)
DW_FORM (DW_FORM_GNU_str_index, 0x1f02)
Jakub
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8}
2016-12-20 18:59 [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8} Jakub Jelinek
@ 2016-12-21 23:47 ` Jason Merrill
2017-01-01 22:26 ` Jeff Law
1 sibling, 0 replies; 3+ messages in thread
From: Jason Merrill @ 2016-12-21 23:47 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: gcc-patches List, Mark Wielaard
OK.
On Tue, Dec 20, 2016 at 1:57 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> Hi!
>
> Recently DW_FORM_ref_sup (which is meant e.g. for dwz, gcc doesn't emit it)
> has been renamed to DW_FORM_ref_sup4 (and changed so that it is always 4
> byte) and DW_FORM_ref_sup8 (always 8 byte) has been added.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
>
> 2016-12-20 Jakub Jelinek <jakub@redhat.com>
>
> * dwarf2.def (DW_FORM_ref_sup): Renamed to ...
> (DW_FORM_ref_sup4): ... this. New form.
> (DW_FORM_ref_sup8): New form.
>
> --- include/dwarf2.def.jj 2016-10-31 13:28:05.000000000 +0100
> +++ include/dwarf2.def 2016-12-19 11:40:07.303525953 +0100
> @@ -212,13 +212,14 @@ DW_FORM (DW_FORM_ref_sig8, 0x20)
> /* DWARF 5. */
> DW_FORM (DW_FORM_strx, 0x1a)
> DW_FORM (DW_FORM_addrx, 0x1b)
> -DW_FORM (DW_FORM_ref_sup, 0x1c)
> +DW_FORM (DW_FORM_ref_sup4, 0x1c)
> DW_FORM (DW_FORM_strp_sup, 0x1d)
> DW_FORM (DW_FORM_data16, 0x1e)
> DW_FORM (DW_FORM_line_strp, 0x1f)
> DW_FORM (DW_FORM_implicit_const, 0x21)
> DW_FORM (DW_FORM_loclistx, 0x22)
> DW_FORM (DW_FORM_rnglistx, 0x23)
> +DW_FORM (DW_FORM_ref_sup8, 0x24)
> /* Extensions for Fission. See http://gcc.gnu.org/wiki/DebugFission. */
> DW_FORM (DW_FORM_GNU_addr_index, 0x1f01)
> DW_FORM (DW_FORM_GNU_str_index, 0x1f02)
>
> Jakub
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8}
2016-12-20 18:59 [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8} Jakub Jelinek
2016-12-21 23:47 ` Jason Merrill
@ 2017-01-01 22:26 ` Jeff Law
1 sibling, 0 replies; 3+ messages in thread
From: Jeff Law @ 2017-01-01 22:26 UTC (permalink / raw)
To: Jakub Jelinek, Jason Merrill; +Cc: gcc-patches, Mark Wielaard
On 12/20/2016 11:57 AM, Jakub Jelinek wrote:
> Hi!
>
> Recently DW_FORM_ref_sup (which is meant e.g. for dwz, gcc doesn't emit it)
> has been renamed to DW_FORM_ref_sup4 (and changed so that it is always 4
> byte) and DW_FORM_ref_sup8 (always 8 byte) has been added.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
>
> 2016-12-20 Jakub Jelinek <jakub@redhat.com>
>
> * dwarf2.def (DW_FORM_ref_sup): Renamed to ...
> (DW_FORM_ref_sup4): ... this. New form.
> (DW_FORM_ref_sup8): New form.
OK.
jeff
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-01-01 22:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-20 18:59 [PATCH] Replace DW_FORM_ref_sup with DW_FORM_ref_sup{4,8} Jakub Jelinek
2016-12-21 23:47 ` Jason Merrill
2017-01-01 22:26 ` Jeff Law
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).