diff options
| author | Pierrick Bouvier <pierrick.bouvier@linaro.org> | 2023-02-21 16:30:06 +0100 |
|---|---|---|
| committer | Philippe Mathieu-Daudé <philmd@linaro.org> | 2023-02-27 22:29:01 +0100 |
| commit | a1a65aade61fae15fdb6560e1a618865f6f21012 (patch) | |
| tree | 907b8f5e504b498bf03b5dafba4e32b17346095c /target/ppc | |
| parent | 60f5fadd13740c61fdee0012a0dd21867493c909 (diff) | |
| download | qemu-a1a65aade61fae15fdb6560e1a618865f6f21012.zip qemu-a1a65aade61fae15fdb6560e1a618865f6f21012.tar.gz qemu-a1a65aade61fae15fdb6560e1a618865f6f21012.tar.bz2 | |
target/ppc: Fix warning with clang-15
When compiling for windows-arm64 using clang-15, it reports a sometimes
uninitialized variable. This seems to be a false positive, as a default
case guards switch expressions, preventing to return an uninitialized
value, but clang seems unhappy with assert(0) definition.
Change code to g_assert_not_reached() fix the warning.
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20230221153006.20300-5-pierrick.bouvier@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Diffstat (limited to 'target/ppc')
| -rw-r--r-- | target/ppc/dfp_helper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/target/ppc/dfp_helper.c b/target/ppc/dfp_helper.c index cc02431..5967ea0 100644 --- a/target/ppc/dfp_helper.c +++ b/target/ppc/dfp_helper.c @@ -121,7 +121,7 @@ static void dfp_set_round_mode_from_immediate(uint8_t r, uint8_t rmc, case 3: /* use FPSCR rounding mode */ return; default: - assert(0); /* cannot get here */ + g_assert_not_reached(); } } else { /* r == 1 */ switch (rmc & 3) { @@ -138,7 +138,7 @@ static void dfp_set_round_mode_from_immediate(uint8_t r, uint8_t rmc, rnd = DEC_ROUND_HALF_DOWN; break; default: - assert(0); /* cannot get here */ + g_assert_not_reached(); } } decContextSetRounding(&dfp->context, rnd); |