Fix typo; pass trace_line request as arg; pass common stuff in static struct

2 files changed, 28 insertions, 22 deletions

diff --git a/sim/common/sim-trace.c b/sim/common/sim-trace.c
index d6d85f8..0371b44 100644
--- a/sim/common/sim-trace.c
+++ b/sim/common/sim-trace.c
@@ -262,22 +262,16 @@ trace_uninstall (SIM_DESC sd)
 }
 
 void
-trace_one_insn (SIM_DESC sd, sim_cpu *cpu, const char *filename, 
-		int linenum, int idecode, address_word pc, const char *name)
+trace_one_insn (SIM_DESC sd, sim_cpu *cpu, address_word pc,
+		int line_p, const TRACE_INSN_DATA *insn_data)
 {
-  if (idecode)
-    trace_printf(sd, cpu, "%s:%-*d 0x%.*lx (decode) %s\n",
-		 filename,
-		 SIZE_LINE_NUMBER, linenum,
+  if (!line_p)
+    trace_printf(sd, cpu, "trace-%s: %s:%-*d 0x%.*lx %s\n",
+		 insn_data->phase,
+		 *(insn_data->p_filename),
+		 SIZE_LINE_NUMBER, insn_data->linenum,
 		 SIZE_PC, (long)pc,
-		 name);
-
-  else if (!TRACE_LINENUM_P (cpu))
-    trace_printf(sd, cpu, "%s:%-*d 0x%.*lx %s\n",
-		 filename,
-		 SIZE_LINE_NUMBER, linenum,
-		 SIZE_PC, (long)pc,
-		 name);
+		 *(insn_data->p_name));
 
   else
     {
@@ -315,10 +309,10 @@ trace_one_insn (SIM_DESC sd, sim_cpu *cpu, const char *filename,
 		  sprintf (p, "%s ", pc_function);
 		  p += strlen (p);
 		}
-	      else if (filename)
+	      else if (pc_filename)
 		{
-		  char *q = (char *) strrchr (filename, '/');
-		  sprintf (p, "%s ", (q) ? q+1 : filename);
+		  char *q = (char *) strrchr (pc_filename, '/');
+		  sprintf (p, "%s ", (q) ? q+1 : pc_filename);
 		  p += strlen (p);
 		}
 
@@ -327,10 +321,13 @@ trace_one_insn (SIM_DESC sd, sim_cpu *cpu, const char *filename,
 	    }
 	}
 
-      trace_printf (sd, cpu, "0x%.*x %-*.*s %s\n",
+      trace_printf (sd, cpu, "trace-%s: %s:%-*d 0x%.*x %-*.*s %s\n",
+		    insn_data->phase,
+		    *(insn_data->p_filename),
+		    SIZE_LINE_NUMBER, insn_data->linenum,
 		    SIZE_PC, (unsigned) pc,
 		    SIZE_LOCATION, SIZE_LOCATION, buf,
-		    name);
+		    *(insn_data->p_name));
     }
 }
 
diff --git a/sim/common/sim-trace.h b/sim/common/sim-trace.h
index 91d0be6..8e1eebc 100644
--- a/sim/common/sim-trace.h
+++ b/sim/common/sim-trace.h
@@ -89,6 +89,15 @@ typedef struct {
   FILE *trace_file;
 #define TRACE_FILE(t) ((t)->trace_file)
 } TRACE_DATA;
+
+/* Structure containing constant stuff to pass to trace_one_insn */
+
+typedef struct {
+  const char *phase;		/* which phase this in (decode,insn) */
+  char **p_filename;		/* ptr to filename insns where defined in */
+  char **p_name;		/* ptr to instruction name */
+  int linenum;			/* line number of line where insn is defined */
+} TRACE_INSN_DATA;
 
 /* Usage macros.  */
 
@@ -113,9 +122,9 @@ struct _sim_cpu;
 /* Non-zero if "--trace-fpu" specified for CPU. */
 #define TRACE_FPU_P(cpu) TRACE_P (cpu, TRACE_FPU_IDX)
 
-extern void trace_one_insn PARAMS ((SIM_DESC, sim_cpu *, const char *, 
-				    int, int, address_word,
-				    const char *name));
+extern void trace_one_insn PARAMS ((SIM_DESC, sim_cpu *,
+				    address_word, int,
+				    const TRACE_INSN_DATA *));
 
 extern void trace_printf PARAMS ((SIM_DESC, sim_cpu *, const char *, ...))
      __attribute__((format (printf, 3, 4)));