Re: [Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-22 Thread Laurent Desnogues
On Thu, Oct 22, 2009 at 9:33 AM, wrote: > > On Oct 22, 2009, at 10:18, ext Laurent Desnogues wrote: > >> On Thu, Oct 22, 2009 at 8:49 AM,   wrote: >>> >>> On Oct 21, 2009, at 13:46, ext Laurent Desnogues wrote: >>> > @@ -4624,31 +4624,31 @@ static int disas_neon_data_insn(CPUState * > env

Re: [Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-22 Thread Juha.Riihimaki
On Oct 22, 2009, at 10:18, ext Laurent Desnogues wrote: > On Thu, Oct 22, 2009 at 8:49 AM, wrote: >> >> On Oct 21, 2009, at 13:46, ext Laurent Desnogues wrote: >> @@ -4624,31 +4624,31 @@ static int disas_neon_data_insn(CPUState * env, DisasContext *s, uint32_t insn)

Re: [Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-22 Thread Laurent Desnogues
On Thu, Oct 22, 2009 at 8:49 AM, wrote: > > On Oct 21, 2009, at 13:46, ext Laurent Desnogues wrote: > >>> @@ -4624,31 +4624,31 @@ static int disas_neon_data_insn(CPUState * >>> env, DisasContext *s, uint32_t insn) >>>                              switch (size) { >>>                              c

Re: [Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-21 Thread Juha.Riihimaki
On Oct 21, 2009, at 13:46, ext Laurent Desnogues wrote: >> @@ -4624,31 +4624,31 @@ static int disas_neon_data_insn(CPUState * >> env, DisasContext *s, uint32_t insn) >> switch (size) { >> case 0: >> if (op

Re: [Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-21 Thread Laurent Desnogues
On Wed, Oct 21, 2009 at 12:17 PM, wrote: > Shift immediate value is incorrectly overwritten by a temporary > variable in the processing of NEON vsri, vshl and vsli instructions. > > Signed-off-by: Juha Riihimäki > --- > diff --git a/target-arm/translate.c b/target-arm/translate.c > index 59bf7bc

[Qemu-devel] [PATCH 07/12] target-arm: fix neon vsri, vshl and vsli ops

2009-10-21 Thread Juha.Riihimaki
Shift immediate value is incorrectly overwritten by a temporary variable in the processing of NEON vsri, vshl and vsli instructions. Signed-off-by: Juha Riihimäki --- diff --git a/target-arm/translate.c b/target-arm/translate.c index 59bf7bc..c92ecc6 100644 --- a/target-arm/translate.c +++ b/ta