sim: mn10300: delete unused memory code

Only one place used get_word/put_word, so inline the usage there.
All the rest is dead code so trim it.

diff --git a/sim/mn10300/ChangeLog b/sim/mn10300/ChangeLog
index 7cc0f320cd7f5e909c6639aef2ee7376bad0fb82..d12997b4abdc9354f08051fc9c1844626aef3fb6 100644 (file)
--- a/sim/mn10300/ChangeLog
+++ b/sim/mn10300/ChangeLog
@@ -1,3 +1,14 @@
+2015-06-11  Mike Frysinger  <vapier@gentoo.org>
+
+       * interp.c (get_byte, get_half, get_word, put_byte, put_half,
+       put_word): Delete.
+       (sim_fetch_register): Inline put_word call.
+       (sim_store_register): Inline get_word call.
+       * mn10300_sim.h (struct _state): Delete mem member.
+       (OP, Simops, get_byte, get_half, get_word, put_byte, put_half,
+       put_word, map): Delete.
+       * op_utils.c (MEMPTR): Delete.
+
 2015-06-11  Mike Frysinger  <vapier@gentoo.org>
 
        * interp.c (round_64, fpu_status_ok): Add static.

diff --git a/sim/mn10300/interp.c b/sim/mn10300/interp.c
index e24cbc59628cf775472c3496dbfdf59b82af25ca..14344fc2ecba635370a496dbcd76b449b1beca0e 100644 (file)
--- a/sim/mn10300/interp.c
+++ b/sim/mn10300/interp.c
@@ -348,58 +348,18 @@ sim_create_inferior (SIM_DESC sd,
 /* FIXME These would more efficient to use than load_mem/store_mem,
    but need to be changed to use the memory map.  */
 
-uint8
-get_byte (uint8 *x)
-{
-  return *x;
-}
-
-uint16
-get_half (uint8 *x)
-{
-  uint8 *a = x;
-  return (a[1] << 8) + (a[0]);
-}
-
-uint32
-get_word (uint8 *x)
-{
-  uint8 *a = x;
-  return (a[3]<<24) + (a[2]<<16) + (a[1]<<8) + (a[0]);
-}
-
-void
-put_byte (uint8 *addr, uint8 data)
-{
-  uint8 *a = addr;
-  a[0] = data;
-}
-
-void
-put_half (uint8 *addr, uint16 data)
-{
-  uint8 *a = addr;
-  a[0] = data & 0xff;
-  a[1] = (data >> 8) & 0xff;
-}
-
-void
-put_word (uint8 *addr, uint32 data)
-{
-  uint8 *a = addr;
-  a[0] = data & 0xff;
-  a[1] = (data >> 8) & 0xff;
-  a[2] = (data >> 16) & 0xff;
-  a[3] = (data >> 24) & 0xff;
-}
-
 int
 sim_fetch_register (SIM_DESC sd,
                    int rn,
                    unsigned char *memory,
                    int length)
 {
-  put_word (memory, State.regs[rn]);
+  reg_t reg = State.regs[rn];
+  uint8 *a = memory;
+  a[0] = reg;
+  a[1] = reg >> 8;
+  a[2] = reg >> 16;
+  a[3] = reg >> 24;
   return length;
 }
  
@@ -409,7 +369,8 @@ sim_store_register (SIM_DESC sd,
                    unsigned char *memory,
                    int length)
 {
-  State.regs[rn] = get_word (memory);
+  uint8 *a = memory;
+  State.regs[rn] = (a[3] << 24) + (a[2] << 16) + (a[1] << 8) + a[0];
   return length;
 }
 

diff --git a/sim/mn10300/mn10300_sim.h b/sim/mn10300/mn10300_sim.h
index 23b8cc2fb0fb840c7527b29f56a57f7c691844e5..f021aee9c00b63aa52899015516a43904b387aaf 100644 (file)
--- a/sim/mn10300/mn10300_sim.h
+++ b/sim/mn10300/mn10300_sim.h
@@ -47,7 +47,6 @@ struct _state
     reg_t fs[32]; /* FS0-31 */
     dword fd[16]; /* FD0,2,...,30 */
   } fpregs;
-  uint8 *mem;                  /* main memory */
 
   /* All internal state modified by signal_exception() that may need to be
      rolled back for passing moment-of-exception image back to gdb. */
@@ -61,8 +60,6 @@ struct _state
 };
 
 extern struct _state State;
-extern uint32 OP[4];
-extern struct simops Simops[];
 
 #define PC     (State.regs[REG_PC])
 #define SP     (State.regs[REG_SP])
@@ -197,15 +194,6 @@ dw2u64 (dword data)
 
 /* Function declarations.  */
 
-uint32 get_word (uint8 *);
-uint16 get_half (uint8 *);
-uint8 get_byte (uint8 *);
-void put_word (uint8 *, uint32);
-void put_half (uint8 *, uint16);
-void put_byte (uint8 *, uint8);
-
-extern uint8 *map (SIM_ADDR addr);
-
 INLINE_SIM_MAIN (void) genericAdd (unsigned32 source, unsigned32 destReg);
 INLINE_SIM_MAIN (void) genericSub (unsigned32 source, unsigned32 destReg);
 INLINE_SIM_MAIN (void) genericCmp (unsigned32 leftOpnd, unsigned32 rightOpnd);

diff --git a/sim/mn10300/op_utils.c b/sim/mn10300/op_utils.c
index 518df9b7e084f359c69b280ab67332c7f4fffc29..316cf7597886747fd3d21369d6d7ead67aa9bfc9 100644 (file)
--- a/sim/mn10300/op_utils.c
+++ b/sim/mn10300/op_utils.c
@@ -188,9 +188,6 @@ do_syscall (void)
 #define RETVAL State.regs[0]   /* return value */
 #define RETERR State.regs[1]   /* return error code */
 
-/* Turn a pointer in a register into a pointer into real memory. */
-#define MEMPTR(x) (State.mem + x)
-
   if ( FUNC == TARGET_SYS_exit )
     {
       /* EXIT - caller can look in PARM1 to work out the reason */