On Tue Aug 22, 2023 at 10:53 PM AEST, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
> ---
> target/ppc/translate.c | 10 +---------
> target/ppc/translate/vsx-impl.c.inc | 19 ++-----------------
> 2 files changed, 3 insertions(+), 26 deletions(-)
>
> diff --git a/target/ppc/translate.c b/target/ppc/translate.c
> index 74796ec7ba..91a9ec2d1c 100644
> --- a/target/ppc/translate.c
> +++ b/target/ppc/translate.c
> @@ -6435,15 +6435,7 @@ static void gen_brw(DisasContext *ctx)
> /* brh */
> static void gen_brh(DisasContext *ctx)
> {
> - TCGv_i64 mask = tcg_constant_i64(0x00ff00ff00ff00ffull);
> - TCGv_i64 t1 = tcg_temp_new_i64();
> - TCGv_i64 t2 = tcg_temp_new_i64();
> -
> - tcg_gen_shri_i64(t1, cpu_gpr[rS(ctx->opcode)], 8);
> - tcg_gen_and_i64(t2, t1, mask);
> - tcg_gen_and_i64(t1, cpu_gpr[rS(ctx->opcode)], mask);
> - tcg_gen_shli_i64(t1, t1, 8);
> - tcg_gen_or_i64(cpu_gpr[rA(ctx->opcode)], t1, t2);
> + tcg_gen_hrev64_i64(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)]);
> }
> #endif
>
> diff --git a/target/ppc/translate/vsx-impl.c.inc
> b/target/ppc/translate/vsx-impl.c.inc
> index 0f5b0056f1..639ab7f1bc 100644
> --- a/target/ppc/translate/vsx-impl.c.inc
> +++ b/target/ppc/translate/vsx-impl.c.inc
> @@ -154,23 +154,8 @@ static void gen_lxvdsx(DisasContext *ctx)
> static void gen_bswap16x8(TCGv_i64 outh, TCGv_i64 outl,
> TCGv_i64 inh, TCGv_i64 inl)
> {
> - TCGv_i64 mask = tcg_constant_i64(0x00FF00FF00FF00FF);
> - TCGv_i64 t0 = tcg_temp_new_i64();
> - TCGv_i64 t1 = tcg_temp_new_i64();
> -
> - /* outh = ((inh & mask) << 8) | ((inh >> 8) & mask) */
> - tcg_gen_and_i64(t0, inh, mask);
> - tcg_gen_shli_i64(t0, t0, 8);
> - tcg_gen_shri_i64(t1, inh, 8);
> - tcg_gen_and_i64(t1, t1, mask);
> - tcg_gen_or_i64(outh, t0, t1);
> -
> - /* outl = ((inl & mask) << 8) | ((inl >> 8) & mask) */
> - tcg_gen_and_i64(t0, inl, mask);
> - tcg_gen_shli_i64(t0, t0, 8);
> - tcg_gen_shri_i64(t1, inl, 8);
> - tcg_gen_and_i64(t1, t1, mask);
> - tcg_gen_or_i64(outl, t0, t1);
> + tcg_gen_hrev64_i64(outh, inh);
> + tcg_gen_hrev64_i64(outl, inl);
> }
>
> static void gen_bswap32x4(TCGv_i64 outh, TCGv_i64 outl,
Reviewed-by: Nicholas Piggin <npig...@gmail.com>
