Re: [PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2015-04-14 Thread Christian Bruel
committed, thanks sorry for the delay. Christian On 10/14/2014 08:25 PM, Richard Henderson wrote: > On 10/14/2014 06:02 AM, Christian Bruel wrote: >> 2014-09-23 Christian Bruel >> >> * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each >> function. > > It's tempting

Re: [PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Richard Henderson
On 10/14/2014 11:25 AM, Richard Henderson wrote: > On 10/14/2014 06:02 AM, Christian Bruel wrote: >> 2014-09-23 Christian Bruel >> >> * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each >> function. > > It's tempting to make this a local variable within dwarf2out_frame

Re: [PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Richard Henderson
On 10/14/2014 06:02 AM, Christian Bruel wrote: > 2014-09-23 Christian Bruel > > * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each > function. It's tempting to make this a local variable within dwarf2out_frame_debug_expr and not try to cache it at all. But this is

[PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Christian Bruel
Hello, ARM and Thumb modes use different hard_frame_pointer_regnum ABIs. The problem is that dwarf2cfi.c:dw_frame_pointer_regnum cache is initialized only once per file, when creating the CIE. While testing the ARM attribute target to switch modes between functions, I got a few assertion with -g,