On Tue, Jan 29, 2019 at 3:04 PM Peter Maydell <peter.mayd...@linaro.org> wrote: > > The FCMLA (by element) instruction exists in the > "vector x indexed element" encoding group, but not in > the "scalar x indexed element" group. Correctly UNDEF > the unallocated encodings. > > Reported-by: Laurent Desnogues <laurent.desnog...@gmail.com> > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
Reviewed-by: Laurent Desnogues <laurent.desnog...@gmail.com> Thanks, Laurent > --- > target/arm/translate-a64.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c > index 30bc2412fc0..a7b999d2b5a 100644 > --- a/target/arm/translate-a64.c > +++ b/target/arm/translate-a64.c > @@ -12650,7 +12650,7 @@ static void disas_simd_indexed(DisasContext *s, > uint32_t insn) > case 0x13: /* FCMLA #90 */ > case 0x15: /* FCMLA #180 */ > case 0x17: /* FCMLA #270 */ > - if (!dc_isar_feature(aa64_fcma, s)) { > + if (is_scalar || !dc_isar_feature(aa64_fcma, s)) { > unallocated_encoding(s); > return; > } > -- > 2.20.1 >