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