Re: [PATCH v1] [include] Add codes for DWARF v5 .dwp sections to dwarf2.h

2020-09-09 Thread Joseph Myers
On Wed, 9 Sep 2020, Caroline Tice via Gcc-patches wrote:

> For DWARF v5 Dwarf Package Files (.dwp files), the section identifier
> encodings have changed. This patch updates dwarf2.h to contain the new
> encodings.  (see http://dwarfstd.org/doc/DWARF5.pdf, section 7.3.5).
> 
> This patch has already been committed in binutils, but it needs to go into GCC
> as well to avoid the binutils patch being overwritten/lost.
> 
> I tested this by running the regression testsuite; there were no regressions.
> 
> Is this ok to commit?

In my view, anyone with write access should feel free to merge changes to 
shared files from the binutils-gdb tree at any time, without needing 
separate approval.

The only exception would be if there is deliberate divergence.  The only 
case I know of for deliberate divergence is to allow autotools versions to 
be updated separately in the two trees, since even updating one tree is 
quite involved (the last time that happened, it was a while between when 
Simon did the update in binutils-gdb and when I did the corresponding 
update for GCC; between the updates, care was needed about merging changes 
to auto*-related files).

-- 
Joseph S. Myers
jos...@codesourcery.com


[PATCH v1] [include] Add codes for DWARF v5 .dwp sections to dwarf2.h

2020-09-09 Thread Caroline Tice via Gcc-patches
For DWARF v5 Dwarf Package Files (.dwp files), the section identifier
encodings have changed. This patch updates dwarf2.h to contain the new
encodings.  (see http://dwarfstd.org/doc/DWARF5.pdf, section 7.3.5).

This patch has already been committed in binutils, but it needs to go into GCC
as well to avoid the binutils patch being overwritten/lost.

I tested this by running the regression testsuite; there were no regressions.

Is this ok to commit?

-- Caroline Tice
cmt...@google.com

include/ChangeLog

2020-09-09  Caroline Tice  

* dwarf2.h (enum dwarf_sect_v5): A new enum section for the
sections in a DWARF 5 DWP file (DWP version 5).


v1-0001-Add-codes-for-DWARF-v5-.dwp-sections-to-dwarf2.h.gcc.patch
Description: Binary data