aboutsummaryrefslogtreecommitdiff
path: root/gdb/frame.c
diff options
context:
space:
mode:
authorAndrew Cagney <cagney@redhat.com>2003-09-12 18:40:18 +0000
committerAndrew Cagney <cagney@redhat.com>2003-09-12 18:40:18 +0000
commit627b3ba2d386befaf2d1bd97bd780ac1218dd399 (patch)
tree506fc255ee49268770bd8a356538f907eec76279 /gdb/frame.c
parentb0abbc58f0daefaf286d6a008820fef5f69e5353 (diff)
downloadgdb-627b3ba2d386befaf2d1bd97bd780ac1218dd399.zip
gdb-627b3ba2d386befaf2d1bd97bd780ac1218dd399.tar.gz
gdb-627b3ba2d386befaf2d1bd97bd780ac1218dd399.tar.bz2
2003-09-12 Andrew Cagney <cagney@redhat.com>
* objfiles.h (struct entry_info): Deprecate "entry_file_lowpc" and "entry_file_highpc". Update comments. * defs.h (deprecated_inside_entry_file): Rename "inside_entry_file". * blockframe.c (deprecated_inside_entry_file): Rename "inside_entry_file". * frame.c (get_prev_frame): Update. Use if 0 instead of #if 0. * vax-tdep.c (vax_frame_chain): Update. * sh64-tdep.c (sh64_frame_chain): Update. * sh-tdep.c (sh_frame_chain): Update. * rs6000-tdep.c (rs6000_frame_chain): Update. * ns32k-tdep.c (ns32k_frame_chain): Update. * mips-tdep.c (mips_frame_chain): Update. * m68hc11-tdep.c (m68hc11_frame_this_id): Update. * m32r-tdep.c (m32r_frame_this_id): Update. * i386-interix-tdep.c (i386_interix_frame_chain_valid): Update. * frv-tdep.c (frv_frame_this_id): Update. * d10v-tdep.c (d10v_frame_this_id): Update. * cris-tdep.c (cris_frame_chain): Update. * blockframe.c (legacy_frame_chain_valid): Update. * avr-tdep.c (avr_frame_this_id): Update. * arm-tdep.c (arm_prologue_this_id): Update. * alpha-tdep.c (alpha_heuristic_frame_this_id): Update. * objfiles.c (objfile_relocate): Update. * mipsread.c (mipscoff_symfile_read): Update. (mipscoff_symfile_read): Update. * mdebugread.c (parse_partial_symbols): Update. * dwarfread.c (read_file_scope): Update. * dwarf2read.c (read_file_scope): Update. * dbxread.c (read_dbx_symtab): Update. (read_dbx_symtab): Update. * coffread.c (complete_symtab): Update.
Diffstat (limited to 'gdb/frame.c')
-rw-r--r--gdb/frame.c31
1 files changed, 16 insertions, 15 deletions
diff --git a/gdb/frame.c b/gdb/frame.c
index 31cb6c5..6e01026 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -1844,8 +1844,9 @@ get_prev_frame (struct frame_info *this_frame)
checking for "main" in the minimal symbols. With that fixed
asm-source tests now stop in "main" instead of halting the
backtrace in wierd and wonderful ways somewhere inside the entry
- file. Suspect that inside_entry_file and inside_entry_func tests
- were added to work around that (now fixed) case. */
+ file. Suspect that deprecated_inside_entry_file and
+ inside_entry_func tests were added to work around that (now
+ fixed) case. */
/* NOTE: cagney/2003-07-15: danielj (if I'm reading it right)
suggested having the inside_entry_func test use the
inside_main_func msymbol trick (along with entry_point_address I
@@ -1884,7 +1885,6 @@ get_prev_frame (struct frame_info *this_frame)
}
this_frame->prev_p = 1;
-#if 0
/* If we're inside the entry file, it isn't valid. Don't apply this
test to a dummy frame - dummy frame PC's typically land in the
entry file. Don't apply this test to the sentinel frame.
@@ -1896,17 +1896,19 @@ get_prev_frame (struct frame_info *this_frame)
/* NOTE: cagney/2003-01-10: If there is a way of disabling this test
then it should probably be moved to before the ->prev_p test,
above. */
- /* NOTE: vinschen/2003-04-01: Disabled. It turns out that the call to
- inside_entry_file destroys a meaningful backtrace under some
- conditions. E. g. the backtrace tests in the asm-source testcase
- are broken for some targets. In this test the functions are all
- implemented as part of one file and the testcase is not necessarily
- linked with a start file (depending on the target). What happens is,
- that the first frame is printed normaly and following frames are
- treated as being inside the enttry file then. This way, only the
- #0 frame is printed in the backtrace output. */
- if (this_frame->type != DUMMY_FRAME && this_frame->level >= 0
- && inside_entry_file (get_frame_pc (this_frame)))
+ /* NOTE: vinschen/2003-04-01: Disabled. It turns out that the call
+ to deprecated_inside_entry_file destroys a meaningful backtrace
+ under some conditions. E. g. the backtrace tests in the
+ asm-source testcase are broken for some targets. In this test
+ the functions are all implemented as part of one file and the
+ testcase is not necessarily linked with a start file (depending
+ on the target). What happens is, that the first frame is printed
+ normaly and following frames are treated as being inside the
+ enttry file then. This way, only the #0 frame is printed in the
+ backtrace output. */
+ if (0
+ && this_frame->type != DUMMY_FRAME && this_frame->level >= 0
+ && deprecated_inside_entry_file (get_frame_pc (this_frame)))
{
if (frame_debug)
{
@@ -1916,7 +1918,6 @@ get_prev_frame (struct frame_info *this_frame)
}
return NULL;
}
-#endif
/* If any of the old frame initialization methods are around, use
the legacy get_prev_frame method. */