Re: [PATCH] add 'rs6000_in_function_epilogue_p()'

Jim Blandy
2005-12-02T01:07:42Z

Paul wanted to fast-track this patch, in hopes it could get into the
6.4 release. Joel, what are your thoughts?

I think the best alternative to deprecated_read_memory_nobpt would be
to actually call current_frame (this will be cheap, since we just
called it in watchpoint_check, before calling
gdbarch_in_function_epilogue_p) and then call
safe_frame_unwind_memory.

Whatever we do immediately, the underlying cause here is that
unwinding isn't working at every instruction. If it were, then the
call to frame_find_by_id would find the frame containing the watched
variable, and watchpoint_check would correctly conclude that the
variable was still in scope.

Paul, does your test program have Dwarf CFI for the functions in question?