From d564a7d4b1591e23f0c0f53165d0437a97bf6b96 Mon Sep 17 00:00:00 2001 From: Dave Anglin Date: Sat, 20 Nov 2004 23:56:18 +0000 Subject: [PATCH] * infttrace.h (ttrace_write_reg_64): Add prototype. * hppah-nat.c (store_inferior_registers): Fix warning from ambiguous expression. (store_inferior_registers): Correct formatting. Replace uses of deprecated_registers with calls to regcache_raw_read, regcache_raw_write_part and regcache_raw_read_part. --- gdb/ChangeLog | 9 +++++++++ gdb/hppah-nat.c | 19 ++++++++++++++----- gdb/infttrace.h | 1 + 3 files changed, 24 insertions(+), 5 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9c5ffe4ab7..da6270823f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,12 @@ +2004-11-20 John David Anglin + + * infttrace.h (ttrace_write_reg_64): Add prototype. + * hppah-nat.c (store_inferior_registers): Fix warning from ambiguous + expression. + (store_inferior_registers): Correct formatting. Replace uses of + deprecated_registers with calls to regcache_raw_read, + regcache_raw_write_part and regcache_raw_read_part. + 2004-11-21 Mark Kettenis * config/pa/hppahpux.mh: Remove file. diff --git a/gdb/hppah-nat.c b/gdb/hppah-nat.c index 3979ecef3b..2a79b61e0e 100644 --- a/gdb/hppah-nat.c +++ b/gdb/hppah-nat.c @@ -141,11 +141,11 @@ store_inferior_registers (int regno) layering will not allow us to perform a 64bit register store. What a crock. */ - if (regno == HPPA_PCOQ_HEAD_REGNUM || regno == HPPA_PCOQ_TAIL_REGNUM && len == 8) + if ((regno == HPPA_PCOQ_HEAD_REGNUM || regno == HPPA_PCOQ_TAIL_REGNUM) && len == 8) { CORE_ADDR temp; - temp = *(CORE_ADDR *)&deprecated_registers[DEPRECATED_REGISTER_BYTE (regno)]; + regcache_raw_read (current_regcache, regno, &temp); /* Set the priv level (stored in the low two bits of the PC. */ temp |= 0x3; @@ -155,7 +155,7 @@ store_inferior_registers (int regno) /* If we fail to write the PC, give a true error instead of just a warning. */ - if (errno != 0) + if (errno != 0) { char *err = safe_strerror (errno); char *msg = alloca (strlen (err) + 128); @@ -170,15 +170,24 @@ store_inferior_registers (int regno) the high part of IPSW. What will it take for HP to catch a clue about building sensible interfaces? */ if (regno == HPPA_IPSW_REGNUM && len == 8) - *(int *)&deprecated_registers[DEPRECATED_REGISTER_BYTE (regno)] = 0; + { + int temp = 0; + + regcache_raw_write_part (current_regcache, regno, 0, + sizeof (int), &temp); + } #endif for (i = 0; i < len; i += sizeof (int)) { + int temp; + errno = 0; + regcache_raw_read_part (current_regcache, regno, i, + sizeof (int), &temp); call_ptrace (PT_WUREGS, PIDGET (inferior_ptid), (PTRACE_ARG3_TYPE) addr + i, - *(int *) &deprecated_registers[DEPRECATED_REGISTER_BYTE (regno) + i]); + temp); if (errno != 0) { /* Warning, not error, in case we are attached; sometimes diff --git a/gdb/infttrace.h b/gdb/infttrace.h index c947cc02b9..c505b64821 100644 --- a/gdb/infttrace.h +++ b/gdb/infttrace.h @@ -31,5 +31,6 @@ extern int hpux_has_vforked (int pid, int *childpid); extern int hpux_has_execd (int pid, char **execd_pathname); extern int hpux_has_syscall_event (int pid, enum target_waitkind *kind, int *syscall_id); +extern int ttrace_write_reg_64 (int, CORE_ADDR, CORE_ADDR); #endif