[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v3 5/5] target-ppc: add lxvb16x and stxvb16x
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH v3 5/5] target-ppc: add lxvb16x and stxvb16x |
Date: |
Mon, 19 Sep 2016 16:35:37 +1000 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Fri, Sep 16, 2016 at 04:21:51PM +0530, Nikunj A Dadhania wrote:
> lxvb16x: Load VSX Vector Byte*16
> stxvb16x: Store VSX Vector Byte*16
>
> Signed-off-by: Nikunj A Dadhania <address@hidden>
> ---
> target-ppc/translate/vsx-impl.inc.c | 55
> +++++++++++++++++++++++++++++++++++++
> target-ppc/translate/vsx-ops.inc.c | 2 ++
> 2 files changed, 57 insertions(+)
>
> diff --git a/target-ppc/translate/vsx-impl.inc.c
> b/target-ppc/translate/vsx-impl.inc.c
> index 9e7588d..306cc55 100644
> --- a/target-ppc/translate/vsx-impl.inc.c
> +++ b/target-ppc/translate/vsx-impl.inc.c
> @@ -123,6 +123,33 @@ static void gen_lxvh8x(DisasContext *ctx)
> tcg_temp_free(EA);
> }
>
> +static void gen_lxvb16x(DisasContext *ctx)
> +{
> + TCGv EA;
> + TCGv_i64 xth = cpu_vsrh(xT(ctx->opcode));
> + TCGv_i64 xtl = cpu_vsrl(xT(ctx->opcode));
> +
> + if (unlikely(!ctx->vsx_enabled)) {
> + gen_exception(ctx, POWERPC_EXCP_VSXU);
> + return;
> + }
> + gen_set_access_type(ctx, ACCESS_INT);
> + EA = tcg_temp_new();
> + gen_addr_reg_index(ctx, EA);
> + if (ctx->le_mode) {
> + tcg_gen_qemu_ld_i64(xth, EA, ctx->mem_idx, MO_BEQ);
> + tcg_gen_addi_tl(EA, EA, 8);
> + tcg_gen_qemu_ld_i64(xtl, EA, ctx->mem_idx, MO_BEQ);
> + } else {
> + tcg_gen_qemu_ld_i64(xth, EA, ctx->mem_idx, MO_LEQ);
> + gen_helper_deposit32x2(xth, xth);
> + tcg_gen_addi_tl(EA, EA, 8);
> + tcg_gen_qemu_ld_i64(xtl, EA, ctx->mem_idx, MO_LEQ);
> + gen_helper_deposit32x2(xtl, xtl);
Again, not correct. Here just a BE load should suffice, you don't
need any byte or word swapping at all. The instruction is defined as
loading bytes at increasing address into bytes of decreasing
significance. This is exactly a big-endian load.
> + }
> + tcg_temp_free(EA);
> +}
> +
> #define VSX_STORE_SCALAR(name, operation) \
> static void gen_##name(DisasContext *ctx) \
> { \
> @@ -211,6 +238,34 @@ static void gen_stxvh8x(DisasContext *ctx)
> tcg_temp_free(EA);
> }
>
> +static void gen_stxvb16x(DisasContext *ctx)
> +{
> + TCGv_i64 xsh = cpu_vsrh(xS(ctx->opcode));
> + TCGv_i64 xsl = cpu_vsrl(xS(ctx->opcode));
> + TCGv EA;
> +
> + if (unlikely(!ctx->vsx_enabled)) {
> + gen_exception(ctx, POWERPC_EXCP_VSXU);
> + return;
> + }
> + gen_set_access_type(ctx, ACCESS_INT);
> + EA = tcg_temp_new();
> + gen_addr_reg_index(ctx, EA);
> +
> + if (ctx->le_mode) {
> + tcg_gen_qemu_st_i64(xsh, EA, ctx->mem_idx, MO_BEQ);
> + tcg_gen_addi_tl(EA, EA, 8);
> + tcg_gen_qemu_st_i64(xsl, EA, ctx->mem_idx, MO_BEQ);
> + } else {
> + gen_helper_deposit32x2(xsh, xsh);
> + tcg_gen_qemu_st_i64(xsh, EA, ctx->mem_idx, MO_LEQ);
> + tcg_gen_addi_tl(EA, EA, 8);
> + gen_helper_deposit32x2(xsl, xsl);
> + tcg_gen_qemu_st_i64(xsl, EA, ctx->mem_idx, MO_LEQ);
Ditto.
> + }
> + tcg_temp_free(EA);
> +}
> +
> #define MV_VSRW(name, tcgop1, tcgop2, target, source) \
> static void gen_##name(DisasContext *ctx) \
> { \
> diff --git a/target-ppc/translate/vsx-ops.inc.c
> b/target-ppc/translate/vsx-ops.inc.c
> index 21f9064..f5afa0f 100644
> --- a/target-ppc/translate/vsx-ops.inc.c
> +++ b/target-ppc/translate/vsx-ops.inc.c
> @@ -8,6 +8,7 @@ GEN_HANDLER_E(lxvd2x, 0x1F, 0x0C, 0x1A, 0, PPC_NONE,
> PPC2_VSX),
> GEN_HANDLER_E(lxvdsx, 0x1F, 0x0C, 0x0A, 0, PPC_NONE, PPC2_VSX),
> GEN_HANDLER_E(lxvw4x, 0x1F, 0x0C, 0x18, 0, PPC_NONE, PPC2_VSX),
> GEN_HANDLER_E(lxvh8x, 0x1F, 0x0C, 0x19, 0, PPC_NONE, PPC2_ISA300),
> +GEN_HANDLER_E(lxvb16x, 0x1F, 0x0C, 0x1B, 0, PPC_NONE, PPC2_ISA300),
>
> GEN_HANDLER_E(stxsdx, 0x1F, 0xC, 0x16, 0, PPC_NONE, PPC2_VSX),
> GEN_HANDLER_E(stxsibx, 0x1F, 0xD, 0x1C, 0, PPC_NONE, PPC2_ISA300),
> @@ -17,6 +18,7 @@ GEN_HANDLER_E(stxsspx, 0x1F, 0xC, 0x14, 0, PPC_NONE,
> PPC2_VSX207),
> GEN_HANDLER_E(stxvd2x, 0x1F, 0xC, 0x1E, 0, PPC_NONE, PPC2_VSX),
> GEN_HANDLER_E(stxvw4x, 0x1F, 0xC, 0x1C, 0, PPC_NONE, PPC2_VSX),
> GEN_HANDLER_E(stxvh8x, 0x1F, 0x0C, 0x1D, 0, PPC_NONE, PPC2_ISA300),
> +GEN_HANDLER_E(stxvb16x, 0x1F, 0x0C, 0x1F, 0, PPC_NONE, PPC2_ISA300),
>
> GEN_HANDLER_E(mfvsrwz, 0x1F, 0x13, 0x03, 0x0000F800, PPC_NONE, PPC2_VSX207),
> GEN_HANDLER_E(mtvsrwa, 0x1F, 0x13, 0x06, 0x0000F800, PPC_NONE, PPC2_VSX207),
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, (continued)
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/19
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, David Gibson, 2016/09/20
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/20
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, David Gibson, 2016/09/20
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/20
- Re: [Qemu-ppc] [PATCH v3 2/5] target-ppc: improve lxvw4x implementation, Nikunj A Dadhania, 2016/09/19
[Qemu-ppc] [PATCH v3 4/5] target-ppc: add lxvh8x and stxvh8x, Nikunj A Dadhania, 2016/09/16
[Qemu-ppc] [PATCH v3 3/5] target-ppc: improve stxvw4x implementation, Nikunj A Dadhania, 2016/09/16
[Qemu-ppc] [PATCH v3 5/5] target-ppc: add lxvb16x and stxvb16x, Nikunj A Dadhania, 2016/09/16
- Re: [Qemu-ppc] [PATCH v3 5/5] target-ppc: add lxvb16x and stxvb16x,
David Gibson <=
Re: [Qemu-ppc] [PATCH v3 0/5] POWER9 TCG enablements - part4(pending), David Gibson, 2016/09/19