[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for 4.1 4/4] target/mips: Add missing 'break' fo
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH for 4.1 4/4] target/mips: Add missing 'break' for certain cases of MTTR handling |
Date: |
Fri, 12 Jul 2019 15:58:29 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 7/12/19 1:37 PM, Aleksandar Markovic wrote:
> From: Aleksandar Markovic <address@hidden>
>
> This was found by GCC 8.3 static analysis.
>
Similar to the previous one:
Fixes: ead9360e2fb
Reported-by: Stefan Weil <address@hidden>
> Signed-off-by: Aleksandar Markovic <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Thanks for cleaning this!
Regards,
Phil.
> ---
> target/mips/translate.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/target/mips/translate.c b/target/mips/translate.c
> index b1cf5f0..ca62800 100644
> --- a/target/mips/translate.c
> +++ b/target/mips/translate.c
> @@ -10055,6 +10055,7 @@ static void gen_mttr(CPUMIPSState *env, DisasContext
> *ctx, int rd, int rt,
> gen_mtc0(ctx, t0, rd, sel);
> break;
> }
> + break;
> case 12:
> switch (sel) {
> case 0:
> @@ -10064,6 +10065,7 @@ static void gen_mttr(CPUMIPSState *env, DisasContext
> *ctx, int rd, int rt,
> gen_mtc0(ctx, t0, rd, sel);
> break;
> }
> + break;
> case 13:
> switch (sel) {
> case 0:
>
- [Qemu-devel] [PATCH for 4.1 2/4] target/mips: Add missing 'break' for a case of MTHC0 handling, (continued)
- [Qemu-devel] [PATCH for 4.1 2/4] target/mips: Add missing 'break' for a case of MTHC0 handling, Aleksandar Markovic, 2019/07/12
- [Qemu-devel] [PATCH for 4.1 3/4] target/mips: Add missing 'break' for certain cases of MFTR handling, Aleksandar Markovic, 2019/07/12
- [Qemu-devel] [PATCH for 4.1 1/4] target/mips: Add 'fall through' comments for handling nanoMips' SHXS, SWXS, Aleksandar Markovic, 2019/07/12
- [Qemu-devel] [PATCH for 4.1 v2] linux-user: Fix structure target_ucontext for MIPS, Aleksandar Markovic, 2019/07/12
- [Qemu-devel] [PATCH for 4.1 4/4] target/mips: Add missing 'break' for certain cases of MTTR handling, Aleksandar Markovic, 2019/07/12
- Re: [Qemu-devel] [PATCH for 4.1 4/4] target/mips: Add missing 'break' for certain cases of MTTR handling,
Philippe Mathieu-Daudé <=