On 06/30/2011 10:42 AM, H.J. Lu wrote:
Register may be saved/restored either by address or value. My patch doesn't change the reg field. The other way will be#ifdef USE_UNWIND_WORD _Unwind_Word reg[DWARF_FRAME_REGISTERS+1]; #else void *reg[DWARF_FRAME_REGISTERS+1]; #endif We need it so that we are binary compatible with the existing unwind context. Once we do that we need many #ifdef USE_UNWIND_WORD #endif whenever the reg field is accessed since the reg field is changed.
But your patch already changes all but one place where reg is accessed. And we can avoid lots of ifdefs by abstraction with macros/inlines so there's one interface.
Also, why change SIGNAL_FRAME_BIT? Jason
