Re: [patch] Fix PR debug/59418

2013-12-18 Thread Eric Botcazou
> Is it really a good idea to put the XVECLEN into the loop condition? > I mean, the functions that are called in the loop are unlikely pure > and thus the length will need to be uselessly reread for each iteration. I'm not sure we are supposed to care about this kind of micro-optimization in the

Re: [patch] Fix PR debug/59418

2013-12-17 Thread Jakub Jelinek
On Sun, Dec 15, 2013 at 06:53:07PM +0100, Eric Botcazou wrote: > 2013-12-15 Eric Botcazou > > PR debug/59418 > * dwarf2cfi.c (dwarf2out_frame_debug_cfa_offset): Fix comment and clean > up implementation. > (dwarf2out_frame_debug_cfa_restore): Handle TARGET_DWARF_REGISTE

[patch] Fix PR debug/59418

2013-12-15 Thread Eric Botcazou
Hi, this is a latent bug exposed on the mainline for the ARM by: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01755.html The problem is that the CFI expressions for: (insn/f:TI 102 11 103 (parallel [ (set (mem/c:BLK (pre_modify:SI (reg/f:SI 13 sp) (plus:SI (