Re: [PATCH] DWARF5 DW_FORM_data16 support

2016-10-31 Thread Jason Merrill
On Mon, Oct 31, 2016 at 12:44 PM, Jakub Jelinek wrote: > On Mon, Oct 31, 2016 at 10:15:11AM -0400, Jason Merrill wrote: >> On 10/17/2016 05:39 PM, Jakub Jelinek wrote: >> >(dwarf_version >= 5 ? 128 : 64) >> >> Please make this a macro. OK with that change. > > DWARF_LARGEST_DATA_FORM_BITS ? Or d

Re: [PATCH] DWARF5 DW_FORM_data16 support

2016-10-31 Thread Jakub Jelinek
On Mon, Oct 31, 2016 at 10:15:11AM -0400, Jason Merrill wrote: > On 10/17/2016 05:39 PM, Jakub Jelinek wrote: > >(dwarf_version >= 5 ? 128 : 64) > > Please make this a macro. OK with that change. DWARF_LARGEST_DATA_FORM_BITS ? Or do you have better name? Jakub

Re: [PATCH] DWARF5 DW_FORM_data16 support

2016-10-31 Thread Jason Merrill
On 10/17/2016 05:39 PM, Jakub Jelinek wrote: (dwarf_version >= 5 ? 128 : 64) Please make this a macro. OK with that change. Jason

[PATCH] DWARF5 DW_FORM_data16 support

2016-10-17 Thread Jakub Jelinek
Hi! DWARF5 has a new 128-bit constant class form, this patch uses it e.g. for __int128 values. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-10-17 Jakub Jelinek * dwarf2out.c (size_of_die, value_format, output_die): Use DW_FORM_data16 for 128-bit d