On 12.10.2016 18:23, Michael Walle wrote: > Both branches of the ternary operator have the same expressions. Drop the > operator. > > This fixes: https://bugs.launchpad.net/qemu/+bug/1414293 > > Signed-off-by: Michael Walle <mich...@walle.cc> > --- > target-lm32/translate.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target-lm32/translate.c b/target-lm32/translate.c > index 2d8caeb..534c17c 100644 > --- a/target-lm32/translate.c > +++ b/target-lm32/translate.c > @@ -343,7 +343,7 @@ static void dec_calli(DisasContext *dc) > static inline void gen_compare(DisasContext *dc, int cond) > { > int rX = (dc->format == OP_FMT_RR) ? dc->r2 : dc->r1; > - int rY = (dc->format == OP_FMT_RR) ? dc->r0 : dc->r0; > + int rY = dc->r0; > int rZ = (dc->format == OP_FMT_RR) ? dc->r1 : -1; > int i;
Reviewed-by: Thomas Huth <h...@tuxfamily.org>
signature.asc
Description: OpenPGP digital signature
