diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2023-12-21 00:38:31 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2023-12-21 01:59:22 -0500 |
| commit | d137b254d9aa4c91d5d2e7865d971bbdaa40e769 (patch) | |
| tree | a9ddb0f1f76874b83a4efce003732b2e305b5664 /sim/mips/interp.c | |
| parent | f184f3a224405473d605ba1d7a455b011e8f2b9e (diff) | |
| download | binutils-d137b254d9aa4c91d5d2e7865d971bbdaa40e769.zip binutils-d137b254d9aa4c91d5d2e7865d971bbdaa40e769.tar.gz binutils-d137b254d9aa4c91d5d2e7865d971bbdaa40e769.tar.bz2 | |
sim: signal: mark signal callback funcs as noreturn since they don't return
All funcs already call other funcs that don't return. The mips port is
the only exception because its generic exception handler can return in
the case of normal exceptions. So while the exceptions its signal handler
triggers doesn't return, we can't express that conditional logic. So add
some useless abort calls to make the compiler happy.
Diffstat (limited to 'sim/mips/interp.c')
| -rw-r--r-- | sim/mips/interp.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sim/mips/interp.c b/sim/mips/interp.c index cb650a5..255807f 100644 --- a/sim/mips/interp.c +++ b/sim/mips/interp.c @@ -2498,7 +2498,8 @@ mips_core_signal (SIM_DESC sd, (unsigned long) addr, (unsigned long) ip); COP0_BADVADDR = addr; SignalExceptionDataReference(); - break; + /* Shouldn't actually be reached. */ + abort (); case sim_core_unaligned_signal: sim_io_eprintf (sd, "mips-core: %d byte %s to unaligned address 0x%lx at 0x%lx\n", @@ -2509,7 +2510,8 @@ mips_core_signal (SIM_DESC sd, SignalExceptionAddressLoad(); else SignalExceptionAddressStore(); - break; + /* Shouldn't actually be reached. */ + abort (); default: sim_engine_abort (sd, cpu, cia, |