From: Andrew Cagney Date: Sun, 1 Jun 2003 22:30:30 +0000 (+0000) Subject: Oops, really commit. Part of: X-Git-Tag: ezannoni_pie-20030916-branchpoint~1477 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1fd4ae226388e78eba387080d1bb8b13fa08fb93;p=external%2Fbinutils.git Oops, really commit. Part of: * infcall.c (call_function_by_hand): Update comment on DEPRECATED_DUMMY_WRITE_SP. * mips-tdep.c (mips_gdbarch_init): Do not set deprecated_dummy_write_sp. (mips_eabi_push_dummy_call): Set the SP register. (mips_o64_push_dummy_call): Set the SP register. (mips_o32_push_dummy_call): Set the SP register. (mips_n32n64_push_dummy_call): Set the SP register. --- diff --git a/gdb/infcall.c b/gdb/infcall.c index 82c18ae..546e33c 100644 --- a/gdb/infcall.c +++ b/gdb/infcall.c @@ -809,15 +809,16 @@ You must use a pointer to function type variable. Command ignored.", arg_name); if (struct_return && DEPRECATED_STORE_STRUCT_RETURN_P ()) DEPRECATED_STORE_STRUCT_RETURN (struct_addr, sp); - /* Write the stack pointer. This is here because the statements above - might fool with it. On SPARC, this write also stores the register - window into the right place in the new stack frame, which otherwise - wouldn't happen. (See store_inferior_registers in sparc-nat.c.) */ - /* NOTE: cagney/2003-03-23: Disable this code when there is a - push_dummy_call() method. Since that method will have already - stored the stack pointer (as part of creating the fake call - frame), and none of the code following that code adjusts the - stack-pointer value, the below call is entirely redundant. */ + /* Write the stack pointer. This is here because the statements + above might fool with it. On SPARC, this write also stores the + register window into the right place in the new stack frame, + which otherwise wouldn't happen (see store_inferior_registers in + sparc-nat.c). */ + /* NOTE: cagney/2003-03-23: Since the architecture method + push_dummy_call() should have already stored the stack pointer + (as part of creating the fake call frame), and none of the code + following that call adjusts the stack-pointer value, the below + call is entirely redundant. */ if (DEPRECATED_DUMMY_WRITE_SP_P ()) DEPRECATED_DUMMY_WRITE_SP (sp);