[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 14/60] AArch64: Add orr instruction emulation
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 14/60] AArch64: Add orr instruction emulation |
Date: |
Mon, 18 Nov 2013 13:15:20 +0000 |
On 18 November 2013 13:12, Michael Matz <address@hidden> wrote:
> Hi,
>
> On Mon, 18 Nov 2013, Claudio Fontana wrote:
>
>> >> + case 3:
>> >> + tcg_gen_rotr_i64(r, cpu_reg(reg), tcg_shift);
>> >> + break;
>> >
>> > Incorrect rotate for 32bit?
>
> 32bit rotates and shifts were fixed in a patch later than the 60er series
> Alex posted. See attached. (Generally there are many fixes to emulated
> instructions in that branch)
I think we're going to need to look through and fold in those
fixes, otherwise we'll end up reduplicating that work in the
course of code review :-(
-- PMM