Re: [Qemu-devel] [PATCH 20/21] target-mips: switch to AREG0 free mode

[Aurelien Jarno]

On Mon, Sep 03, 2012 at 07:15:29PM +0000, Blue Swirl wrote:
> On Mon, Sep 3, 2012 at 3:50 PM, Aurelien Jarno <address@hidden> wrote:
> > On Sun, Sep 02, 2012 at 05:33:49PM +0000, Blue Swirl wrote:
> >> Add an explicit CPUState parameter instead of relying on AREG0
> >> and switch to AREG0 free mode.
> >>
> >> Signed-off-by: Blue Swirl <address@hidden>
> >> ---
> >>  configure                 |    2 +-
> >>  target-mips/Makefile.objs |    2 -
> >>  target-mips/cpu.h         |   16 +-
> >>  target-mips/helper.h      |  410 +++++++++---------
> >>  target-mips/op_helper.c   | 1065 
> >> ++++++++++++++++++++++++---------------------
> >>  target-mips/translate.c   |  754 ++++++++++++++++----------------
> >>  6 files changed, 1163 insertions(+), 1086 deletions(-)
> >
> > Acked-by: Aurelien Jarno <address@hidden>
> >
> > Please commit this patch asap after the 1.2 release, even if the patches
> > for the other targets are not ready, so that it doesn't hold the
> > development.
> 
> Thanks. There's at least Jia Liu's MIPS DSP series, it probably does
> not conflict so much but it hasn't been converted to new AREG0-free
> order either.
> 

I am still reviewing it, but I don't think it is ready yet, so it's
better to get the AREG0 patches first.

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