diff options
| author | Philippe Mathieu-Daudé | 2021-01-19 07:27:39 +0100 |
|---|---|---|
| committer | Peter Maydell | 2021-01-19 16:45:14 +0100 |
| commit | 0ae4f11ee57350dac0e705ba79516310400ff43c (patch) | |
| tree | 86aac5f474716122f292ce44f8d31114d1b575a1 /target/arm | |
| parent | npcm7xx_adc-test: Fix memleak in adc_qom_set (diff) | |
| download | qemu-0ae4f11ee57350dac0e705ba79516310400ff43c.tar.gz qemu-0ae4f11ee57350dac0e705ba79516310400ff43c.tar.xz qemu-0ae4f11ee57350dac0e705ba79516310400ff43c.zip | |
target/arm/m_helper: Silence GCC 10 maybe-uninitialized error
When building with GCC 10.2 configured with --extra-cflags=-Os, we get:
target/arm/m_helper.c: In function ‘arm_v7m_cpu_do_interrupt’:
target/arm/m_helper.c:1811:16: error: ‘restore_s16_s31’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
1811 | if (restore_s16_s31) {
| ^
target/arm/m_helper.c:1350:10: note: ‘restore_s16_s31’ was declared here
1350 | bool restore_s16_s31;
| ^~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
Initialize the 'restore_s16_s31' variable to silence the warning.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-id: 20210119062739.589049-1-f4bug@amsat.org
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'target/arm')
| -rw-r--r-- | target/arm/m_helper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/arm/m_helper.c b/target/arm/m_helper.c index 6176003029..731c435c00 100644 --- a/target/arm/m_helper.c +++ b/target/arm/m_helper.c @@ -1347,7 +1347,7 @@ static void do_v7m_exception_exit(ARMCPU *cpu) bool exc_secure = false; bool return_to_secure; bool ftype; - bool restore_s16_s31; + bool restore_s16_s31 = false; /* * If we're not in Handler mode then jumps to magic exception-exit |