diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-04-22 12:07:14 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2021-04-22 12:07:14 -0400 |
| commit | 2f63213381e3b073f169ee776043486b44ea2e68 (patch) | |
| tree | 55947613d8a4dc4a2262758c5ae053b950e7f0e1 /sim/common/sim-engine.h | |
| parent | 4efeb0d3e8ee210cd61b15355cca39b16b66004d (diff) | |
| download | binutils-2f63213381e3b073f169ee776043486b44ea2e68.zip binutils-2f63213381e3b073f169ee776043486b44ea2e68.tar.gz binutils-2f63213381e3b073f169ee776043486b44ea2e68.tar.bz2 | |
sim: sprinkle some ATTRIBUTE_PRINTF
Add some ATTRIBUTE_PRINTF attributes to functions that take a format
string, to fix a few -Wformat-nonliteral warnings. Use the
ATTRIBUTE_PRINTF macro like we use in GDB, instead of spelling out
__attribute__((format...)). Use ATTRIBUTE_NULL_PRINTF at one place,
because callers expect to be able to pass NULL.
sim/common/ChangeLog:
* callback.c (os_printf_filtered, os_vprintf_filtered,
os_evprintf_filtered, os_error): Use ATTRIBUTE_PRINTF.
* sim-engine.h (sim_engine_abort, sim_engine_vabort): Likewise.
* sim-events.h (sim_events_schedule_tracef,
sim_events_schedule_vtracef): Use ATTRIBUTE_NULL_PRINTF.
Change-Id: Icd206f7b2c325e8b144f72eb129fb2a6b5af2fa3
Diffstat (limited to 'sim/common/sim-engine.h')
| -rw-r--r-- | sim/common/sim-engine.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sim/common/sim-engine.h b/sim/common/sim-engine.h index 25d06f7..5adce8d 100644 --- a/sim/common/sim-engine.h +++ b/sim/common/sim-engine.h @@ -115,14 +115,14 @@ extern void sim_engine_abort sim_cpu *cpu, sim_cia cia, const char *fmt, - ...) __attribute__ ((format (printf, 4, 5))) __attribute__ ((noreturn)); + ...) ATTRIBUTE_PRINTF (4, 5) __attribute__ ((noreturn)); extern void sim_engine_vabort (SIM_DESC sd, sim_cpu *cpu, sim_cia cia, const char *fmt, - va_list ap) __attribute__ ((noreturn)); + va_list ap) ATTRIBUTE_PRINTF (4, 0) __attribute__ ((noreturn)); /* No abort hook - when possible this function exits using the engine_halt function (and SIM_ENGINE_HALT_HOOK). */ |