From 28bcfd306b8cf443e9d2612a2f86ec4b05b86ce2 Mon Sep 17 00:00:00 2001 From: Mark Kettenis Date: Sun, 5 Jan 2003 14:07:03 +0000 Subject: [PATCH] * i386-tdep.c (i386_get_longjmp_target): Make usable on x86-64. * x86-64-tdep.c (x86_64_init_abi): Remove FIXME about i386_get_longjmp_target. --- gdb/ChangeLog | 6 ++++++ gdb/i386-tdep.c | 13 +++++++------ gdb/x86-64-tdep.c | 4 ---- 3 files changed, 13 insertions(+), 10 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 703f654..89c2f44 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2003-01-05 Mark Kettenis + + * i386-tdep.c (i386_get_longjmp_target): Make usable on x86-64. + * x86-64-tdep.c (x86_64_init_abi): Remove FIXME about + i386_get_longjmp_target. + 2003-01-05 Andrew Cagney * arm-tdep.c (prologue_cache): Change to a pointer. diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c index 9b4cf02..f4a7df0 100644 --- a/gdb/i386-tdep.c +++ b/gdb/i386-tdep.c @@ -891,15 +891,16 @@ i386_pop_frame (void) /* Figure out where the longjmp will land. Slurp the args out of the stack. We expect the first arg to be a pointer to the jmp_buf structure from which we extract the address that we will land at. - This address is copied into PC. This routine returns true on + This address is copied into PC. This routine returns non-zero on success. */ static int i386_get_longjmp_target (CORE_ADDR *pc) { - char buf[4]; + char buf[8]; CORE_ADDR sp, jb_addr; int jb_pc_offset = gdbarch_tdep (current_gdbarch)->jb_pc_offset; + int len = TARGET_PTR_BIT / TARGET_CHAR_BIT; /* If JB_PC_OFFSET is -1, we have no way to find out where the longjmp will land. */ @@ -907,14 +908,14 @@ i386_get_longjmp_target (CORE_ADDR *pc) return 0; sp = read_register (SP_REGNUM); - if (target_read_memory (sp + 4, buf, 4)) + if (target_read_memory (sp + len, buf, len)) return 0; - jb_addr = extract_address (buf, 4); - if (target_read_memory (jb_addr + jb_pc_offset, buf, 4)) + jb_addr = extract_address (buf, len); + if (target_read_memory (jb_addr + jb_pc_offset, buf, len)) return 0; - *pc = extract_address (buf, 4); + *pc = extract_address (buf, len); return 1; } diff --git a/gdb/x86-64-tdep.c b/gdb/x86-64-tdep.c index 2cf005e..a47ef17 100644 --- a/gdb/x86-64-tdep.c +++ b/gdb/x86-64-tdep.c @@ -967,10 +967,6 @@ x86_64_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch) set_gdbarch_register_byte (gdbarch, x86_64_register_byte); set_gdbarch_register_virtual_type (gdbarch, x86_64_register_virtual_type); - /* FIXME: kettenis/20021026: As long as we don't support longjmp, - that is, as long as we have `tdep->jb_pc_offset == -1', using - i386_get_longjmp_target is fine. */ - set_gdbarch_register_convertible (gdbarch, x86_64_register_convertible); set_gdbarch_register_convert_to_virtual (gdbarch, x86_64_register_convert_to_virtual); -- 2.7.4