[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/21] target-arm: A64: add support for ld/st pa
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 01/21] target-arm: A64: add support for ld/st pair |
Date: |
Thu, 19 Dec 2013 17:25:27 +0000 |
On 19 December 2013 16:58, Richard Henderson <address@hidden> wrote:
> On 12/17/2013 07:12 AM, Peter Maydell wrote:
>> + TCGMemOp memop = MO_TE + size;
>
> Watch the double space (multiple occurrences).
>
>
>> + if (is_vector) {
>> + if (opc == 3) {
>> + unallocated_encoding(s);
>> + return;
>> + }
>
> In the integer LDP description I see
>
> if L:opc<0> == '01' || opc == '11' then UnallocatedEncoding();
>
> I think that opc == 3 check should be hoisted outside is_vector.
I thought I'd correctly rolled that into the opc&1 check
in the non-vector case but looking more closely you're
right. Hoisting the opc==3 out is going to be easiest.
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 11/21] target-arm: Update generic cpreg code for AArch64, (continued)
[Qemu-devel] [PATCH 01/21] target-arm: A64: add support for ld/st pair, Peter Maydell, 2013/12/17
[Qemu-devel] [PATCH 05/21] target-arm: A64: add support for add, addi, sub, subi, Peter Maydell, 2013/12/17