target/ppc: Move VMX int add/sub saturate insns

[PATCH v2 0/2] target/ppc: Move VMX int add/sub saturate insns

Chinmay Rath posted 2 patches 6 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240523094454.123114-1-rathc@linux.ibm.com

Maintainers: Nicholas Piggin <npiggin@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>

target/ppc/helper.h                 |  24 +--
target/ppc/insn32.decode            |  16 ++
target/ppc/int_helper.c             |  22 +--
target/ppc/translate/vmx-impl.c.inc | 238 ++++++++++++++++++++--------
target/ppc/translate/vmx-ops.c.inc  |  19 +--
5 files changed, 220 insertions(+), 99 deletions(-)

Expand all Fold all

[PATCH v2 0/2] target/ppc: Move VMX int add/sub saturate insns

Posted by Chinmay Rath 6 months ago

Change Log :
1. Changes addressing all review comments by Richard in v1 :
   Having seperate ops table instead of using a 3D array, using
   TRANS_FLAGS instead of sinking flag check in the helper, proper flag
   checks for the insns left behind from GEN_VXFORM_DUAL declarations.
2. Added a second patch to improve the moved insns as per suggestion by
   Richard in v1.                                                                                   

v1: https://lore.kernel.org/qemu-devel/20240512093847.18099-1-rathc@linux.ibm.com/

Chinmay Rath (2):
  target/ppc: Move VMX integer add/sub saturate insns to decodetree.
  target/ppc: Improve VMX integer add/sub saturate instructions.

 target/ppc/helper.h                 |  24 +--
 target/ppc/insn32.decode            |  16 ++
 target/ppc/int_helper.c             |  22 +--
 target/ppc/translate/vmx-impl.c.inc | 238 ++++++++++++++++++++--------
 target/ppc/translate/vmx-ops.c.inc  |  19 +--
 5 files changed, 220 insertions(+), 99 deletions(-)

-- 
2.39.3

Re: [PATCH v2 0/2] target/ppc: Move VMX int add/sub saturate insns

Posted by Richard Henderson 6 months ago

On 5/23/24 02:44, Chinmay Rath wrote:
> Chinmay Rath (2):
>    target/ppc: Move VMX integer add/sub saturate insns to decodetree.
>    target/ppc: Improve VMX integer add/sub saturate instructions.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~