[PATCH v2 0/4] target/ppc: Fix VSX instructions register access

Victor Colombo posted 4 patches 4 years, 1 month ago
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211213120958.24443-1-victor.colombo@eldorado.org.br
Maintainers: "Cédric Le Goater" <clg@kaod.org>, Daniel Henrique Barboza <danielhb413@gmail.com>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>
target/ppc/fpu_helper.c             | 14 +++-----
target/ppc/helper.h                 | 10 +++---
target/ppc/insn32.decode            | 21 +++++++++--
target/ppc/translate/vsx-impl.c.inc | 55 +++++++++++++++++++++--------
target/ppc/translate/vsx-ops.c.inc  |  5 ---
5 files changed, 68 insertions(+), 37 deletions(-)
[PATCH v2 0/4] target/ppc: Fix VSX instructions register access
Posted by Victor Colombo 4 years, 1 month ago
Instructions xscvqpdp, xsmaxcdp, xsmincdp, xsmaxjdp, and xsminjdp are
using the wrong registers, which yields the wrong result when using
them.

This patch series fixes this issue by correcting the registers used.
It also takes the opportunity to move these instructions to decodetree.

v2:
- Change ISA310 flag to ISA300 in xscvqpdp

Matheus Ferst (2):
  target/ppc: fix xscvqpdp register access
  target/ppc: move xscvqpdp to decodetree

Victor Colombo (2):
  target/ppc: Fix xs{max,min}[cj]dp to use VSX registers
  target/ppc: Move xs{max,min}[cj]dp to decodetree

 target/ppc/fpu_helper.c             | 14 +++-----
 target/ppc/helper.h                 | 10 +++---
 target/ppc/insn32.decode            | 21 +++++++++--
 target/ppc/translate/vsx-impl.c.inc | 55 +++++++++++++++++++++--------
 target/ppc/translate/vsx-ops.c.inc  |  5 ---
 5 files changed, 68 insertions(+), 37 deletions(-)

-- 
2.25.1


Re: [PATCH v2 0/4] target/ppc: Fix VSX instructions register access
Posted by Cédric Le Goater 4 years, 1 month ago
On 12/13/21 13:09, Victor Colombo wrote:
> Instructions xscvqpdp, xsmaxcdp, xsmincdp, xsmaxjdp, and xsminjdp are
> using the wrong registers, which yields the wrong result when using
> them.
> 
> This patch series fixes this issue by correcting the registers used.
> It also takes the opportunity to move these instructions to decodetree.


Applied to ppc-next.

Thanks,

C.