On Mon, Sep 12, 2016 at 12:11:30PM +0530, Nikunj A Dadhania wrote: > Implement macro to consolidate store operations using newer > tcg_gen_qemu_ld functions.
s/store/load/, but I can fix that as I apply if I don't find anything
else in the series which requires a respin.
>
> Signed-off-by: Nikunj A Dadhania <nik...@linux.vnet.ibm.com>
> ---
> target-ppc/translate.c | 58
> +++++++++++++++++---------------------------------
> 1 file changed, 20 insertions(+), 38 deletions(-)
>
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index a27f455..6606969 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -2462,50 +2462,32 @@ static inline void gen_align_no_le(DisasContext *ctx)
> }
>
> /*** Integer load
> ***/
> -static inline void gen_qemu_ld8u(DisasContext *ctx, TCGv arg1, TCGv arg2)
> -{
> - tcg_gen_qemu_ld8u(arg1, arg2, ctx->mem_idx);
> -}
> -
> -static inline void gen_qemu_ld16u(DisasContext *ctx, TCGv arg1, TCGv arg2)
> -{
> - TCGMemOp op = MO_UW | ctx->default_tcg_memop_mask;
> - tcg_gen_qemu_ld_tl(arg1, arg2, ctx->mem_idx, op);
> -}
> -
> -static inline void gen_qemu_ld16s(DisasContext *ctx, TCGv arg1, TCGv arg2)
> -{
> - TCGMemOp op = MO_SW | ctx->default_tcg_memop_mask;
> - tcg_gen_qemu_ld_tl(arg1, arg2, ctx->mem_idx, op);
> -}
> +#define DEF_MEMOP(op) ((op) | ctx->default_tcg_memop_mask)
>
> -static inline void gen_qemu_ld32u(DisasContext *ctx, TCGv arg1, TCGv arg2)
> -{
> - TCGMemOp op = MO_UL | ctx->default_tcg_memop_mask;
> - tcg_gen_qemu_ld_tl(arg1, arg2, ctx->mem_idx, op);
> +#define GEN_QEMU_LOAD_TL(ldop, op) \
> +static void glue(gen_qemu_, ldop)(DisasContext *ctx, \
> + TCGv val, \
> + TCGv addr) \
> +{ \
> + tcg_gen_qemu_ld_tl(val, addr, ctx->mem_idx, op); \
> }
>
> -static void gen_qemu_ld32u_i64(DisasContext *ctx, TCGv_i64 val, TCGv addr)
> -{
> - TCGv tmp = tcg_temp_new();
> - gen_qemu_ld32u(ctx, tmp, addr);
> - tcg_gen_extu_tl_i64(val, tmp);
> - tcg_temp_free(tmp);
> -}
> +GEN_QEMU_LOAD_TL(ld8u, DEF_MEMOP(MO_UB))
> +GEN_QEMU_LOAD_TL(ld16u, DEF_MEMOP(MO_UW))
> +GEN_QEMU_LOAD_TL(ld16s, DEF_MEMOP(MO_SW))
> +GEN_QEMU_LOAD_TL(ld32u, DEF_MEMOP(MO_UL))
> +GEN_QEMU_LOAD_TL(ld32s, DEF_MEMOP(MO_SL))
>
> -static inline void gen_qemu_ld32s(DisasContext *ctx, TCGv arg1, TCGv arg2)
> -{
> - TCGMemOp op = MO_SL | ctx->default_tcg_memop_mask;
> - tcg_gen_qemu_ld_tl(arg1, arg2, ctx->mem_idx, op);
> +#define GEN_QEMU_LOAD_64(ldop, op) \
> +static void glue(gen_qemu_, glue(ldop, _i64))(DisasContext *ctx, \
> + TCGv_i64 val, \
> + TCGv addr) \
> +{ \
> + tcg_gen_qemu_ld_i64(val, addr, ctx->mem_idx, op); \
> }
>
> -static void gen_qemu_ld32s_i64(DisasContext *ctx, TCGv_i64 val, TCGv addr)
> -{
> - TCGv tmp = tcg_temp_new();
> - gen_qemu_ld32s(ctx, tmp, addr);
> - tcg_gen_ext_tl_i64(val, tmp);
> - tcg_temp_free(tmp);
> -}
> +GEN_QEMU_LOAD_64(ld32u, DEF_MEMOP(MO_UL))
> +GEN_QEMU_LOAD_64(ld32s, DEF_MEMOP(MO_SL))
>
> static inline void gen_qemu_ld64(DisasContext *ctx, TCGv_i64 arg1, TCGv arg2)
> {
--
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
