sim: cr16: drop global callback state

Now that we have access to the sim state in all the right places,
use existing sim helpers in place of cr16_callback directly.

diff --git a/sim/cr16/ChangeLog b/sim/cr16/ChangeLog
index 0bf6e848c3d11edb7006b65072b4c11d6fb02a6f..6f7b850a60929bd47b7350c0396aca7810900f16 100644 (file)
--- a/sim/cr16/ChangeLog
+++ b/sim/cr16/ChangeLog
@@ -1,3 +1,19 @@
+2015-11-15  Mike Frysinger  <vapier@gentoo.org>
+
+       * cr16_sim.h (cr16_callback): Delete.
+       * interp.c (cr16_callback): Delete.
+       (do_run): Delete cr16_callback.  Replace
+       cr16_callback->printf_filtered with sim_io_printf.
+       (sim_open): Delete cr16_callback assignment.
+       (step_once, sim_create_inferior): Replace
+       cr16_callback->printf_filtered with sim_io_printf.
+       * simops.c (move_to_cr, trace_input_func, do_trace_output_flush,
+       do_trace_output_finish, trace_output_40, trace_output_32,
+       trace_output_16, trace_output_void, trace_output_flag): Replace
+       cr16_callback->printf_filtered with sim_io_printf and
+       cr16_callback->flush_stdout with sim_io_flush_stdout.
+       (OP_14C_14): Likewise.  Rename cr16_callback to cb.
+
 2015-11-15  Mike Frysinger  <vapier@gentoo.org>
 
        * Makefile.in (SIM_OBJS): Add sim-reason.o, sim-resume.o, and

diff --git a/sim/cr16/cr16_sim.h b/sim/cr16/cr16_sim.h
index bbba90894d197a49a8292e465d0429cb5df36b89..c1c9a388e8f33b203a7c051640a9feeac8141b88 100644 (file)
--- a/sim/cr16/cr16_sim.h
+++ b/sim/cr16/cr16_sim.h
@@ -246,7 +246,6 @@ struct _state
 } State;
 
 
-extern host_callback *cr16_callback;
 extern uint32 OP[4];
 extern uint32 sign_flag;
 extern struct simops Simops[];

diff --git a/sim/cr16/interp.c b/sim/cr16/interp.c
index 8925d856de13ca3e8b0480005b13f1860fe6ddf9..967f1b12228dc84762039e15cd606e20204c5605 100644 (file)
--- a/sim/cr16/interp.c
+++ b/sim/cr16/interp.c
@@ -35,8 +35,6 @@
 
 int cr16_debug;
 
-host_callback *cr16_callback;
-
 uint32 OP[4];
 uint32 sign_flag;
 
@@ -326,12 +324,11 @@ get_operands (operand_desc *s, uint64 ins, int isize, int nops)
 static int
 do_run (SIM_DESC sd, SIM_CPU *cpu, uint64 mcode)
 {
-  host_callback *cr16_callback = STATE_CALLBACK (sd);
   struct hash_entry *h;
 
 #ifdef DEBUG
   if ((cr16_debug & DEBUG_INSTRUCTION) != 0)
-    (*cr16_callback->printf_filtered) (cr16_callback, "do_long 0x%x\n", mcode);
+    sim_io_printf (sd, "do_long 0x%x\n", mcode);
 #endif
 
    h = lookup_hash (sd, cpu, mcode, 1);
@@ -463,8 +460,6 @@ sim_open (SIM_OPEN_KIND kind, struct host_callback_struct *cb, struct bfd *abfd,
      but it matches what the toolchain currently expects.  Ugh.  */
   sim_do_commandf (sd, "memory-size %#x", 20 * 1024 * 1024);
 
-  cr16_callback = cb;
-
   /* put all the opcodes in the hash table.  */
   if (!init_p++)
     {
@@ -585,7 +580,7 @@ step_once (SIM_DESC sd, SIM_CPU *cpu)
   curr_ins_size = do_run (sd, cpu, mcode);
 
 #if CR16_DEBUG
-  (*cr16_callback->printf_filtered) (cr16_callback, "INS: PC=0x%X, mcode=0x%X\n", PC, mcode);
+  sim_io_printf (sd, "INS: PC=0x%X, mcode=0x%X\n", PC, mcode);
 #endif
 
   if (curr_ins_size == 0)
@@ -674,7 +669,7 @@ sim_create_inferior (SIM_DESC sd, struct bfd *abfd, char **argv, char **env)
     start_address = 0x0;
 #ifdef DEBUG
   if (cr16_debug)
-    (*cr16_callback->printf_filtered) (cr16_callback, "sim_create_inferior:  PC=0x%lx\n", (long) start_address);
+    sim_io_printf (sd, "sim_create_inferior:  PC=0x%lx\n", (long) start_address);
 #endif
   {
     SIM_CPU *cpu = STATE_CPU (sd, 0);

diff --git a/sim/cr16/simops.c b/sim/cr16/simops.c
index 9186c7f4a687cf00fc5ea82c2f1cc39b34c7968c..537e2a6bbcac4940f01078afe267b21e3a26b74c 100644 (file)
--- a/sim/cr16/simops.c
+++ b/sim/cr16/simops.c
@@ -183,8 +183,8 @@ move_to_cr (SIM_DESC sd, SIM_CPU *cpu, int cr, creg_t mask, creg_t val, int psw_
 #if 0
       else
        val &= PSR_MASK;
-             (*cr16_callback->printf_filtered)
-               (cr16_callback,
+         sim_io_printf
+               (sd,
                 "ERROR at PC 0x%x: ST can only be set when FX is set.\n", PC);
          EXCEPTION (SIM_SIGILL);
 #endif
@@ -259,7 +259,7 @@ trace_input_func (SIM_DESC sd, const char *name, enum op_types in1, enum op_type
     }
 
   if ((cr16_debug & DEBUG_LINE_NUMBER) == 0)
-    (*cr16_callback->printf_filtered) (cr16_callback,
+    sim_io_printf (sd,
                                       "0x%.*x %s: %-*s ",
                                       SIZE_PC, (unsigned)PC,
                                       type,
@@ -311,7 +311,7 @@ trace_input_func (SIM_DESC sd, const char *name, enum op_types in1, enum op_type
            }
        }
 
-      (*cr16_callback->printf_filtered) (cr16_callback,
+      sim_io_printf (sd,
                                         "0x%.*x %s: %-*.*s %-*s ",
                                         SIZE_PC, (unsigned)PC,
                                         type,
@@ -386,12 +386,12 @@ trace_input_func (SIM_DESC sd, const char *name, enum op_types in1, enum op_type
     {
       *p++ = '\n';
       *p = '\0';
-      (*cr16_callback->printf_filtered) (cr16_callback, "%s", buf);
+      sim_io_printf (sd, "%s", buf);
     }
   else
     {
       *p = '\0';
-      (*cr16_callback->printf_filtered) (cr16_callback, "%-*s", SIZE_OPERANDS, buf);
+      sim_io_printf (sd, "%-*s", SIZE_OPERANDS, buf);
 
       p = buf;
       for (i = 0; i < 3; i++)
@@ -400,43 +400,43 @@ trace_input_func (SIM_DESC sd, const char *name, enum op_types in1, enum op_type
          switch (in[i])
            {
            case OP_VOID:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s", SIZE_VALUES, "");
+             sim_io_printf (sd, "%*s", SIZE_VALUES, "");
              break;
 
            case OP_REG:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16) GPR (OP[i]));
              break;
 
            case OP_REGP:
              tmp = (long)((((uint32) GPR (OP[i])) << 16) | ((uint32) GPR (OP[i] + 1)));
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.8lx", SIZE_VALUES-10, "", tmp);
+             sim_io_printf (sd, "%*s0x%.8lx", SIZE_VALUES-10, "", tmp);
              break;
 
            case OP_PROC_REG:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16) CREG (OP[i]));
              break;
 
            case OP_CONSTANT16:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16)OP[i]);
              break;
 
            case OP_CONSTANT4:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16)SEXT4(OP[i]));
              break;
 
            case OP_CONSTANT3:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16)SEXT3(OP[i]));
              break;
 
            case OP_MEMREF2:
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16)OP[i]);
-             (*cr16_callback->printf_filtered) (cr16_callback, "%*s0x%.4x", SIZE_VALUES-6, "",
+             sim_io_printf (sd, "%*s0x%.4x", SIZE_VALUES-6, "",
                                                 (uint16)GPR (OP[i + 1]));
              i++;
              break;
@@ -444,24 +444,24 @@ trace_input_func (SIM_DESC sd, const char *name, enum op_types in1, enum op_type
        }
     }
 
-  (*cr16_callback->flush_stdout) (cr16_callback);
+  sim_io_flush_stdout (sd);
 }
 
 static void
 do_trace_output_flush (SIM_DESC sd)
 {
-  (*cr16_callback->flush_stdout) (cr16_callback);
+  sim_io_flush_stdout (sd);
 }
 
 static void
 do_trace_output_finish (SIM_DESC sd)
 {
-  (*cr16_callback->printf_filtered) (cr16_callback,
+  sim_io_printf (sd,
                                     " F0=%d F1=%d C=%d\n",
                                     (State.trace.psw & PSR_F_BIT) != 0,
                                     (State.trace.psw & PSR_F_BIT) != 0,
                                     (State.trace.psw & PSR_C_BIT) != 0);
-  (*cr16_callback->flush_stdout) (cr16_callback);
+  sim_io_flush_stdout (sd);
 }
 
 #if 0
@@ -470,7 +470,7 @@ trace_output_40 (SIM_DESC sd, uint64 val)
 {
   if ((cr16_debug & (DEBUG_TRACE | DEBUG_VALUES)) == (DEBUG_TRACE | DEBUG_VALUES))
     {
-      (*cr16_callback->printf_filtered) (cr16_callback,
+      sim_io_printf (sd,
                                         " :: %*s0x%.2x%.8lx",
                                         SIZE_VALUES - 12,
                                         "",
@@ -486,7 +486,7 @@ trace_output_32 (SIM_DESC sd, uint32 val)
 {
   if ((cr16_debug & (DEBUG_TRACE | DEBUG_VALUES)) == (DEBUG_TRACE | DEBUG_VALUES))
     {
-      (*cr16_callback->printf_filtered) (cr16_callback,
+      sim_io_printf (sd,
                                         " :: %*s0x%.8x",
                                         SIZE_VALUES - 10,
                                         "",
@@ -500,7 +500,7 @@ trace_output_16 (SIM_DESC sd, uint16 val)
 {
   if ((cr16_debug & (DEBUG_TRACE | DEBUG_VALUES)) == (DEBUG_TRACE | DEBUG_VALUES))
     {
-      (*cr16_callback->printf_filtered) (cr16_callback,
+      sim_io_printf (sd,
                                         " :: %*s0x%.4x",
                                         SIZE_VALUES - 6,
                                         "",
@@ -514,7 +514,7 @@ trace_output_void (SIM_DESC sd)
 {
   if ((cr16_debug & (DEBUG_TRACE | DEBUG_VALUES)) == (DEBUG_TRACE | DEBUG_VALUES))
     {
-      (*cr16_callback->printf_filtered) (cr16_callback, "\n");
+      sim_io_printf (sd, "\n");
       do_trace_output_flush (sd);
     }
 }
@@ -524,7 +524,7 @@ trace_output_flag (SIM_DESC sd)
 {
   if ((cr16_debug & (DEBUG_TRACE | DEBUG_VALUES)) == (DEBUG_TRACE | DEBUG_VALUES))
     {
-      (*cr16_callback->printf_filtered) (cr16_callback,
+      sim_io_printf (sd,
                                         " :: %*s",
                                         SIZE_VALUES,
                                         "");
@@ -5053,6 +5053,7 @@ OP_14C_14 (SIM_DESC sd, SIM_CPU *cpu)
 void
 OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
 {
+  host_callback *cb = STATE_CALLBACK (sd);
   uint32 tmp;
   uint16 a;
   trace_input ("excp", OP_CONSTANT4, OP_VOID, OP_VOID);
@@ -5078,25 +5079,25 @@ OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
        if (first_time)
          {
            first_time = 0;
-           (*cr16_callback->printf_filtered) (cr16_callback, "Trap  #     PC ");
+           sim_io_printf (sd, "Trap  #     PC ");
            for (i = 0; i < 16; i++)
-             (*cr16_callback->printf_filtered) (cr16_callback, "  %sr%d", (i > 9) ? "" : " ", i);
-           (*cr16_callback->printf_filtered) (cr16_callback, "         a0         a1 f0 f1 c\n");
+             sim_io_printf (sd, "  %sr%d", (i > 9) ? "" : " ", i);
+           sim_io_printf (sd, "         a0         a1 f0 f1 c\n");
          }
 
-       (*cr16_callback->printf_filtered) (cr16_callback, "Trap %2d 0x%.4x:", (int)OP[0], (int)PC);
+       sim_io_printf (sd, "Trap %2d 0x%.4x:", (int)OP[0], (int)PC);
 
        for (i = 0; i < 16; i++)
-         (*cr16_callback->printf_filtered) (cr16_callback, " %.4x", (int) GPR (i));
+         sim_io_printf (sd, " %.4x", (int) GPR (i));
 
        for (i = 0; i < 2; i++)
-         (*cr16_callback->printf_filtered) (cr16_callback, " %.2x%.8lx",
+         sim_io_printf (sd, " %.2x%.8lx",
                                             ((int)(ACC (i) >> 32) & 0xff),
                                             ((unsigned long) ACC (i)) & 0xffffffff);
 
-       (*cr16_callback->printf_filtered) (cr16_callback, "  %d  %d %d\n",
+       sim_io_printf (sd, "  %d  %d %d\n",
                                           PSR_F != 0, PSR_F != 0, PSR_C != 0);
-       (*cr16_callback->flush_stdout) (cr16_callback);
+       sim_io_flush_stdout (sd);
        break;
       }
 #endif
@@ -5258,8 +5259,8 @@ OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
                if (os_sig == -1)
                  {
                    trace_output_void (sd);
-                   (*cr16_callback->printf_filtered) (cr16_callback, "Unknown signal %d\n", PARM2);
-                   (*cr16_callback->flush_stdout) (cr16_callback);
+                   sim_io_printf (sd, "Unknown signal %d\n", PARM2);
+                   sim_io_flush_stdout (sd);
                    EXCEPTION (SIM_SIGILL);
                  }
                else
@@ -5332,47 +5333,45 @@ OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
 
          case TARGET_SYS_read:
            trace_input ("<read>", OP_REG, OP_MEMREF, OP_REG);
-           RETVAL (cr16_callback->read (cr16_callback, PARM1,
-                       MEMPTR (((unsigned long)PARM3 << 16)
-                               |((unsigned long)PARM2)), PARM4));
+           RETVAL (cb->read (cb, PARM1,
+                             MEMPTR (((unsigned long)PARM3 << 16)
+                                     | ((unsigned long)PARM2)), PARM4));
            trace_output_16 (sd, result);
            break;
 
          case TARGET_SYS_write:
            trace_input ("<write>", OP_REG, OP_MEMREF, OP_REG);
-           RETVAL ((int)cr16_callback->write (cr16_callback, PARM1,
-                      MEMPTR (((unsigned long)PARM3 << 16) | PARM2), PARM4));
+           RETVAL ((int)cb->write (cb, PARM1,
+                                   MEMPTR (((unsigned long)PARM3 << 16)
+                                           | PARM2), PARM4));
            trace_output_16 (sd, result);
            break;
 
          case TARGET_SYS_lseek:
            trace_input ("<lseek>", OP_REG, OP_REGP, OP_REG);
-           RETVAL32 (cr16_callback->lseek (cr16_callback, PARM1,
-                                           ((((long) PARM3) << 16) | PARM2), 
-                                           PARM4));
+           RETVAL32 (cb->lseek (cb, PARM1, ((((long) PARM3) << 16) | PARM2),
+                                PARM4));
            trace_output_32 (sd, result);
            break;
 
          case TARGET_SYS_close:
            trace_input ("<close>", OP_REG, OP_VOID, OP_VOID);
-           RETVAL (cr16_callback->close (cr16_callback, PARM1));
+           RETVAL (cb->close (cb, PARM1));
            trace_output_16 (sd, result);
            break;
 
          case TARGET_SYS_open:
            trace_input ("<open>", OP_MEMREF, OP_REG, OP_VOID);
-           RETVAL32 (cr16_callback->open (cr16_callback, 
-                                MEMPTR ((((unsigned long)PARM2)<<16)|PARM1), 
-                                PARM3));
+           RETVAL32 (cb->open (cb, MEMPTR ((((unsigned long)PARM2) << 16)
+                                           | PARM1), PARM3));
            trace_output_32 (sd, result);
            break;
 
 #ifdef TARGET_SYS_rename
          case TARGET_SYS_rename:
            trace_input ("<rename>", OP_MEMREF, OP_MEMREF, OP_VOID);
-           RETVAL (cr16_callback->rename (cr16_callback, 
-                                  MEMPTR ((((unsigned long)PARM2)<<16) |PARM1),
-                                  MEMPTR ((((unsigned long)PARM4)<<16) |PARM3)));
+           RETVAL (cb->rename (cb, MEMPTR ((((unsigned long)PARM2) << 16) | PARM1),
+                                   MEMPTR ((((unsigned long)PARM4) << 16) | PARM3)));
            trace_output_16 (sd, result);
            break;
 #endif
@@ -5391,8 +5390,7 @@ OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
 
          case TARGET_SYS_unlink:
            trace_input ("<unlink>", OP_MEMREF, OP_VOID, OP_VOID);
-           RETVAL (cr16_callback->unlink (cr16_callback, 
-                                MEMPTR (((unsigned long)PARM2<<16)|PARM1)));
+           RETVAL (cb->unlink (cb, MEMPTR (((unsigned long)PARM2 << 16) | PARM1)));
            trace_output_16 (sd, result);
            break;
 
@@ -5475,12 +5473,12 @@ OP_C_C (SIM_DESC sd, SIM_CPU *cpu)
                sim_engine_halt (sd, cpu, NULL, PC, sim_exited, GPR (2));
                break;
              default:
-               cr16_callback->error (cr16_callback, "Unknown syscall %d", FUNC);
+               cb->error (cb, "Unknown syscall %d", FUNC);
                break;
            }
          }
        if ((uint16) result == (uint16) -1)
-         RETERR (cr16_callback->get_errno(cr16_callback));
+         RETERR (cb->get_errno (cb));
        else
          RETERR (0);
        break;