sim/ppc: mark device_error function as ATTRIBUTE_NORETURN
authorAndrew Burgess <aburgess@redhat.com>
Wed, 12 Oct 2022 10:14:26 +0000 (11:14 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Wed, 19 Oct 2022 13:32:22 +0000 (14:32 +0100)
The device_error function always ends up calling the error function,
which is itself marked as ATTRIBUTE_NORETURN, so it makes sense that
device_error should also be marked ATTRIBUTE_NORETURN.

Doing this resolves a few warnings from hw_ide.c about possibly
uninitialized variables - the variables are only uninitialized after
passing through a call to device_error, which obviously means the
variables are never really used uninitialized, the simulation will
terminate with the device_error call.

sim/ppc/device.h

index bd5390951608eaea081671dea245dafcecbebc6b..65c85e4ddd3590d47191e8560441da91fe2e1a11 100644 (file)
@@ -729,7 +729,7 @@ EXTERN_DEVICE\
 (void) device_error
 (device *me,
  const char *fmt,
- ...) ATTRIBUTE_PRINTF_2;
+ ...) ATTRIBUTE_NORETURN ATTRIBUTE_PRINTF_2;
 
 INLINE_DEVICE\
 (int) device_trace