[RFA] Support for x86 on-stack trampolines

Pedro Alves pedro@codesourcery.com
Wed May 4 08:55:00 GMT 2011


One think I noted on a quick skim over the patch:

On Wednesday 04 May 2011 01:20:24, Jerome Guitton wrote:
> +static const struct frame_unwind i386_stack_tramp_frame_unwind =
> +{
> +  NORMAL_FRAME,
> +  default_frame_unwind_stop_reason,

default_frame_unwind_stop_reason is only used by 
archs/targets that don't support tracepoints at the moment.
It shouldn't be used in any x86-specific unwinder.

If you reuse the this _frame_this_id method of the
epilogue unwinder, you should reuse the _unwind_stop_reason
method of the same unwinder, since you're effectively
also reusing i386_epilogue_frame_cache.

> +  i386_epilogue_frame_this_id,
> +  i386_frame_prev_register,
> +  NULL, 
> +  i386_stack_tramp_frame_sniffer
> +};
> +


-- 
Pedro Alves



More information about the Gdb-patches mailing list