Re: [patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-11-18 Thread Richard Henderson
On 09/30/2014 12:47 PM, Olivier Hainque wrote: 2014-09-30 Olivier Hainque hain...@adacore.com libgcc/ * unwind-dw2.c (DWARF_REG_TO_UNWIND_COLUMN): Move default def to ... gcc/ * defaults.h: ... here. * dwarf2cfi.c (init_one_dwarf_reg_size): New

Re: [patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-11-18 Thread Olivier Hainque
On Nov 18, 2014, at 04:01 , Jeff Law l...@redhat.com wrote: Best for Jason, Richard or Jakub. My knowledge of dwarf2 and our implementation in dwarf*out.c is minimal at best. Thanks for your answer Jeff. Richard Jason have provided feedback (thanks for this as well :) on which I'll

Re: [patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-11-17 Thread David Edelsohn
Olivier, The patch is okay with me, but I cannot approve it. Maybe Jason, Richard or Jeff can take a look. https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01369.html https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02625.html Thanks, David

Re: [patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-11-17 Thread Olivier Hainque
Hi David, Thanks for your support :) Note that the DWARF_REG_TO_UNWIND_COLUMN part is essentially a noop today and is not necessary to fix the breakage. It's just something that ISTM should be there in principle. Olivier On Nov 17, 2014, at 16:56 , David Edelsohn dje@gmail.com wrote:

Re: [patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-11-17 Thread Jeff Law
On 11/17/14 08:56, David Edelsohn wrote: Olivier, The patch is okay with me, but I cannot approve it. Maybe Jason, Richard or Jeff can take a look. https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01369.html https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02625.html Best for Jason, Richard or

[patch] fix expand_builtin_init_dwarf_reg_sizes wrt register spans

2014-09-30 Thread Olivier Hainque
Hello, Exception propagation has been failing for a while for the SPE/e500 family of powerpc targets. The issue boils down to an assert failure through: uw_init_context_1 () ... _Unwind_SetSpColumn (context, outer_cfa, sp_slot); then _Unwind_SetSpColumn () ... int size