From 720aff63daf0b3a4fded993eaf1b6334a809a471 Mon Sep 17 00:00:00 2001 From: Aleksandar Markovic Date: Thu, 8 Aug 2019 19:37:32 -0400 Subject: [PATCH] target/mips: Add missing 'break' for certain cases of MFTR handling This was found by GCC 8.3 static analysis. Fixes: ead9360e2fb Backports commit f1fadbb27af04115c9d24e0902d88a38d0266854 from qemu --- qemu/target/mips/translate.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/qemu/target/mips/translate.c b/qemu/target/mips/translate.c index c6d50a9b..15fb27e1 100644 --- a/qemu/target/mips/translate.c +++ b/qemu/target/mips/translate.c @@ -9886,6 +9886,7 @@ static void gen_mftr(CPUMIPSState *env, DisasContext *ctx, int rt, int rd, gen_mfc0(ctx, t0, rt, sel); break; } + break; case 12: switch (sel) { case 0: @@ -9895,6 +9896,7 @@ static void gen_mftr(CPUMIPSState *env, DisasContext *ctx, int rt, int rd, gen_mfc0(ctx, t0, rt, sel); break; } + break; case 13: switch (sel) { case 0: