This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [commit] Delete MIPS only DEPRECATED_IGNORE_HELPER_CALLER
- From: Andrew Cagney <cagney at gnu dot org>
- To: gdb-patches at sources dot redhat dot com
- Date: Sun, 31 Oct 2004 11:26:10 -0500
- Subject: Re: [commit] Delete MIPS only DEPRECATED_IGNORE_HELPER_CALLER
- References: <418510D1.8000500@gnu.org>
Andrew Cagney wrote:
This is was added as a hack for mips16 floating-point. Given that
MIPS16 backtraces are pretty much AWAL, the loss of this hack is the
least of the MIPS16 problems. If anyone is interested in renovating the
MIPS16 I'm all ears.
with patch.
2004-10-31 Andrew Cagney <cagney@gnu.org>
* mips-tdep.c (mips_ignore_helper): Delete.
* config/mips/tm-mips.h (DEPRECATED_IGNORE_HELPER_CALL): Delete.
* infrun.c (handle_inferior_event): Delete #ifdef
DEPRECATED_IGNORE_HELPER_CALL code.
Index: infrun.c
===================================================================
RCS file: /cvs/src/src/gdb/infrun.c,v
retrieving revision 1.179
diff -p -u -r1.179 infrun.c
--- infrun.c 29 Oct 2004 20:23:08 -0000 1.179
+++ infrun.c 31 Oct 2004 16:13:03 -0000
@@ -2303,27 +2303,6 @@ process_event_stop_test:
return;
}
-#ifdef DEPRECATED_IGNORE_HELPER_CALL
- /* On MIPS16, a function that returns a floating point value may
- call a library helper function to copy the return value to a
- floating point register. The DEPRECATED_IGNORE_HELPER_CALL
- macro returns non-zero if we should ignore (i.e. step over)
- this function call. */
- /* FIXME: cagney/2004-07-21: These custom ``ignore frame when
- stepping'' function attributes (SIGTRAMP_FRAME,
- DEPRECATED_IGNORE_HELPER_CALL, SKIP_TRAMPOLINE_CODE,
- skip_language_trampoline frame, et.al.) need to be replaced
- with generic attributes bound to the frame's function. */
- if (DEPRECATED_IGNORE_HELPER_CALL (stop_pc))
- {
- /* We're doing a "next", set a breakpoint at callee's return
- address (the address at which the caller will
- resume). */
- insert_step_resume_breakpoint_at_frame (get_prev_frame (get_current_frame ()));
- keep_going (ecs);
- return;
- }
-#endif
if (step_over_calls == STEP_OVER_ALL)
{
/* We're doing a "next", set a breakpoint at callee's return
Index: mips-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/mips-tdep.c,v
retrieving revision 1.357
diff -p -u -r1.357 mips-tdep.c
--- mips-tdep.c 31 Oct 2004 03:11:54 -0000 1.357
+++ mips-tdep.c 31 Oct 2004 16:13:03 -0000
@@ -4590,27 +4590,6 @@ mips_in_solib_return_trampoline (CORE_AD
return 0; /* not a stub */
}
-
-/* Return non-zero if the PC is in a library helper function that
- should be ignored. This implements the
- DEPRECATED_IGNORE_HELPER_CALL macro. */
-
-int
-mips_ignore_helper (CORE_ADDR pc)
-{
- char *name;
-
- /* Find the starting address and name of the function containing the PC. */
- if (find_pc_partial_function (pc, &name, NULL, NULL) == 0)
- return 0;
-
- /* If the PC is in __mips16_ret_{d,s}f, this is a library helper function
- that we want to ignore. */
- return (strcmp (name, "__mips16_ret_sf") == 0
- || strcmp (name, "__mips16_ret_df") == 0);
-}
-
-
/* Convert a dbx stab register number (from `r' declaration) to a GDB
[1 * NUM_REGS .. 2 * NUM_REGS) REGNUM. */
Index: config/mips/tm-mips.h
===================================================================
RCS file: /cvs/src/src/gdb/config/mips/tm-mips.h,v
retrieving revision 1.75
diff -p -u -r1.75 tm-mips.h
--- config/mips/tm-mips.h 31 Oct 2004 15:50:54 -0000 1.75
+++ config/mips/tm-mips.h 31 Oct 2004 16:13:04 -0000
@@ -44,8 +44,4 @@ extern int mips_step_skips_delay (CORE_A
#define MDEBUG_EFI_SYMBOL_NAME "__GDB_EFI_INFO__"
-/* Functions for dealing with MIPS16 call and return stubs. */
-#define DEPRECATED_IGNORE_HELPER_CALL(pc) mips_ignore_helper (pc)
-extern int mips_ignore_helper (CORE_ADDR pc);
-
#endif /* TM_MIPS_H */