Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-11-01 Thread Jason Merrill
On Tue, Nov 1, 2016 at 2:04 PM, Jason Merrill wrote: > On Tue, Nov 1, 2016 at 9:56 AM, Jakub Jelinek wrote: >> On Mon, Oct 31, 2016 at 09:56:28AM -0400, Jason Merrill wrote: >>> >Or by changing get_qualified_die (in particular check_base_type) to use a >>> >langhook, we could at least for DW_AT_{

Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-11-01 Thread Jason Merrill
On Tue, Nov 1, 2016 at 9:56 AM, Jakub Jelinek wrote: > On Mon, Oct 31, 2016 at 09:56:28AM -0400, Jason Merrill wrote: >> >Or by changing get_qualified_die (in particular check_base_type) to use a >> >langhook, we could at least for DW_AT_{reference,rvalue_reference} just use >> >equate_type_number

Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-11-01 Thread Jakub Jelinek
On Mon, Oct 31, 2016 at 09:56:28AM -0400, Jason Merrill wrote: > >Or by changing get_qualified_die (in particular check_base_type) to use a > >langhook, we could at least for DW_AT_{reference,rvalue_reference} just use > >equate_type_number_to_die/lookup_type_die reliably (DW_AT_endianity issue > >

Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-10-31 Thread Jason Merrill
On 10/24/2016 03:09 PM, Jakub Jelinek wrote: On Mon, Oct 24, 2016 at 02:34:04PM -0400, Jason Merrill wrote: On Mon, Oct 24, 2016 at 10:29 AM, Jakub Jelinek wrote: This is another addition in DWARF5. The patch emits these attributes only for DW_TAG_subprogram for non-static ref-qualified membe

Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-10-24 Thread Jakub Jelinek
On Mon, Oct 24, 2016 at 02:34:04PM -0400, Jason Merrill wrote: > On Mon, Oct 24, 2016 at 10:29 AM, Jakub Jelinek wrote: > > This is another addition in DWARF5. The patch emits these attributes > > only for DW_TAG_subprogram for non-static ref-qualified member functions. > > Bootstrapped/regtested

Re: [PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-10-24 Thread Jason Merrill
On Mon, Oct 24, 2016 at 10:29 AM, Jakub Jelinek wrote: > This is another addition in DWARF5. The patch emits these attributes > only for DW_TAG_subprogram for non-static ref-qualified member functions. > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK. > We really should

[PATCH] Emit DWARF5 DW_AT_reference and DW_AT_rvalue_reference

2016-10-24 Thread Jakub Jelinek
Hi! This is another addition in DWARF5. The patch emits these attributes only for DW_TAG_subprogram for non-static ref-qualified member functions. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? We really should emit it also for DW_TAG_subroutine_type for PMF types, the prob