[prev in list] [next in list] [prev in thread] [next in thread] 

List:       qemu-ppc
Subject:    Re: [Qemu-ppc] [PULL 00/30] ppc patch queue 2013-04-26
From:       Aurelien Jarno <aurelien () aurel32 ! net>
Date:       2013-04-26 23:01:06
Message-ID: 20130426230106.GD25132 () hall ! aurel32 ! net
[Download RAW message or body]

On Fri, Apr 26, 2013 at 11:42:19PM +0200, Alexander Graf wrote:
> 
> 
> Am 26.04.2013 um 23:17 schrieb Aurelien Jarno <aurelien@aurel32.net>:
> 
> > On Fri, Apr 26, 2013 at 11:03:12PM +0200, Alexander Graf wrote:
> > > 
> > > 
> > > Am 26.04.2013 um 22:09 schrieb Blue Swirl <blauwirbel@gmail.com>:
> > > 
> > > > On Fri, Apr 26, 2013 at 6:21 PM, Alexander Graf <agraf@suse.de> wrote:
> > > > > Hi Blue / Aurelien,
> > > > > 
> > > > > This is my current patch queue for ppc.  Please pull.
> > > > 
> > > > Does not build with --enable-debug:
> > > > CC    ppc-softmmu/target-ppc/translate.o
> > > > /src/qemu/target-ppc/translate.c: In function 'gen_lfiwax':
> > > > /src/qemu/target-ppc/translate.c:3378:5: error: incompatible type for
> > > > argument 2 of 'gen_qemu_ld32s'
> > > > /src/qemu/target-ppc/translate.c:2554:20: note: expected 'TCGv_i32'
> > > > but argument is of type 'TCGv_i64'
> > > 
> > > I pushed an updated, fixed tree.
> > 
> > Thanks, and sorry about that. Would it be possible to use the following
> > patch instead? It generates slightly better code (by folding the sign
> > extension in the load when possible).
> 
> How about you just apply the pull request, diff your version against what's in \
> there and apply the diff on top :).

This considered as an abuse of my commit powers and usually leads to
complains.

> I'm quite sure that makes everyone's life easier ;).

Done anyway, I hope you won't complain ;)


Aurelien

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net


[prev in list] [next in list] [prev in thread] [next in thread]