[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 18/26] tcg-aarch64: Pass qemu_ld/st arguments
From: |
Claudio Fontana |
Subject: |
Re: [Qemu-devel] [PATCH v3 18/26] tcg-aarch64: Pass qemu_ld/st arguments directly |
Date: |
Fri, 11 Apr 2014 14:34:29 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:24.0) Gecko/20100101 Thunderbird/24.0.1 |
On 03.04.2014 21:56, Richard Henderson wrote:
> Instead of passing them the "args" array.
>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> tcg/aarch64/tcg-target.c | 49
> +++++++++++++++++-------------------------------
> 1 file changed, 17 insertions(+), 32 deletions(-)
>
> diff --git a/tcg/aarch64/tcg-target.c b/tcg/aarch64/tcg-target.c
> index 68305ea..3a2955f 100644
> --- a/tcg/aarch64/tcg-target.c
> +++ b/tcg/aarch64/tcg-target.c
> @@ -1271,20 +1271,13 @@ static void tcg_out_qemu_st_direct(TCGContext *s,
> TCGMemOp memop,
> }
> }
>
> -static void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, TCGMemOp
> memop)
> +static void tcg_out_qemu_ld(TCGContext *s, TCGReg data_reg, TCGReg addr_reg,
> + TCGMemOp memop, int mem_index)
> {
> - TCGReg addr_reg, data_reg;
> #ifdef CONFIG_SOFTMMU
> - int mem_index;
> - TCGMemOp s_bits;
> + TCGMemOp s_bits = memop & MO_SIZE;
> uint8_t *label_ptr;
> -#endif
> - data_reg = args[0];
> - addr_reg = args[1];
>
> -#ifdef CONFIG_SOFTMMU
> - mem_index = args[2];
> - s_bits = memop & MO_SIZE;
> tcg_out_tlb_read(s, addr_reg, s_bits, &label_ptr, mem_index, 1);
> tcg_out_qemu_ld_direct(s, memop, data_reg, addr_reg, TCG_REG_X1);
> add_qemu_ldst_label(s, 1, memop, data_reg, addr_reg,
> @@ -1295,20 +1288,12 @@ static void tcg_out_qemu_ld(TCGContext *s, const
> TCGArg *args, TCGMemOp memop)
> #endif /* CONFIG_SOFTMMU */
> }
>
> -static void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, TCGMemOp
> memop)
> +static void tcg_out_qemu_st(TCGContext *s, TCGReg data_reg, TCGReg addr_reg,
> + TCGMemOp memop, int mem_index)
> {
> - TCGReg addr_reg, data_reg;
> #ifdef CONFIG_SOFTMMU
> - int mem_index;
> - TCGMemOp s_bits;
> + TCGMemOp s_bits = memop & MO_SIZE;
> uint8_t *label_ptr;
> -#endif
> - data_reg = args[0];
> - addr_reg = args[1];
> -
> -#ifdef CONFIG_SOFTMMU
> - mem_index = args[2];
> - s_bits = memop & MO_SIZE;
>
> tcg_out_tlb_read(s, addr_reg, s_bits, &label_ptr, mem_index, 0);
> tcg_out_qemu_st_direct(s, memop, data_reg, addr_reg, TCG_REG_X1);
> @@ -1588,38 +1573,38 @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc,
> break;
>
> case INDEX_op_qemu_ld8u:
> - tcg_out_qemu_ld(s, args, MO_UB);
> + tcg_out_qemu_ld(s, a0, a1, MO_UB, a2);
> break;
> case INDEX_op_qemu_ld8s:
> - tcg_out_qemu_ld(s, args, MO_SB);
> + tcg_out_qemu_ld(s, a0, a1, MO_SB, a2);
> break;
> case INDEX_op_qemu_ld16u:
> - tcg_out_qemu_ld(s, args, MO_TEUW);
> + tcg_out_qemu_ld(s, a0, a1, MO_TEUW, a2);
> break;
> case INDEX_op_qemu_ld16s:
> - tcg_out_qemu_ld(s, args, MO_TESW);
> + tcg_out_qemu_ld(s, a0, a1, MO_TESW, a2);
> break;
> case INDEX_op_qemu_ld32u:
> case INDEX_op_qemu_ld32:
> - tcg_out_qemu_ld(s, args, MO_TEUL);
> + tcg_out_qemu_ld(s, a0, a1, MO_TEUL, a2);
> break;
> case INDEX_op_qemu_ld32s:
> - tcg_out_qemu_ld(s, args, MO_TESL);
> + tcg_out_qemu_ld(s, a0, a1, MO_TESL, a2);
> break;
> case INDEX_op_qemu_ld64:
> - tcg_out_qemu_ld(s, args, MO_TEQ);
> + tcg_out_qemu_ld(s, a0, a1, MO_TEQ, a2);
> break;
> case INDEX_op_qemu_st8:
> - tcg_out_qemu_st(s, args, MO_UB);
> + tcg_out_qemu_st(s, a0, a1, MO_UB, a2);
> break;
> case INDEX_op_qemu_st16:
> - tcg_out_qemu_st(s, args, MO_TEUW);
> + tcg_out_qemu_st(s, a0, a1, MO_TEUW, a2);
> break;
> case INDEX_op_qemu_st32:
> - tcg_out_qemu_st(s, args, MO_TEUL);
> + tcg_out_qemu_st(s, a0, a1, MO_TEUL, a2);
> break;
> case INDEX_op_qemu_st64:
> - tcg_out_qemu_st(s, args, MO_TEQ);
> + tcg_out_qemu_st(s, a0, a1, MO_TEQ, a2);
> break;
>
> case INDEX_op_bswap32_i64:
>
Reviewed-by: Claudio Fontana <address@hidden>
- [Qemu-devel] [PATCH v3 12/26] tcg-aarch64: Introduce tcg_out_insn_3314, (continued)
- [Qemu-devel] [PATCH v3 12/26] tcg-aarch64: Introduce tcg_out_insn_3314, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 13/26] tcg-aarch64: Implement tcg_register_jit, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 14/26] tcg-aarch64: Avoid add with zero in tlb load, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 15/26] tcg-aarch64: Use tcg_out_call for qemu_ld/st, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 16/26] tcg-aarch64: Use ADR to pass the return address to the ld/st helpers, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 17/26] tcg-aarch64: Use TCGMemOp in qemu_ld/st, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 19/26] tcg-aarch64: Implement TCG_TARGET_HAS_new_ldst, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 18/26] tcg-aarch64: Pass qemu_ld/st arguments directly, Richard Henderson, 2014/04/03
- Re: [Qemu-devel] [PATCH v3 18/26] tcg-aarch64: Pass qemu_ld/st arguments directly,
Claudio Fontana <=
- [Qemu-devel] [PATCH v3 21/26] tcg-aarch64: Introduce tcg_out_insn_3507, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 20/26] tcg-aarch64: Support stores of zero, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 23/26] tcg-aarch64: Replace aarch64_ldst_op_data with TCGMemOp, Richard Henderson, 2014/04/03
- [Qemu-devel] [PATCH v3 22/26] tcg-aarch64: Merge aarch64_ldst_get_data/type into tcg_out_op, Richard Henderson, 2014/04/03