Re: [PR63238] output alignment debug information

2017-03-16 Thread Cary Coutant
>> This is OK so far, but the DW_AT_alignment attribute also needs to be >> added to the checksum computation in die_checksum and >> die_checksum_ordered. > > Thanks. I see what to do in die_checksum_ordered, but die_checksum? It > seems to handle attributes by value class, and AFAICT the

Re: [PR63238] output alignment debug information

2017-02-01 Thread Jakub Jelinek
On Fri, Jan 27, 2017 at 04:24:58AM -0200, Alexandre Oliva wrote: > Output DWARFv5+ DW_AT_alignment for non-default alignment of > variables, fields and types. The new tests all fail on targets that default to -gstrict-dwarf because they have buggy or prehistoric linkers/debug info consumers like

Re: [PR63238] output alignment debug information

2017-01-30 Thread Alexandre Oliva
On Jan 29, 2017, Cary Coutant wrote: >> for gcc/ChangeLog >> >> PR debug/63238 > This is OK so far, but the DW_AT_alignment attribute also needs to be > added to the checksum computation in die_checksum and > die_checksum_ordered. Thanks. I see what to do in

Re: [PR63238] output alignment debug information

2017-01-29 Thread Cary Coutant
> for gcc/ChangeLog > > PR debug/63238 > * dwarf2out.c (clone_as_declaration): Drop DW_AT_alignment. > (add_alignment_attribute): New. > (base_type_die): Add alignment attribute. > (subrange_type_die): Likewise. > (modified_type_die): Likewise. >

Re: [PR63238] output alignment debug information

2017-01-26 Thread Alexandre Oliva
t of >>> variables, fields and types. >>> Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? >> Ping? > Ping? (conflicts resolved, patch refreshed and retested) Ping? (refreshed, not sure there were conflicts to resolve) [PR63238] output alignment debug informa

Re: [PR63238] output alignment debug information

2016-10-19 Thread Alexandre Oliva
On Sep 23, 2016, Alexandre Oliva wrote: > On Aug 27, 2016, Alexandre Oliva wrote: >> Output DWARFv5+ DW_AT_alignment for non-default alignment of >> variables, fields and types. >> Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? >

Re: [PR63238] output alignment debug information

2016-09-23 Thread Alexandre Oliva
On Aug 27, 2016, Alexandre Oliva wrote: > Output DWARFv5+ DW_AT_alignment for non-default alignment of > variables, fields and types. > Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? Ping? https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01911.html --

[PR63238] output alignment debug information

2016-08-26 Thread Alexandre Oliva
Output DWARFv5+ DW_AT_alignment for non-default alignment of variables, fields and types. Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? for include/ChangeLog PR debug/63238 * dwarf2.def (DW_AT_alignment): New. for gcc/ChangeLog PR debug/63238