From: Aurelien Jarno <aurelien@aurel32.net>
To: Richard Henderson <rth@twiddle.net>
Cc: qemu-devel@nongnu.org, aliguori@amazon.com
Subject: Re: [Qemu-devel] [PATCH 01/20] tcg-ia64: Use TCGMemOp within qemu_ldst routines
Date: Wed, 6 Nov 2013 23:05:10 +0100 [thread overview]
Message-ID: <20131106220510.GA28960@ohm.rr44.fr> (raw)
In-Reply-To: <20131103225555.GA21123@ohm.rr44.fr>
On Sun, Nov 03, 2013 at 11:55:55PM +0100, Aurelien Jarno wrote:
> On Thu, Oct 31, 2013 at 01:21:50PM -0700, Richard Henderson wrote:
> > Signed-off-by: Richard Henderson <rth@twiddle.net>
> > ---
> > tcg/ia64/tcg-target.c | 171 ++++++++++++++++++++++++++------------------------
> > 1 file changed, 90 insertions(+), 81 deletions(-)
> >
> > diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
> > index 0656d39..e09ad24 100644
> > --- a/tcg/ia64/tcg-target.c
> > +++ b/tcg/ia64/tcg-target.c
> > @@ -1496,7 +1496,7 @@ static inline void tcg_out_movcond(TCGContext *s, TCGCond cond, TCGArg ret,
> > R2 is loaded with the address of the addend TLB entry.
> > R57 is loaded with the address, zero extented on 32-bit targets. */
> > static inline void tcg_out_qemu_tlb(TCGContext *s, TCGArg addr_reg,
> > - int s_bits, uint64_t offset_rw,
> > + TCGMemOp s_bits, uint64_t offset_rw,
> > uint64_t offset_addend)
> > {
> > tcg_out_bundle(s, mII,
> > @@ -1538,23 +1538,24 @@ static const void * const qemu_ld_helpers[4] = {
> > helper_ldq_mmu,
> > };
> >
> > -static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > +static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args,
> > + TCGMemOp opc)
> > {
> > - int addr_reg, data_reg, mem_index, s_bits, bswap;
> > - uint64_t opc_ld_m1[4] = { OPC_LD1_M1, OPC_LD2_M1, OPC_LD4_M1, OPC_LD8_M1 };
> > - uint64_t opc_ext_i29[8] = { OPC_ZXT1_I29, OPC_ZXT2_I29, OPC_ZXT4_I29, 0,
> > - OPC_SXT1_I29, OPC_SXT2_I29, OPC_SXT4_I29, 0 };
> > + static const uint64_t opc_ld_m1[4] = {
> > + OPC_LD1_M1, OPC_LD2_M1, OPC_LD4_M1, OPC_LD8_M1
> > + };
> > + static const uint64_t opc_ext_i29[8] = {
> > + OPC_ZXT1_I29, OPC_ZXT2_I29, OPC_ZXT4_I29, 0,
> > + OPC_SXT1_I29, OPC_SXT2_I29, OPC_SXT4_I29, 0
> > + };
> > + int addr_reg, data_reg, mem_index;
> > + TCGMemOp s_bits, bswap;
> >
> > data_reg = *args++;
> > addr_reg = *args++;
> > mem_index = *args;
> > - s_bits = opc & 3;
> > -
> > -#ifdef TARGET_WORDS_BIGENDIAN
> > - bswap = 1;
> > -#else
> > - bswap = 0;
> > -#endif
> > + s_bits = opc & MO_SIZE;
> > + bswap = opc & MO_BSWAP;
> >
> > /* Read the TLB entry */
> > tcg_out_qemu_tlb(s, addr_reg, s_bits,
> > @@ -1575,14 +1576,14 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > TCG_REG_R3, TCG_REG_R57),
> > tcg_opc_i21(TCG_REG_P7, OPC_MOV_I21, TCG_REG_B6,
> > TCG_REG_R3, 0));
> > - if (bswap && s_bits == 1) {
> > + if (bswap && s_bits == MO_16) {
> > tcg_out_bundle(s, MmI,
> > tcg_opc_m1 (TCG_REG_P6, opc_ld_m1[s_bits],
> > TCG_REG_R8, TCG_REG_R3),
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1, TCG_REG_R1, TCG_REG_R2),
> > tcg_opc_i12(TCG_REG_P6, OPC_DEP_Z_I12,
> > TCG_REG_R8, TCG_REG_R8, 15, 15));
> > - } else if (bswap && s_bits == 2) {
> > + } else if (bswap && s_bits == MO_32) {
> > tcg_out_bundle(s, MmI,
> > tcg_opc_m1 (TCG_REG_P6, opc_ld_m1[s_bits],
> > TCG_REG_R8, TCG_REG_R3),
> > @@ -1596,7 +1597,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1, TCG_REG_R1, TCG_REG_R2),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > }
> > - if (!bswap || s_bits == 0) {
> > + if (!bswap) {
> > tcg_out_bundle(s, miB,
> > tcg_opc_a5 (TCG_REG_P7, OPC_ADDL_A5, TCG_REG_R58,
> > mem_index, TCG_REG_R0),
> > @@ -1613,7 +1614,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > TCG_REG_B0, TCG_REG_B6));
> > }
> >
> > - if (opc == 3) {
> > + if (s_bits == MO_64) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > @@ -1623,7 +1624,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > - tcg_opc_i29(TCG_REG_P0, opc_ext_i29[opc],
> > + tcg_opc_i29(TCG_REG_P0, opc_ext_i29[opc & MO_SSIZE],
> > data_reg, TCG_REG_R8));
> > }
> > }
> > @@ -1637,22 +1638,21 @@ static const void * const qemu_st_helpers[4] = {
> > helper_stq_mmu,
> > };
> >
> > -static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > +static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args,
> > + TCGMemOp opc)
> > {
> > - int addr_reg, data_reg, mem_index, bswap;
> > - uint64_t opc_st_m4[4] = { OPC_ST1_M4, OPC_ST2_M4, OPC_ST4_M4, OPC_ST8_M4 };
> > + static const uint64_t opc_st_m4[4] = {
> > + OPC_ST1_M4, OPC_ST2_M4, OPC_ST4_M4, OPC_ST8_M4
> > + };
> > + int addr_reg, data_reg, mem_index;
> > + TCGMemOp s_bits;
> >
> > data_reg = *args++;
> > addr_reg = *args++;
> > mem_index = *args;
> > + s_bits = opc & MO_SIZE;
> >
> > -#ifdef TARGET_WORDS_BIGENDIAN
> > - bswap = 1;
> > -#else
> > - bswap = 0;
> > -#endif
> > -
> > - tcg_out_qemu_tlb(s, addr_reg, opc,
> > + tcg_out_qemu_tlb(s, addr_reg, s_bits,
> > offsetof(CPUArchState, tlb_table[mem_index][0].addr_write),
> > offsetof(CPUArchState, tlb_table[mem_index][0].addend));
> >
> > @@ -1660,9 +1660,9 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_out_bundle(s, mLX,
> > tcg_opc_a4 (TCG_REG_P7, OPC_ADDS_A4,
> > TCG_REG_R56, 0, TCG_AREG0),
> > - tcg_opc_l2 ((tcg_target_long) qemu_st_helpers[opc]),
> > + tcg_opc_l2 ((tcg_target_long) qemu_st_helpers[s_bits]),
> > tcg_opc_x2 (TCG_REG_P7, OPC_MOVL_X2, TCG_REG_R2,
> > - (tcg_target_long) qemu_st_helpers[opc]));
> > + (tcg_target_long) qemu_st_helpers[s_bits]));
> > tcg_out_bundle(s, MmI,
> > tcg_opc_m3 (TCG_REG_P0, OPC_LD8_M3, TCG_REG_R3,
> > TCG_REG_R2, 8),
> > @@ -1671,14 +1671,20 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i21(TCG_REG_P7, OPC_MOV_I21, TCG_REG_B6,
> > TCG_REG_R3, 0));
> >
> > - if (!bswap || opc == 0) {
> > + switch (opc) {
> > + case MO_8:
> > + case MO_16:
> > + case MO_32:
> > + case MO_64:
> > tcg_out_bundle(s, mii,
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1,
> > TCG_REG_R1, TCG_REG_R2),
> > tcg_opc_a4 (TCG_REG_P7, OPC_ADDS_A4, TCG_REG_R58,
> > 0, data_reg),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > - } else if (opc == 1) {
> > + break;
> > +
> > + case MO_16 | MO_BSWAP:
> > tcg_out_bundle(s, miI,
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1,
> > TCG_REG_R1, TCG_REG_R2),
> > @@ -1692,7 +1698,9 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P6, OPC_MUX1_I3,
> > TCG_REG_R2, TCG_REG_R2, 0xb));
> > data_reg = TCG_REG_R2;
> > - } else if (opc == 2) {
> > + break;
> > +
> > + case MO_32 | MO_BSWAP:
> > tcg_out_bundle(s, miI,
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1,
> > TCG_REG_R1, TCG_REG_R2),
> > @@ -1706,7 +1714,9 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P6, OPC_MUX1_I3,
> > TCG_REG_R2, TCG_REG_R2, 0xb));
> > data_reg = TCG_REG_R2;
> > - } else if (opc == 3) {
> > + break;
> > +
> > + case MO_64 | MO_BSWAP:
> > tcg_out_bundle(s, miI,
> > tcg_opc_m1 (TCG_REG_P7, OPC_LD8_M1,
> > TCG_REG_R1, TCG_REG_R2),
> > @@ -1715,6 +1725,10 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P6, OPC_MUX1_I3,
> > TCG_REG_R2, data_reg, 0xb));
> > data_reg = TCG_REG_R2;
> > + break;
> > +
> > + default:
> > + tcg_abort();
> > }
> >
> > tcg_out_bundle(s, miB,
> > @@ -1728,7 +1742,8 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> >
> > #else /* !CONFIG_SOFTMMU */
> >
> > -static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > +static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args,
> > + TCGMemOp opc)
> > {
> > static uint64_t const opc_ld_m1[4] = {
> > OPC_LD1_M1, OPC_LD2_M1, OPC_LD4_M1, OPC_LD8_M1
> > @@ -1736,17 +1751,13 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > static uint64_t const opc_sxt_i29[4] = {
> > OPC_SXT1_I29, OPC_SXT2_I29, OPC_SXT4_I29, 0
> > };
> > - int addr_reg, data_reg, s_bits, bswap;
> > + int addr_reg, data_reg;
> > + TCGMemOp s_bits, bswap;
> >
> > data_reg = *args++;
> > addr_reg = *args++;
> > - s_bits = opc & 3;
> > -
> > -#ifdef TARGET_WORDS_BIGENDIAN
> > - bswap = 1;
> > -#else
> > - bswap = 0;
> > -#endif
> > + s_bits = opc & MO_SIZE;
> > + bswap = opc & MO_BSWAP;
> >
> > #if TARGET_LONG_BITS == 32
> > if (GUEST_BASE != 0) {
> > @@ -1764,8 +1775,8 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > }
> >
> > - if (!bswap || s_bits == 0) {
> > - if (s_bits == opc) {
> > + if (!bswap) {
> > + if (!(opc & MO_SIGN)) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m1 (TCG_REG_P0, opc_ld_m1[s_bits],
> > data_reg, TCG_REG_R2),
> > @@ -1779,7 +1790,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i29(TCG_REG_P0, opc_sxt_i29[s_bits],
> > data_reg, data_reg));
> > }
> > - } else if (s_bits == 3) {
> > + } else if (s_bits == MO_64) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m1 (TCG_REG_P0, opc_ld_m1[s_bits],
> > data_reg, TCG_REG_R2),
> > @@ -1787,7 +1798,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > data_reg, data_reg, 0xb));
> > } else {
> > - if (s_bits == 1) {
> > + if (s_bits == MO_16) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m1 (TCG_REG_P0, opc_ld_m1[s_bits],
> > data_reg, TCG_REG_R2),
> > @@ -1802,7 +1813,7 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > data_reg, data_reg, 31, 31));
> > }
> > - if (opc == s_bits) {
> > + if (!(opc & MO_SIGN)) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > @@ -1833,28 +1844,28 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > }
> >
> > - if (bswap && s_bits == 1) {
> > + if (bswap && s_bits == MO_16) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > data_reg, data_reg, 15, 15),
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > data_reg, data_reg, 0xb));
> > - } else if (bswap && s_bits == 2) {
> > + } else if (bswap && s_bits == MO_32) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > data_reg, data_reg, 31, 31),
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > data_reg, data_reg, 0xb));
> > - } else if (bswap && s_bits == 3) {
> > + } else if (bswap && s_bits == MO_64) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > data_reg, data_reg, 0xb));
> > }
> > - if (s_bits != opc) {
> > + if (opc & MO_SIGN) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > @@ -1864,24 +1875,22 @@ static inline void tcg_out_qemu_ld(TCGContext *s, const TCGArg *args, int opc)
> > #endif
> > }
> >
> > -static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > +static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args,
> > + TCGMemOp opc)
> > {
> > static uint64_t const opc_st_m4[4] = {
> > OPC_ST1_M4, OPC_ST2_M4, OPC_ST4_M4, OPC_ST8_M4
> > };
> > - int addr_reg, data_reg, bswap;
> > + int addr_reg, data_reg;
> > #if TARGET_LONG_BITS == 64
> > uint64_t add_guest_base;
> > #endif
> > + TCGMemOp s_bits, bswap;
> >
> > data_reg = *args++;
> > addr_reg = *args++;
> > -
> > -#ifdef TARGET_WORDS_BIGENDIAN
> > - bswap = 1;
> > -#else
> > - bswap = 0;
> > -#endif
> > + s_bits = opc & MO_SIZE;
> > + bswap = opc & MO_BSWAP;
> >
> > #if TARGET_LONG_BITS == 32
> > if (GUEST_BASE != 0) {
> > @@ -1900,7 +1909,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > }
> >
> > if (bswap) {
> > - if (opc == 1) {
> > + if (s_bits == MO_16) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > @@ -1908,7 +1917,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > TCG_REG_R3, TCG_REG_R3, 0xb));
> > data_reg = TCG_REG_R3;
> > - } else if (opc == 2) {
> > + } else if (s_bits == MO_32) {
> > tcg_out_bundle(s, mII,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > @@ -1916,7 +1925,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > TCG_REG_R3, TCG_REG_R3, 0xb));
> > data_reg = TCG_REG_R3;
> > - } else if (opc == 3) {
> > + } else if (s_bits == MO_64) {
> > tcg_out_bundle(s, miI,
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > @@ -1926,7 +1935,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > }
> > }
> > tcg_out_bundle(s, mmI,
> > - tcg_opc_m4 (TCG_REG_P0, opc_st_m4[opc],
> > + tcg_opc_m4 (TCG_REG_P0, opc_st_m4[s_bits],
> > data_reg, TCG_REG_R2),
> > tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > @@ -1939,14 +1948,14 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > add_guest_base = tcg_opc_m48(TCG_REG_P0, OPC_NOP_M48, 0);
> > }
> >
> > - if (!bswap || opc == 0) {
> > + if (!bswap) {
> > tcg_out_bundle(s, (GUEST_BASE ? MmI : mmI),
> > add_guest_base,
> > - tcg_opc_m4 (TCG_REG_P0, opc_st_m4[opc],
> > + tcg_opc_m4 (TCG_REG_P0, opc_st_m4[s_bits],
> > data_reg, addr_reg),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > } else {
> > - if (opc == 1) {
> > + if (s_bits == MO_16) {
> > tcg_out_bundle(s, mII,
> > add_guest_base,
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > @@ -1954,7 +1963,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > TCG_REG_R3, TCG_REG_R3, 0xb));
> > data_reg = TCG_REG_R3;
> > - } else if (opc == 2) {
> > + } else if (s_bits == MO_32) {
> > tcg_out_bundle(s, mII,
> > add_guest_base,
> > tcg_opc_i12(TCG_REG_P0, OPC_DEP_Z_I12,
> > @@ -1962,7 +1971,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > tcg_opc_i3 (TCG_REG_P0, OPC_MUX1_I3,
> > TCG_REG_R3, TCG_REG_R3, 0xb));
> > data_reg = TCG_REG_R3;
> > - } else if (opc == 3) {
> > + } else if (s_bits == MO_64) {
> > tcg_out_bundle(s, miI,
> > add_guest_base,
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > @@ -1971,7 +1980,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args, int opc)
> > data_reg = TCG_REG_R3;
> > }
> > tcg_out_bundle(s, miI,
> > - tcg_opc_m4 (TCG_REG_P0, opc_st_m4[opc],
> > + tcg_opc_m4 (TCG_REG_P0, opc_st_m4[s_bits],
> > data_reg, addr_reg),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0),
> > tcg_opc_i18(TCG_REG_P0, OPC_NOP_I18, 0));
> > @@ -2203,39 +2212,39 @@ static inline void tcg_out_op(TCGContext *s, TCGOpcode opc,
> > break;
> >
> > case INDEX_op_qemu_ld8u:
> > - tcg_out_qemu_ld(s, args, 0);
> > + tcg_out_qemu_ld(s, args, MO_UB);
> > break;
> > case INDEX_op_qemu_ld8s:
> > - tcg_out_qemu_ld(s, args, 0 | 4);
> > + tcg_out_qemu_ld(s, args, MO_SB);
> > break;
> > case INDEX_op_qemu_ld16u:
> > - tcg_out_qemu_ld(s, args, 1);
> > + tcg_out_qemu_ld(s, args, MO_TEUW);
> > break;
> > case INDEX_op_qemu_ld16s:
> > - tcg_out_qemu_ld(s, args, 1 | 4);
> > + tcg_out_qemu_ld(s, args, MO_TESW);
> > break;
> > case INDEX_op_qemu_ld32:
> > case INDEX_op_qemu_ld32u:
> > - tcg_out_qemu_ld(s, args, 2);
> > + tcg_out_qemu_ld(s, args, MO_TEUL);
> > break;
> > case INDEX_op_qemu_ld32s:
> > - tcg_out_qemu_ld(s, args, 2 | 4);
> > + tcg_out_qemu_ld(s, args, MO_TESL);
> > break;
> > case INDEX_op_qemu_ld64:
> > - tcg_out_qemu_ld(s, args, 3);
> > + tcg_out_qemu_ld(s, args, MO_TEQ);
> > break;
> >
> > case INDEX_op_qemu_st8:
> > - tcg_out_qemu_st(s, args, 0);
> > + tcg_out_qemu_st(s, args, MO_UB);
> > break;
> > case INDEX_op_qemu_st16:
> > - tcg_out_qemu_st(s, args, 1);
> > + tcg_out_qemu_st(s, args, MO_TEUW);
> > break;
> > case INDEX_op_qemu_st32:
> > - tcg_out_qemu_st(s, args, 2);
> > + tcg_out_qemu_st(s, args, MO_TEUL);
> > break;
> > case INDEX_op_qemu_st64:
> > - tcg_out_qemu_st(s, args, 3);
> > + tcg_out_qemu_st(s, args, MO_TEQ);
> > break;
> >
> > default:
>
> Acked-by: Aurelien Jarno <aurelien@aurel32.net>
>
Actually I spoke too fast, you also need the following patch for big
endian targets:
diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c
index 9ed587a..2d8e00c 100644
--- a/tcg/ia64/tcg-target.c
+++ b/tcg/ia64/tcg-target.c
@@ -1770,7 +1770,7 @@ static inline void tcg_out_qemu_st(TCGContext *s, const TCGArg *args,
}
tcg_out_bundle(s, miB,
- tcg_opc_m4 (TCG_REG_P6, opc_st_m4[opc],
+ tcg_opc_m4 (TCG_REG_P6, opc_st_m4[s_bits],
data_reg, TCG_REG_R3),
tcg_opc_movi_a(TCG_REG_P7, TCG_REG_R59, mem_index),
tcg_opc_b5 (TCG_REG_P7, OPC_BR_CALL_SPTK_MANY_B5,
Please squash it in this patch for the next version.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
aurelien@aurel32.net http://www.aurel32.net
next prev parent reply other threads:[~2013-11-06 22:07 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-31 20:21 [Qemu-devel] [PATCH 00/20] tcg improvements for ia64 Richard Henderson
2013-10-31 20:21 ` [Qemu-devel] [PATCH 01/20] tcg-ia64: Use TCGMemOp within qemu_ldst routines Richard Henderson
2013-11-03 22:55 ` Aurelien Jarno
2013-11-06 22:05 ` Aurelien Jarno [this message]
2013-10-31 20:21 ` [Qemu-devel] [PATCH 02/20] tcg-ia64: Use shortcuts for nop insns Richard Henderson
2013-11-03 22:55 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 03/20] tcg-ia64: Handle constant calls Richard Henderson
2013-11-03 22:56 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 04/20] tcg-ia64: Simplify brcond Richard Henderson
2013-11-03 22:56 ` Aurelien Jarno
2013-11-03 23:34 ` Richard Henderson
2013-11-04 4:24 ` Richard Henderson
2013-11-06 22:05 ` Aurelien Jarno
2013-11-06 22:04 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 05/20] tcg-ia64: Move AREG0 to R32 Richard Henderson
2013-11-06 22:33 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 06/20] tcg-ia64: Avoid unnecessary stop bit in tcg_out_alu Richard Henderson
2013-11-06 22:33 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 07/20] tcg-ia64: Use ADDS for small addition Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 08/20] tcg-ia64: Use SUB_A3 and ADDS_A4 for subtraction Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 09/20] tcg-ia64: Use A3 form of logical operations Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:21 ` [Qemu-devel] [PATCH 10/20] tcg-ia64: Introduce tcg_opc_mov_a Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:22 ` [Qemu-devel] [PATCH 11/20] tcg-ia64: Introduce tcg_opc_movi_a Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:22 ` [Qemu-devel] [PATCH 12/20] tcg-ia64: Introduce tcg_opc_ext_i Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:22 ` [Qemu-devel] [PATCH 13/20] tcg-ia64: Introduce tcg_opc_bswap64_i Richard Henderson
2013-11-06 22:34 ` Aurelien Jarno
2013-10-31 20:22 ` [Qemu-devel] [PATCH 14/20] tcg-ia64: Re-bundle the tlb load Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 15/20] tcg-ia64: Move bswap for store into " Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 16/20] tcg-ia64: Move tlb addend load into tlb read Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 17/20] tcg-ia64: Reduce code duplication in tcg_out_qemu_ld Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 18/20] tcg-ia64: Convert to new ldst helpers Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 19/20] tcg-ia64: Move part of softmmu slow path out of line Richard Henderson
2013-10-31 20:22 ` [Qemu-devel] [PATCH 20/20] tcg-ia64: Optimize small arguments to exit_tb Richard Henderson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131106220510.GA28960@ohm.rr44.fr \
--to=aurelien@aurel32.net \
--cc=aliguori@amazon.com \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).