tcg: Introduce tcg_type_size

Add a helper function for computing the size of a type.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Richard Henderson 2022-10-19 12:41:15 +10:00
parent 89496a85b4
commit 31c9641746
2 changed files with 28 additions and 15 deletions

View File

@ -319,6 +319,22 @@ typedef enum TCGType {
#endif #endif
} TCGType; } TCGType;
/**
* tcg_type_size
* @t: type
*
* Return the size of the type in bytes.
*/
static inline int tcg_type_size(TCGType t)
{
unsigned i = t;
if (i >= TCG_TYPE_V64) {
tcg_debug_assert(i < TCG_TYPE_COUNT);
i -= TCG_TYPE_V64 - 1;
}
return 4 << i;
}
/** /**
* get_alignment_bits * get_alignment_bits
* @memop: MemOp value * @memop: MemOp value

View File

@ -3038,22 +3038,22 @@ static bool liveness_pass_2(TCGContext *s)
static void temp_allocate_frame(TCGContext *s, TCGTemp *ts) static void temp_allocate_frame(TCGContext *s, TCGTemp *ts)
{ {
intptr_t off, size, align; int size = tcg_type_size(ts->type);
int align;
intptr_t off;
switch (ts->type) { switch (ts->type) {
case TCG_TYPE_I32: case TCG_TYPE_I32:
size = align = 4; align = 4;
break; break;
case TCG_TYPE_I64: case TCG_TYPE_I64:
case TCG_TYPE_V64: case TCG_TYPE_V64:
size = align = 8; align = 8;
break; break;
case TCG_TYPE_V128: case TCG_TYPE_V128:
size = align = 16;
break;
case TCG_TYPE_V256: case TCG_TYPE_V256:
/* Note that we do not require aligned storage for V256. */ /* Note that we do not require aligned storage for V256. */
size = 32, align = 16; align = 16;
break; break;
default: default:
g_assert_not_reached(); g_assert_not_reached();
@ -3593,8 +3593,8 @@ static void tcg_reg_alloc_dup(TCGContext *s, const TCGOp *op)
TCGRegSet dup_out_regs, dup_in_regs; TCGRegSet dup_out_regs, dup_in_regs;
TCGTemp *its, *ots; TCGTemp *its, *ots;
TCGType itype, vtype; TCGType itype, vtype;
intptr_t endian_fixup;
unsigned vece; unsigned vece;
int lowpart_ofs;
bool ok; bool ok;
ots = arg_temp(op->args[0]); ots = arg_temp(op->args[0]);
@ -3662,15 +3662,12 @@ static void tcg_reg_alloc_dup(TCGContext *s, const TCGOp *op)
/* fall through */ /* fall through */
case TEMP_VAL_MEM: case TEMP_VAL_MEM:
#if HOST_BIG_ENDIAN lowpart_ofs = 0;
endian_fixup = itype == TCG_TYPE_I32 ? 4 : 8; if (HOST_BIG_ENDIAN) {
endian_fixup -= 1 << vece; lowpart_ofs = tcg_type_size(itype) - (1 << vece);
#else }
endian_fixup = 0;
#endif
/* Attempt to dup directly from the input memory slot. */
if (tcg_out_dupm_vec(s, vtype, vece, ots->reg, its->mem_base->reg, if (tcg_out_dupm_vec(s, vtype, vece, ots->reg, its->mem_base->reg,
its->mem_offset + endian_fixup)) { its->mem_offset + lowpart_ofs)) {
goto done; goto done;
} }
/* Load the input into the destination vector register. */ /* Load the input into the destination vector register. */