[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-arm: Reinsert SRS missing return stateme
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] target-arm: Reinsert SRS missing return statements. |
Date: |
Mon, 15 Apr 2013 08:32:42 +0100 |
On 15 April 2013 08:29, Peter Chubb <address@hidden> wrote:
>>>>>> "Peter" == Peter Maydell <address@hidden> writes:
> Peter> On 15 April 2013 05:50, Peter Chubb <address@hidden>
> Peter> wrote:
>>> The problem is a return directive that was removed in the
>>> refactoring, so after decoding the instruction, qemu would fall
>>> through to do stuff that it should not have done.
>
> Peter> Nice catch for the ARM decoder, but not needed for thumb2 I
> Peter> think?
>
> It was there in the code that was removed.
No it wasn't, I just looked at the broken commit which is why I
pointed it out. (Also you can see that the gen_rfe() above isn't
followed by a return, so I checked code flow to be sure it was OK.)
thanks
-- PMM