From: Andrew Cagney Date: Thu, 16 Oct 2003 18:24:13 +0000 (+0000) Subject: 2003-10-16 Andrew Cagney X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=d631a80438c01b3084c6afa05ad0ea6e3d0c196d;p=pf3gnuchains%2Fpf3gnuchains3x.git 2003-10-16 Andrew Cagney * infrun.c (handle_inferior_event): Add comment about "frame_id_inner" being too weak. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 3834c5481e..654f63f5d9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2003-10-16 Andrew Cagney + + * infrun.c (handle_inferior_event): Add comment about + "frame_id_inner" being too weak. + 2003-10-16 Elena Zannoni * minsyms.c (lookup_minimal_symbol_solib_trampoline): Remove diff --git a/gdb/infrun.c b/gdb/infrun.c index e22a90a926..7412080ba3 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2636,6 +2636,22 @@ process_event_stop_test: /* In the case where we just stepped out of a function into the middle of a line of the caller, continue stepping, but step_frame_id must be modified to current frame */ +#if 0 + /* NOTE: cagney/2003-10-16: I think this frame ID inner test is too + generous. It will trigger on things like a step into a frameless + stackless leaf function. I think the logic should instead look + at the unwound frame ID has that should give a more robust + indication of what happened. */ + if (step-ID == current-ID) + still stepping in same function; + else if (step-ID == unwind (current-ID)) + stepped into a function; + else + stepped out of a function; + /* Of course this assumes that the frame ID unwind code is robust + and we're willing to introduce frame unwind logic into this + function. Fortunatly, those days are nearly upon us. */ +#endif { struct frame_id current_frame = get_frame_id (get_current_frame ()); if (!(frame_id_inner (current_frame, step_frame_id)))