diff options
Diffstat (limited to 'sim')
-rw-r--r-- | sim/arm/ChangeLog | 5 | ||||
-rw-r--r-- | sim/arm/armdefs.h | 4 | ||||
-rw-r--r-- | sim/arm/wrapper.c | 4 | ||||
-rw-r--r-- | sim/bfin/ChangeLog | 4 | ||||
-rw-r--r-- | sim/bfin/interp.c | 2 | ||||
-rw-r--r-- | sim/common/ChangeLog | 13 | ||||
-rw-r--r-- | sim/common/hw-events.c | 7 | ||||
-rw-r--r-- | sim/common/hw-events.h | 4 | ||||
-rw-r--r-- | sim/common/hw-tree.h | 2 | ||||
-rw-r--r-- | sim/common/sim-profile.c | 2 | ||||
-rw-r--r-- | sim/common/sim-trace.c | 2 | ||||
-rw-r--r-- | sim/common/sim-trace.h | 5 | ||||
-rw-r--r-- | sim/common/sim-utils.h | 3 | ||||
-rw-r--r-- | sim/rx/ChangeLog | 4 | ||||
-rw-r--r-- | sim/rx/trace.c | 2 | ||||
-rw-r--r-- | sim/v850/ChangeLog | 4 | ||||
-rw-r--r-- | sim/v850/interp.c | 4 |
17 files changed, 50 insertions, 21 deletions
diff --git a/sim/arm/ChangeLog b/sim/arm/ChangeLog index 639942a..54712e3 100644 --- a/sim/arm/ChangeLog +++ b/sim/arm/ChangeLog @@ -1,3 +1,8 @@ +2021-05-03 Simon Marchi <simon.marchi@polymtl.ca> + + * armdefs.h (ARMul_ConsolePrint): Use format attribute. + * wrapper.c (op_printf): Likewise. + 2021-05-01 Mike Frysinger <vapier@gentoo.org> * config.in, configure: Regenerate. diff --git a/sim/arm/armdefs.h b/sim/arm/armdefs.h index 2b5ff9b..68cf199 100644 --- a/sim/arm/armdefs.h +++ b/sim/arm/armdefs.h @@ -18,6 +18,7 @@ #include <stdio.h> #include <stdlib.h> #include <stdint.h> +#include <ansidecl.h> #define FALSE 0 #define TRUE 1 @@ -415,5 +416,6 @@ extern unsigned ARMul_OSHandleSWI (ARMul_State * state, ARMword number); extern void ARMul_UndefInstr (ARMul_State *, ARMword); extern void ARMul_FixCPSR (ARMul_State *, ARMword, ARMword); extern void ARMul_FixSPSR (ARMul_State *, ARMword, ARMword); -extern void ARMul_ConsolePrint (ARMul_State *, const char *, ...); +extern void ARMul_ConsolePrint (ARMul_State *, const char *, ...) + ATTRIBUTE_PRINTF (2, 3); extern void ARMul_SelectProcessor (ARMul_State *, unsigned); diff --git a/sim/arm/wrapper.c b/sim/arm/wrapper.c index c89efe4..54a04f4 100644 --- a/sim/arm/wrapper.c +++ b/sim/arm/wrapper.c @@ -62,8 +62,8 @@ int trace_funcs = 0; static struct disassemble_info info; static char opbuf[1000]; -static int -op_printf (char *buf, char *fmt, ...) +static int ATTRIBUTE_PRINTF (2, 3) +op_printf (char *buf, const char *fmt, ...) { int ret; va_list ap; diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog index 98d8e66..832effc 100644 --- a/sim/bfin/ChangeLog +++ b/sim/bfin/ChangeLog @@ -1,3 +1,7 @@ +2021-05-03 Simon Marchi <simon.marchi@polymtl.ca> + + * interp.c (sim_open): Adjust format string specifier. + 2021-05-02 Mike Frysinger <vapier@gentoo.org> * bfin-sim.c: Include ansidecl.h. diff --git a/sim/bfin/interp.c b/sim/bfin/interp.c index 2aa9279..a4333da 100644 --- a/sim/bfin/interp.c +++ b/sim/bfin/interp.c @@ -747,7 +747,7 @@ sim_open (SIM_OPEN_KIND kind, host_callback *callback, if (sim_core_read_buffer (sd, NULL, read_map, &c, 4, 1) == 0) { bu16 emuexcpt = 0x25; - sim_do_commandf (sd, "memory-size 0x%lx", BFIN_DEFAULT_MEM_SIZE); + sim_do_commandf (sd, "memory-size 0x%x", BFIN_DEFAULT_MEM_SIZE); sim_write (sd, 0, (void *)&emuexcpt, 2); } diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog index fd9a5dd..17b711c 100644 --- a/sim/common/ChangeLog +++ b/sim/common/ChangeLog @@ -1,3 +1,16 @@ +2021-05-03 Simon Marchi <simon.marchi@polymtl.ca> + + * hw-events.h (hw_event_queue_schedule_tracef): Use format attribute. + (hw_event_queue_schedule_vtracef): Likewise. + * hw-tree.h (hw_tree_vparse): Likewise. + * sim-profile.c (profile_vprintf): Likewise. + * sim-trace.c (dis_printf): Likewise. + * sim-trace.h (trace_printf): Likewise. + (trace_vprintf): Likewise. + * sim-utils.h (sim_do_commandf): Likewise. + * hw-events.c (hw_event_queue_schedule): Use + hw_event_queue_schedule_tracef. + 2021-05-02 Simon Marchi <simon.marchi@polymtl.ca> * sim-options.c (standard_option_handler): Add default cases to diff --git a/sim/common/hw-events.c b/sim/common/hw-events.c index 1c749dd..81628f0 100644 --- a/sim/common/hw-events.c +++ b/sim/common/hw-events.c @@ -88,12 +88,7 @@ hw_event_queue_schedule (struct hw *me, hw_event_callback *callback, void *data) { - struct hw_event *event; - va_list dummy; - memset (&dummy, 0, sizeof dummy); - event = hw_event_queue_schedule_vtracef (me, delta_time, callback, data, - NULL, dummy); - return event; + return hw_event_queue_schedule_tracef (me, delta_time, callback, data, NULL); } struct hw_event * diff --git a/sim/common/hw-events.h b/sim/common/hw-events.h index 901063b..51ca5bd 100644 --- a/sim/common/hw-events.h +++ b/sim/common/hw-events.h @@ -39,7 +39,7 @@ struct hw_event *hw_event_queue_schedule_tracef hw_event_callback *handler, void *data, const char *fmt, - ...) ATTRIBUTE_PRINTF (5, 6); + ...) ATTRIBUTE_NULL_PRINTF (5, 6); struct hw_event *hw_event_queue_schedule_vtracef (struct hw *me, @@ -47,7 +47,7 @@ struct hw_event *hw_event_queue_schedule_vtracef hw_event_callback *handler, void *data, const char *fmt, - va_list ap); + va_list ap) ATTRIBUTE_NULL_PRINTF (5, 0); void hw_event_queue_deschedule diff --git a/sim/common/hw-tree.h b/sim/common/hw-tree.h index 86ebf17..85c91ad 100644 --- a/sim/common/hw-tree.h +++ b/sim/common/hw-tree.h @@ -39,7 +39,7 @@ struct hw *hw_tree_parse struct hw *hw_tree_vparse (struct hw *root, const char *fmt, - va_list ap); + va_list ap) ATTRIBUTE_PRINTF (2, 0); void hw_tree_finish diff --git a/sim/common/sim-profile.c b/sim/common/sim-profile.c index 75bdc70..455e1a2 100644 --- a/sim/common/sim-profile.c +++ b/sim/common/sim-profile.c @@ -454,7 +454,7 @@ profile_option_handler (SIM_DESC sd, /* Profiling output hooks. */ -static void +static void ATTRIBUTE_PRINTF (3, 0) profile_vprintf (SIM_DESC sd, sim_cpu *cpu, const char *fmt, va_list ap) { FILE *fp = PROFILE_FILE (CPU_PROFILE_DATA (cpu)); diff --git a/sim/common/sim-trace.c b/sim/common/sim-trace.c index 5f161da..26d9556 100644 --- a/sim/common/sim-trace.c +++ b/sim/common/sim-trace.c @@ -889,7 +889,7 @@ dis_read (bfd_vma memaddr, bfd_byte *myaddr, unsigned int length, return 0; } -static int +static int ATTRIBUTE_PRINTF (2, 3) dis_printf (SIM_CPU *cpu, const char *fmt, ...) { SIM_DESC sd = CPU_STATE (cpu); diff --git a/sim/common/sim-trace.h b/sim/common/sim-trace.h index 7ff7b02..c89084e 100644 --- a/sim/common/sim-trace.h +++ b/sim/common/sim-trace.h @@ -653,9 +653,10 @@ do { \ extern void trace_printf (SIM_DESC, sim_cpu *, const char *, ...) - ATTRIBUTE_PRINTF (3, 4); + ATTRIBUTE_PRINTF (3, 4); -extern void trace_vprintf (SIM_DESC, sim_cpu *, const char *, va_list); +extern void trace_vprintf (SIM_DESC, sim_cpu *, const char *, va_list) + ATTRIBUTE_PRINTF (3, 0); /* Debug support. This is included here because there isn't enough of it to justify diff --git a/sim/common/sim-utils.h b/sim/common/sim-utils.h index f089245..9cc19f5 100644 --- a/sim/common/sim-utils.h +++ b/sim/common/sim-utils.h @@ -68,7 +68,8 @@ struct bfd *sim_load_file (SIM_DESC sd, const char *myname, int lma_p, sim_write_fn do_load); /* Internal version of sim_do_command, include formatting */ -void sim_do_commandf (SIM_DESC sd, const char *fmt, ...); +void sim_do_commandf (SIM_DESC sd, const char *fmt, ...) + ATTRIBUTE_PRINTF (2, 3); /* These are defined in callback.c as cover functions to the vprintf diff --git a/sim/rx/ChangeLog b/sim/rx/ChangeLog index d49ff89..4d7e93d 100644 --- a/sim/rx/ChangeLog +++ b/sim/rx/ChangeLog @@ -1,3 +1,7 @@ +2021-05-03 Simon Marchi <simon.marchi@polymtl.ca> + + * trace.c (op_printf): Likewise. + 2021-05-01 Mike Frysinger <vapier@gentoo.org> * config.in, configure: Regenerate. diff --git a/sim/rx/trace.c b/sim/rx/trace.c index caa51b7..a005797 100644 --- a/sim/rx/trace.c +++ b/sim/rx/trace.c @@ -96,7 +96,7 @@ compare_symbols (const PTR ap, const PTR bp) static char opbuf[1000]; -static int +static int ATTRIBUTE_PRINTF (2, 3) op_printf (char *buf, char *fmt, ...) { int ret; diff --git a/sim/v850/ChangeLog b/sim/v850/ChangeLog index 643d28b..9e5f770 100644 --- a/sim/v850/ChangeLog +++ b/sim/v850/ChangeLog @@ -1,3 +1,7 @@ +2021-05-03 Simon Marchi <simon.marchi@polymtl.ca> + + * interp.c (sim_open): Adjust format string specifier. + 2021-05-01 Mike Frysinger <vapier@gentoo.org> * config.in, configure: Regenerate. diff --git a/sim/v850/interp.c b/sim/v850/interp.c index e72dab6..ff2f6a2 100644 --- a/sim/v850/interp.c +++ b/sim/v850/interp.c @@ -215,9 +215,9 @@ sim_open (SIM_OPEN_KIND kind, /* Allocate core managed memory */ /* "Mirror" the ROM addresses below 1MB. */ - sim_do_commandf (sd, "memory region 0,0x100000,0x%lx", V850_ROM_SIZE); + sim_do_commandf (sd, "memory region 0,0x100000,0x%x", V850_ROM_SIZE); /* Chunk of ram adjacent to rom */ - sim_do_commandf (sd, "memory region 0x100000,0x%lx", V850_LOW_END-0x100000); + sim_do_commandf (sd, "memory region 0x100000,0x%x", V850_LOW_END-0x100000); /* peripheral I/O region - mirror 1K across 4k (0x1000) */ sim_do_command (sd, "memory region 0xfff000,0x1000,1024"); /* similarly if in the internal RAM region */ |