Re: [PULL 03/54] target/ppc: Move load and store floating point instructions to decodetree

[Cédric Le Goater]

On 11/10/21 18:04, Laurent Vivier wrote:
> On 10/11/2021 17:56, Cédric Le Goater wrote:
> > On 11/10/21 17:33, Laurent Vivier wrote:
> > > On 09/11/2021 06:51, David Gibson wrote:
> > > > From: Fernando Eckhardt Valle <phervalle@gmail.com>
> > > >
> > > > Move load floating point instructions (lfs, lfsu, lfsx, lfsux, lfd, lfdu, lfdx, 
> > > > lfdux)
> > > > and store floating point instructions(stfs, stfsu, stfsx, stfsux, stfd, stfdu, 
> > > > stfdx,
> > > > stfdux) from legacy system to decodetree.
> > > >
> > > > Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> > > > Signed-off-by: Fernando Eckhardt Valle <fernando.valle@eldorado.org.br>
> > > > Signed-off-by: Matheus Ferst <matheus.ferst@eldorado.org.br>
> > > > Message-Id: <20211029202424.175401-4-matheus.ferst@eldorado.org.br>
> > > > Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> > > > ---
> > > >   target/ppc/insn32.decode           |  24 +++
> > > >   target/ppc/translate/fp-impl.c.inc | 247 +++++++++--------------------
> > > >   target/ppc/translate/fp-ops.c.inc  |  29 ----
> > > >   3 files changed, 95 insertions(+), 205 deletions(-)
> > >
> > > This patch breaks qemu linux-user with an ubuntu bionic chroot.
> > >
> > > The fix proposed by Matheus [1] fixes it for me.
> > > When will it be merged?
> > > It's needed in 6.2
> >
> >
> > It's queued for 6.2 :
> >
> >    https://github.com/legoater/qemu/commits/ppc-6.2
> >
> > I wanted to wait the end of the week before sending a PR. Unless
> > this is critical for you of course.
>
> Not it's not critical. It can wait the end of the week.

It's merged.

Thanks,

C.