Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-22 Thread Aurelien Jarno
On Sun, Dec 22, 2013 at 06:16:44PM +0100, Alexander Graf wrote: > > On 22.12.2013, at 17:37, Aurelien Jarno wrote: > > > On Fri, Dec 20, 2013 at 11:01:50AM +0100, Alexander Graf wrote: > >> The recent VSX patches broken compilation of QEMU when configurated > >> with --enable-debug, as it was tr

Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-22 Thread Alexander Graf
On 22.12.2013, at 17:37, Aurelien Jarno wrote: > On Fri, Dec 20, 2013 at 11:01:50AM +0100, Alexander Graf wrote: >> The recent VSX patches broken compilation of QEMU when configurated >> with --enable-debug, as it was treating "target long" TCG variables >> as "i64" which is not true for 32bit t

Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-22 Thread Aurelien Jarno
On Fri, Dec 20, 2013 at 11:01:50AM +0100, Alexander Graf wrote: > The recent VSX patches broken compilation of QEMU when configurated > with --enable-debug, as it was treating "target long" TCG variables > as "i64" which is not true for 32bit targets. > > This patch fixes all the places that the c

Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-20 Thread Richard Henderson
On 12/20/2013 07:27 AM, Alexander Graf wrote: > Yes, please. Make sure to always compile ppc-softmmu alongside ppc64-softmmu > and use --enable-debug when working on TCG. Or --enable-debug-tcg. If you only ever use --enable-debug, you can miss warnings generated by the optimizer (e.g. may be use

Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-20 Thread Alexander Graf
On 20.12.2013, at 15:44, Tom Musta wrote: > On 12/20/2013 4:01 AM, Alexander Graf wrote: >> The recent VSX patches broken compilation of QEMU when configurated >> with --enable-debug, as it was treating "target long" TCG variables >> as "i64" which is not true for 32bit targets. >> >> This patc

Re: [Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-20 Thread Tom Musta
On 12/20/2013 4:01 AM, Alexander Graf wrote: > The recent VSX patches broken compilation of QEMU when configurated > with --enable-debug, as it was treating "target long" TCG variables > as "i64" which is not true for 32bit targets. > > This patch fixes all the places that the compiler has found t

[Qemu-devel] [PATCH] PPC: Fix compilation with TCG debug

2013-12-20 Thread Alexander Graf
The recent VSX patches broken compilation of QEMU when configurated with --enable-debug, as it was treating "target long" TCG variables as "i64" which is not true for 32bit targets. This patch fixes all the places that the compiler has found to use the correct variable type and if necessary manual