Re: [Qemu-devel] [PATCH 0/3] tcg: Support debugging leakage of temporaries

2011-03-07 Thread Aurelien Jarno
On Sun, Mar 06, 2011 at 09:52:56PM +, Peter Maydell wrote: On 6 March 2011 18:18, Aurelien Jarno aurel...@aurel32.net wrote: On Wed, Feb 23, 2011 at 03:19:33PM +, Peter Maydell wrote: This patchset removes the ad-hoc debug code in target-arm for identifying cases where we leaked TCG

Re: [Qemu-devel] [PATCH 0/3] tcg: Support debugging leakage of temporaries

2011-03-07 Thread Peter Maydell
On 7 March 2011 08:42, Aurelien Jarno aurel...@aurel32.net wrote: On Sun, Mar 06, 2011 at 09:52:56PM +, Peter Maydell wrote: In the long term it would be good if we could make the ARM translate.c less brittle like this. Can we at least define a rule for where we're trying to get to so we

Re: [Qemu-devel] [PATCH 0/3] tcg: Support debugging leakage of temporaries

2011-03-06 Thread Aurelien Jarno
On Wed, Feb 23, 2011 at 03:19:33PM +, Peter Maydell wrote: This patchset removes the ad-hoc debug code in target-arm for identifying cases where we leaked TCG temporary variables, in favour of an implementation in tcg itself. Generally any temporaries created by a target while it is

Re: [Qemu-devel] [PATCH 0/3] tcg: Support debugging leakage of temporaries

2011-03-06 Thread Peter Maydell
On 6 March 2011 18:18, Aurelien Jarno aurel...@aurel32.net wrote: On Wed, Feb 23, 2011 at 03:19:33PM +, Peter Maydell wrote: This patchset removes the ad-hoc debug code in target-arm for identifying cases where we leaked TCG temporary variables, in favour of an implementation in tcg

[Qemu-devel] [PATCH 0/3] tcg: Support debugging leakage of temporaries

2011-02-23 Thread Peter Maydell
This patchset removes the ad-hoc debug code in target-arm for identifying cases where we leaked TCG temporary variables, in favour of an implementation in tcg itself. Generally any temporaries created by a target while it is translating an instruction should be freed by the end of that