[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 5/7] target/arm/translate-a64: Don't underdecode a
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH 5/7] target/arm/translate-a64: Don't underdecode add/sub extended register |
Date: |
Mon, 28 Jan 2019 11:17:21 +0000 |
On Mon, 28 Jan 2019 at 11:16, Laurent Desnogues
<address@hidden> wrote:
>
> On Fri, Jan 25, 2019 at 7:26 PM Peter Maydell <address@hidden> wrote:
> >
> > In the "add/subtract (extended register)" encoding group, the "opt"
> > field in bits [23:22] must be zero. Correctly UNDEF the unallocated
> > encodings where this field is not zero.
> >
> > Reported-by: Laurent Desnogues <address@hidden>
> > Signed-off-by: Peter Maydell <address@hidden>
> > ---
> > target/arm/translate-a64.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
> > index 2cade64ed25..efd2f6490b5 100644
> > --- a/target/arm/translate-a64.c
> > +++ b/target/arm/translate-a64.c
> > @@ -4204,12 +4204,13 @@ static void disas_add_sub_ext_reg(DisasContext *s,
> > uint32_t insn)
> > bool setflags = extract32(insn, 29, 1);
> > bool sub_op = extract32(insn, 30, 1);
> > bool sf = extract32(insn, 31, 1);
> > + bool opt = extract32(insn, 22, 2);
>
> I'd prefer an int to a bool.
Oops, yes. I think I got mixed up with all the other bool
flags here, but it's a 2 bit field so definitely should be int.
thanks
-- PMM
- [Qemu-arm] [PATCH 4/7] target/arm/translate-a64: Don't underdecode SIMD ld/st single, (continued)
- [Qemu-arm] [PATCH 4/7] target/arm/translate-a64: Don't underdecode SIMD ld/st single, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 1/7] target/arm/translate-a64: Don't underdecode system instructions, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 3/7] target/arm/translate-a64: Don't underdecode SIMD ld/st multiple, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 6/7] target/arm/translate-a64: Don't underdecode FP insns, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 5/7] target/arm/translate-a64: Don't underdecode add/sub extended register, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 2/7] target/arm/translate-a64: Don't underdecode PRFM, Peter Maydell, 2019/01/25
- [Qemu-arm] [PATCH 7/7] target/arm/translate-a64: Don't underdecode SDOT and UDOT, Peter Maydell, 2019/01/25