From 5a543cf639843b9c317943cd3f846a56b9bf5fb3 Mon Sep 17 00:00:00 2001 From: drow Date: Tue, 28 Jul 2009 16:39:06 +0000 Subject: [PATCH] * arch-utils.c (displaced_step_at_entry_point): Do not call gdbarch_convert_from_func_ptr_addr. * cris-tdep.c: Remove outdated comment. * infcall.c (call_function_by_hand): Do not call gdbarch_convert_from_func_ptr_addr after entry_point_address. * objfiles.c (entry_point_address): Call both gdbarch_convert_from_func_ptr_addr and gdbarch_addr_bits_remove. --- gdb/arch-utils.c | 4 ---- gdb/cris-tdep.c | 1 - gdb/infcall.c | 19 ++++++++----------- gdb/objfiles.c | 21 ++++++++++++++++++++- 4 files changed, 28 insertions(+), 17 deletions(-) diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c index f3246eb0ba..f075922bf0 100644 --- a/gdb/arch-utils.c +++ b/gdb/arch-utils.c @@ -76,10 +76,6 @@ displaced_step_at_entry_point (struct gdbarch *gdbarch) addr = entry_point_address (); - /* Make certain that the address points at real code, and not a - function descriptor. */ - addr = gdbarch_convert_from_func_ptr_addr (gdbarch, addr, ¤t_target); - /* Inferior calls also use the entry point as a breakpoint location. We don't want displaced stepping to interfere with those breakpoints, so leave space. */ diff --git a/gdb/cris-tdep.c b/gdb/cris-tdep.c index 999231cfe0..69e7e5bd4b 100644 --- a/gdb/cris-tdep.c +++ b/gdb/cris-tdep.c @@ -39,7 +39,6 @@ #include "regcache.h" #include "gdb_assert.h" -/* To get entry_point_address. */ #include "objfiles.h" #include "solib.h" /* Support for shared libraries. */ diff --git a/gdb/infcall.c b/gdb/infcall.c index 479492dec2..c9d98cf65e 100644 --- a/gdb/infcall.c +++ b/gdb/infcall.c @@ -592,11 +592,6 @@ call_function_by_hand (struct value *function, int nargs, struct value **args) real_pc = funaddr; dummy_addr = entry_point_address (); - /* Make certain that the address points at real code, and not a - function descriptor. */ - dummy_addr = gdbarch_convert_from_func_ptr_addr (gdbarch, - dummy_addr, - ¤t_target); /* A call dummy always consists of just a single breakpoint, so its address is the same as the address of the dummy. */ bp_addr = dummy_addr; @@ -614,14 +609,16 @@ call_function_by_hand (struct value *function, int nargs, struct value **args) sym = lookup_minimal_symbol ("__CALL_DUMMY_ADDRESS", NULL, NULL); real_pc = funaddr; if (sym) - dummy_addr = SYMBOL_VALUE_ADDRESS (sym); + { + dummy_addr = SYMBOL_VALUE_ADDRESS (sym); + /* Make certain that the address points at real code, and not + a function descriptor. */ + dummy_addr = gdbarch_convert_from_func_ptr_addr (gdbarch, + dummy_addr, + ¤t_target); + } else dummy_addr = entry_point_address (); - /* Make certain that the address points at real code, and not - a function descriptor. */ - dummy_addr = gdbarch_convert_from_func_ptr_addr (gdbarch, - dummy_addr, - ¤t_target); /* A call dummy always consists of just a single breakpoint, so it's address is the same as the address of the dummy. */ bp_addr = dummy_addr; diff --git a/gdb/objfiles.c b/gdb/objfiles.c index 4662e1b121..c983b11613 100644 --- a/gdb/objfiles.c +++ b/gdb/objfiles.c @@ -279,7 +279,26 @@ init_entry_point_info (struct objfile *objfile) CORE_ADDR entry_point_address (void) { - return symfile_objfile ? symfile_objfile->ei.entry_point : 0; + struct gdbarch *gdbarch; + CORE_ADDR entry_point; + + if (symfile_objfile == NULL) + return 0; + + gdbarch = get_objfile_arch (symfile_objfile); + + entry_point = symfile_objfile->ei.entry_point; + + /* Make certain that the address points at real code, and not a + function descriptor. */ + entry_point = gdbarch_convert_from_func_ptr_addr (gdbarch, entry_point, + ¤t_target); + + /* Remove any ISA markers, so that this matches entries in the + symbol table. */ + entry_point = gdbarch_addr_bits_remove (gdbarch, entry_point); + + return entry_point; } /* Create the terminating entry of OBJFILE's minimal symbol table. -- 2.11.0