diff options
author | Andrew Cagney <cagney@redhat.com> | 2003-04-11 03:12:58 +0000 |
---|---|---|
committer | Andrew Cagney <cagney@redhat.com> | 2003-04-11 03:12:58 +0000 |
commit | 81f8a2067a878ff7d4dc2ddac998e34523827fd6 (patch) | |
tree | 37fc8d5cd9ffb0f07897804ae2921a52a8d91440 /gdb | |
parent | 05760fd2f6b41ad63d9278f2d50a385939a2acdd (diff) | |
download | gdb-81f8a2067a878ff7d4dc2ddac998e34523827fd6.zip gdb-81f8a2067a878ff7d4dc2ddac998e34523827fd6.tar.gz gdb-81f8a2067a878ff7d4dc2ddac998e34523827fd6.tar.bz2 |
2003-04-10 Andrew Cagney <cagney@redhat.com>
* d10v-tdep.c (d10v_frame_this_id): Set the code addr to the
frame's function's address. Simplify.
(d10v_frame_unwind_cache): Check that the frame's function is
non-zero.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 7 | ||||
-rw-r--r-- | gdb/d10v-tdep.c | 18 |
2 files changed, 17 insertions, 8 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b83a331..f409973 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2003-04-10 Andrew Cagney <cagney@redhat.com> + + * d10v-tdep.c (d10v_frame_this_id): Set the code addr to the + frame's function's address. Simplify. + (d10v_frame_unwind_cache): Check that the frame's function is + non-zero. + 2003-04-10 Jim Blandy <jimb@redhat.com> * s390-tdep.c (s390_gdbarch_init): Put back accidentally deleted diff --git a/gdb/d10v-tdep.c b/gdb/d10v-tdep.c index 4ae8099..45ac682 100644 --- a/gdb/d10v-tdep.c +++ b/gdb/d10v-tdep.c @@ -710,7 +710,7 @@ d10v_frame_unwind_cache (struct frame_info *next_frame, info->uses_frame = 0; for (pc = frame_func_unwind (next_frame); - pc < frame_pc_unwind (next_frame); + pc > 0 && pc < frame_pc_unwind (next_frame); pc += 4) { op = (unsigned long) read_memory_integer (pc, 4); @@ -1440,14 +1440,15 @@ d10v_frame_this_id (struct frame_info *next_frame, struct d10v_unwind_cache *info = d10v_frame_unwind_cache (next_frame, this_prologue_cache); CORE_ADDR base; - CORE_ADDR pc; + CORE_ADDR func; + struct frame_id id; - /* The PC is easy. */ - pc = frame_pc_unwind (next_frame); + /* The FUNC is easy. */ + func = frame_func_unwind (next_frame); /* This is meant to halt the backtrace at "_start". Make sure we don't halt it at a generic dummy frame. */ - if (pc == IMEM_START || pc <= IMEM_START || inside_entry_file (pc)) + if (func <= IMEM_START || inside_entry_file (func)) return; /* Hopefully the prologue analysis either correctly determined the @@ -1457,17 +1458,18 @@ d10v_frame_this_id (struct frame_info *next_frame, if (base == STACK_START || base == 0) return; + id = frame_id_build (base, func); + /* Check that we're not going round in circles with the same frame ID (but avoid applying the test to sentinel frames which do go round in circles). Can't use frame_id_eq() as that doesn't yet compare the frame's PC value. */ if (frame_relative_level (next_frame) >= 0 && get_frame_type (next_frame) != DUMMY_FRAME - && get_frame_pc (next_frame) == pc - && get_frame_base (next_frame) == base) + && frame_id_eq (get_frame_id (next_frame), id)) return; - (*this_id) = frame_id_build (base, pc); + (*this_id) = id; } static void |