Re: [debug-early] emit early dwarf for locally scoped functions

2015-03-26 Thread Aldy Hernandez
On 03/25/2015 07:07 PM, Jason Merrill wrote: On 03/25/2015 05:05 PM, Aldy Hernandez wrote: Or we could cheat and just remove them as mainline does, but only when reusing a declaration (as in the attached patch). This seems right to me. Jason Ok thanks, committed. Aldy

Re: [debug-early] emit early dwarf for locally scoped functions

2015-03-25 Thread Jason Merrill
On 03/25/2015 05:05 PM, Aldy Hernandez wrote: Or we could cheat and just remove them as mainline does, but only when reusing a declaration (as in the attached patch). This seems right to me. Jason

Re: [debug-early] emit early dwarf for locally scoped functions

2015-03-25 Thread Aldy Hernandez
On 03/25/2015 12:37 PM, Jason Merrill wrote: On 03/24/2015 02:00 PM, Aldy Hernandez wrote: I found that for locally scoped functions we were not emitting early dwarf. Why weren't they being emitted as part of their enclosing function? They should be. Jason Hmm, you're right. Sorry for bei

Re: [debug-early] emit early dwarf for locally scoped functions

2015-03-25 Thread Jason Merrill
On 03/24/2015 02:00 PM, Aldy Hernandez wrote: I found that for locally scoped functions we were not emitting early dwarf. Why weren't they being emitted as part of their enclosing function? They should be. Jason