[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 02/12] target/unicore32/translate: Add missing fallthrough annotat
From: |
Thomas Huth |
Subject: |
[PULL 02/12] target/unicore32/translate: Add missing fallthrough annotations |
Date: |
Wed, 16 Dec 2020 18:29:39 +0100 |
Looking at the way the code is formatted here (there is an empty line
after break statements, but none where the break is missing), and the
instruction set overview at https://en.wikipedia.org/wiki/Unicore the
fallthrough is very likely intended here. So add a fallthrough comment
to make the it compilable with -Werror=implicit-fallthrough.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20201211152426.350966-3-thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
target/unicore32/translate.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/unicore32/translate.c b/target/unicore32/translate.c
index d4b06df672..962f9877a0 100644
--- a/target/unicore32/translate.c
+++ b/target/unicore32/translate.c
@@ -1801,6 +1801,7 @@ static void disas_uc32_insn(CPUUniCore32State *env,
DisasContext *s)
do_misc(env, s, insn);
break;
}
+ /* fallthrough */
case 0x1:
if (((UCOP_OPCODES >> 2) == 2) && !UCOP_SET_S) {
do_misc(env, s, insn);
@@ -1817,6 +1818,7 @@ static void disas_uc32_insn(CPUUniCore32State *env,
DisasContext *s)
if (UCOP_SET(8) || UCOP_SET(5)) {
ILLEGAL;
}
+ /* fallthrough */
case 0x3:
do_ldst_ir(env, s, insn);
break;
--
2.27.0
- [PULL 00/12] Compile QEMU with -Wimplicit-fallthrough, Thomas Huth, 2020/12/16
- [PULL 01/12] disas/libvixl: Fix fall-through annotation for GCC >= 7, Thomas Huth, 2020/12/16
- [PULL 02/12] target/unicore32/translate: Add missing fallthrough annotations,
Thomas Huth <=
- [PULL 03/12] hw/rtc/twl92230: Silence warnings about missing fallthrough statements, Thomas Huth, 2020/12/16
- [PULL 04/12] hw/timer/renesas_tmr: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 05/12] target/i386: silence the compiler warnings in gen_shiftd_rm_T1, Thomas Huth, 2020/12/16
- [PULL 06/12] hw/intc/arm_gicv3_kvm: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 09/12] target/sparc/win_helper: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 07/12] accel/tcg/user-exec: silence the compiler warnings, Thomas Huth, 2020/12/16
- [PULL 11/12] tests/fp: Do not emit implicit-fallthrough warnings in the softfloat tests, Thomas Huth, 2020/12/16
- [PULL 12/12] configure: Compile with -Wimplicit-fallthrough=2, Thomas Huth, 2020/12/16
- [PULL 10/12] tcg/optimize: Add fallthrough annotations, Thomas Huth, 2020/12/16
- [PULL 08/12] target/sparc/translate: silence the compiler warnings, Thomas Huth, 2020/12/16