Re: [Qemu-devel] [PATCH] target-arm: cleanup internal resource leaks

2009-10-22 Thread Juha.Riihimaki
On Oct 22, 2009, at 08:40, Riihimaki Juha (Nokia-D/Helsinki) wrote: >>> @@ -4676,12 +4694,16 @@ static int disas_neon_data_insn(CPUState * >>> env, DisasContext *s, uint32_t insn) >>>gen_neon_narrow_satu(size - 1, tmp, >>> cpu_V0); >>>} >>>

Re: [Qemu-devel] [PATCH] target-arm: cleanup internal resource leaks

2009-10-21 Thread Juha.Riihimaki
Thanks for your feedback. I'll iron out the issues you mentioned, please look at my further inlined comments below. Cheers, Juha On Oct 21, 2009, at 22:20, ext Laurent Desnogues wrote: >> @@ -4676,12 +4694,16 @@ static int disas_neon_data_insn(CPUState * >> env, DisasContext *s, uint32_t insn

Re: [Qemu-devel] [PATCH] target-arm: cleanup internal resource leaks

2009-10-21 Thread Laurent Desnogues
On Mon, Oct 19, 2009 at 1:44 PM, wrote: > Current ARM translator code has several places where it leaves > temporary TCG variables alive. This patch removes all such instances I > have found so far. Sorry for the mangled inlined patch, the mailserver > I use doesn't like patches so I've also incl

[Qemu-devel] [PATCH] target-arm: cleanup internal resource leaks

2009-10-19 Thread Juha.Riihimaki
Current ARM translator code has several places where it leaves temporary TCG variables alive. This patch removes all such instances I have found so far. Sorry for the mangled inlined patch, the mailserver I use doesn't like patches so I've also included it as an attachment that hopefully status cor