Hi Aleksandar, On 7/12/19 1:37 PM, Aleksandar Markovic wrote: > From: Aleksandar Markovic <amarko...@wavecomp.com> > > This was found by GCC 8.3 static analysis. >
I think you can amend: Fixes: 5fb2dcd1792 Reported-by: Stefan Weil <s...@weilnetz.de> > Signed-off-by: Aleksandar Markovic <amarko...@wavecomp.com> > --- > target/mips/translate.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/target/mips/translate.c b/target/mips/translate.c > index 2be5e2d..59d4acd 100644 > --- a/target/mips/translate.c > +++ b/target/mips/translate.c > @@ -6745,6 +6745,7 @@ static void gen_mthc0(DisasContext *ctx, TCGv arg, int > reg, int sel) > default: > goto cp0_unimplemented; > } > + break; > case CP0_REGISTER_17: > switch (sel) { > case 0: > Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> Tested-by: Philippe Mathieu-Daudé <phi...@redhat.com> Thanks, Phil.