From: cagney Date: Sat, 6 Mar 2004 00:10:05 +0000 (+0000) Subject: 2004-03-05 Andrew Cagney X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=9844055a27ceaa8b63df772cc51dffea988e36a6;p=pf3gnuchains%2Fsourceware.git 2004-03-05 Andrew Cagney * infrun.c (step_over_function): When non-legacy code, and no step_frame_id, use the unwinder to get the caller's frame ID. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2b7a298477..7502d1a99f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2004-03-05 Andrew Cagney + + * infrun.c (step_over_function): When non-legacy code, and no + step_frame_id, use the unwinder to get the caller's frame ID. + 2004-03-05 Mark Kettenis * i386bsd-tdep.c (_initialize_i386bsd_tdep): Register diff --git a/gdb/infrun.c b/gdb/infrun.c index 6bbee5e817..d3a6eba2cb 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -2930,6 +2930,7 @@ static void step_over_function (struct execution_control_state *ecs) { struct symtab_and_line sr_sal; + struct frame_id sr_id; init_sal (&sr_sal); /* initialize to zeros */ @@ -2973,13 +2974,29 @@ step_over_function (struct execution_control_state *ecs) sr_sal.section = find_pc_overlay (sr_sal.pc); check_for_old_step_resume_breakpoint (); - step_resume_breakpoint = - set_momentary_breakpoint (sr_sal, get_frame_id (get_current_frame ()), - bp_step_resume); if (frame_id_p (step_frame_id) && !IN_SOLIB_DYNSYM_RESOLVE_CODE (sr_sal.pc)) - step_resume_breakpoint->frame_id = step_frame_id; + /* NOTE: cagney/2004-02-27: Use the global state's idea of the + stepping frame ID. I suspect this is done as it is lighter + weight than a call to get_prev_frame. */ + sr_id = step_frame_id; + else if (legacy_frame_p (current_gdbarch)) + /* NOTE: cagney/2004-02-27: This is the way it was 'cos this is + the way it always was. It should be using the unwound (or + caller's) ID, and not this (or the callee's) ID. It appeared + to work because: legacy architectures used the wrong end of the + frame for the ID.stack (inner-most rather than outer-most) so + that the callee's id.stack (un adjusted) matched the caller's + id.stack giving the "correct" id; more often than not + !IN_SOLIB_DYNSYM_RESOLVE_CODE and hence the code above (it was + originally later in the function) fixed the ID by using global + state. */ + sr_id = get_frame_id (get_current_frame ()); + else + sr_id = get_frame_id (get_prev_frame (get_current_frame ())); + + step_resume_breakpoint = set_momentary_breakpoint (sr_sal, sr_id, bp_step_resume); if (breakpoints_inserted) insert_breakpoints ();