tcg: Propagate TCGOp down to allocators
Reviewed-by: Emilio G. Cota <cota@braap.org> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Richard Henderson <rth@twiddle.net>
This commit is contained in:
parent
efee3746fa
commit
dd18629201
75
tcg/tcg.c
75
tcg/tcg.c
@ -2334,25 +2334,24 @@ static void tcg_reg_alloc_do_movi(TCGContext *s, TCGTemp *ots,
|
||||
}
|
||||
}
|
||||
|
||||
static void tcg_reg_alloc_movi(TCGContext *s, const TCGArg *args,
|
||||
TCGLifeData arg_life)
|
||||
static void tcg_reg_alloc_movi(TCGContext *s, const TCGOp *op)
|
||||
{
|
||||
TCGTemp *ots = &s->temps[args[0]];
|
||||
tcg_target_ulong val = args[1];
|
||||
TCGTemp *ots = &s->temps[op->args[0]];
|
||||
tcg_target_ulong val = op->args[1];
|
||||
|
||||
tcg_reg_alloc_do_movi(s, ots, val, arg_life);
|
||||
tcg_reg_alloc_do_movi(s, ots, val, op->life);
|
||||
}
|
||||
|
||||
static void tcg_reg_alloc_mov(TCGContext *s, const TCGOpDef *def,
|
||||
const TCGArg *args, TCGLifeData arg_life)
|
||||
static void tcg_reg_alloc_mov(TCGContext *s, const TCGOp *op)
|
||||
{
|
||||
const TCGLifeData arg_life = op->life;
|
||||
TCGRegSet allocated_regs;
|
||||
TCGTemp *ts, *ots;
|
||||
TCGType otype, itype;
|
||||
|
||||
allocated_regs = s->reserved_regs;
|
||||
ots = &s->temps[args[0]];
|
||||
ts = &s->temps[args[1]];
|
||||
ots = &s->temps[op->args[0]];
|
||||
ts = &s->temps[op->args[1]];
|
||||
|
||||
/* Note that otype != itype for no-op truncation. */
|
||||
otype = ots->type;
|
||||
@ -2382,7 +2381,7 @@ static void tcg_reg_alloc_mov(TCGContext *s, const TCGOpDef *def,
|
||||
liveness analysis disabled). */
|
||||
tcg_debug_assert(NEED_SYNC_ARG(0));
|
||||
if (!ots->mem_allocated) {
|
||||
temp_allocate_frame(s, args[0]);
|
||||
temp_allocate_frame(s, op->args[0]);
|
||||
}
|
||||
tcg_out_st(s, otype, ts->reg, ots->mem_base->reg, ots->mem_offset);
|
||||
if (IS_DEAD_ARG(1)) {
|
||||
@ -2416,10 +2415,10 @@ static void tcg_reg_alloc_mov(TCGContext *s, const TCGOpDef *def,
|
||||
}
|
||||
}
|
||||
|
||||
static void tcg_reg_alloc_op(TCGContext *s,
|
||||
const TCGOpDef *def, TCGOpcode opc,
|
||||
const TCGArg *args, TCGLifeData arg_life)
|
||||
static void tcg_reg_alloc_op(TCGContext *s, const TCGOp *op)
|
||||
{
|
||||
const TCGLifeData arg_life = op->life;
|
||||
const TCGOpDef * const def = &tcg_op_defs[op->opc];
|
||||
TCGRegSet i_allocated_regs;
|
||||
TCGRegSet o_allocated_regs;
|
||||
int i, k, nb_iargs, nb_oargs;
|
||||
@ -2435,16 +2434,16 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
|
||||
/* copy constants */
|
||||
memcpy(new_args + nb_oargs + nb_iargs,
|
||||
args + nb_oargs + nb_iargs,
|
||||
op->args + nb_oargs + nb_iargs,
|
||||
sizeof(TCGArg) * def->nb_cargs);
|
||||
|
||||
i_allocated_regs = s->reserved_regs;
|
||||
o_allocated_regs = s->reserved_regs;
|
||||
|
||||
/* satisfy input constraints */
|
||||
for(k = 0; k < nb_iargs; k++) {
|
||||
for (k = 0; k < nb_iargs; k++) {
|
||||
i = def->sorted_args[nb_oargs + k];
|
||||
arg = args[i];
|
||||
arg = op->args[i];
|
||||
arg_ct = &def->args_ct[i];
|
||||
ts = &s->temps[arg];
|
||||
|
||||
@ -2462,7 +2461,7 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
if (ts->fixed_reg) {
|
||||
/* if fixed register, we must allocate a new register
|
||||
if the alias is not the same register */
|
||||
if (arg != args[arg_ct->alias_index])
|
||||
if (arg != op->args[arg_ct->alias_index])
|
||||
goto allocate_in_reg;
|
||||
} else {
|
||||
/* if the input is aliased to an output and if it is
|
||||
@ -2503,7 +2502,7 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
/* mark dead temporaries and free the associated registers */
|
||||
for (i = nb_oargs; i < nb_oargs + nb_iargs; i++) {
|
||||
if (IS_DEAD_ARG(i)) {
|
||||
temp_dead(s, &s->temps[args[i]]);
|
||||
temp_dead(s, &s->temps[op->args[i]]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -2527,7 +2526,7 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
/* satisfy the output constraints */
|
||||
for(k = 0; k < nb_oargs; k++) {
|
||||
i = def->sorted_args[k];
|
||||
arg = args[i];
|
||||
arg = op->args[i];
|
||||
arg_ct = &def->args_ct[i];
|
||||
ts = &s->temps[arg];
|
||||
if ((arg_ct->ct & TCG_CT_ALIAS)
|
||||
@ -2566,11 +2565,11 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
}
|
||||
|
||||
/* emit instruction */
|
||||
tcg_out_op(s, opc, new_args, const_args);
|
||||
tcg_out_op(s, op->opc, new_args, const_args);
|
||||
|
||||
/* move the outputs in the correct register if needed */
|
||||
for(i = 0; i < nb_oargs; i++) {
|
||||
ts = &s->temps[args[i]];
|
||||
ts = &s->temps[op->args[i]];
|
||||
reg = new_args[i];
|
||||
if (ts->fixed_reg && ts->reg != reg) {
|
||||
tcg_out_mov(s, ts->type, ts->reg, reg);
|
||||
@ -2589,9 +2588,11 @@ static void tcg_reg_alloc_op(TCGContext *s,
|
||||
#define STACK_DIR(x) (x)
|
||||
#endif
|
||||
|
||||
static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
const TCGArg * const args, TCGLifeData arg_life)
|
||||
static void tcg_reg_alloc_call(TCGContext *s, TCGOp *op)
|
||||
{
|
||||
const int nb_oargs = op->callo;
|
||||
const int nb_iargs = op->calli;
|
||||
const TCGLifeData arg_life = op->life;
|
||||
int flags, nb_regs, i;
|
||||
TCGReg reg;
|
||||
TCGArg arg;
|
||||
@ -2602,8 +2603,8 @@ static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
int allocate_args;
|
||||
TCGRegSet allocated_regs;
|
||||
|
||||
func_addr = (tcg_insn_unit *)(intptr_t)args[nb_oargs + nb_iargs];
|
||||
flags = args[nb_oargs + nb_iargs + 1];
|
||||
func_addr = (tcg_insn_unit *)(intptr_t)op->args[nb_oargs + nb_iargs];
|
||||
flags = op->args[nb_oargs + nb_iargs + 1];
|
||||
|
||||
nb_regs = ARRAY_SIZE(tcg_target_call_iarg_regs);
|
||||
if (nb_regs > nb_iargs) {
|
||||
@ -2622,8 +2623,8 @@ static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
}
|
||||
|
||||
stack_offset = TCG_TARGET_CALL_STACK_OFFSET;
|
||||
for(i = nb_regs; i < nb_iargs; i++) {
|
||||
arg = args[nb_oargs + i];
|
||||
for (i = nb_regs; i < nb_iargs; i++) {
|
||||
arg = op->args[nb_oargs + i];
|
||||
#ifdef TCG_TARGET_STACK_GROWSUP
|
||||
stack_offset -= sizeof(tcg_target_long);
|
||||
#endif
|
||||
@ -2640,8 +2641,8 @@ static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
|
||||
/* assign input registers */
|
||||
allocated_regs = s->reserved_regs;
|
||||
for(i = 0; i < nb_regs; i++) {
|
||||
arg = args[nb_oargs + i];
|
||||
for (i = 0; i < nb_regs; i++) {
|
||||
arg = op->args[nb_oargs + i];
|
||||
if (arg != TCG_CALL_DUMMY_ARG) {
|
||||
ts = &s->temps[arg];
|
||||
reg = tcg_target_call_iarg_regs[i];
|
||||
@ -2663,9 +2664,9 @@ static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
}
|
||||
|
||||
/* mark dead temporaries and free the associated registers */
|
||||
for(i = nb_oargs; i < nb_iargs + nb_oargs; i++) {
|
||||
for (i = nb_oargs; i < nb_iargs + nb_oargs; i++) {
|
||||
if (IS_DEAD_ARG(i)) {
|
||||
temp_dead(s, &s->temps[args[i]]);
|
||||
temp_dead(s, &s->temps[op->args[i]]);
|
||||
}
|
||||
}
|
||||
|
||||
@ -2690,7 +2691,7 @@ static void tcg_reg_alloc_call(TCGContext *s, int nb_oargs, int nb_iargs,
|
||||
|
||||
/* assign output registers and emit moves if needed */
|
||||
for(i = 0; i < nb_oargs; i++) {
|
||||
arg = args[i];
|
||||
arg = op->args[i];
|
||||
ts = &s->temps[arg];
|
||||
reg = tcg_target_call_oarg_regs[i];
|
||||
tcg_debug_assert(s->reg_to_temp[reg] == NULL);
|
||||
@ -2838,8 +2839,6 @@ int tcg_gen_code(TCGContext *s, TranslationBlock *tb)
|
||||
for (oi = s->gen_op_buf[0].next; oi != 0; oi = oi_next) {
|
||||
TCGOp * const op = &s->gen_op_buf[oi];
|
||||
TCGOpcode opc = op->opc;
|
||||
const TCGOpDef *def = &tcg_op_defs[opc];
|
||||
TCGLifeData arg_life = op->life;
|
||||
|
||||
oi_next = op->next;
|
||||
#ifdef CONFIG_PROFILER
|
||||
@ -2849,11 +2848,11 @@ int tcg_gen_code(TCGContext *s, TranslationBlock *tb)
|
||||
switch (opc) {
|
||||
case INDEX_op_mov_i32:
|
||||
case INDEX_op_mov_i64:
|
||||
tcg_reg_alloc_mov(s, def, op->args, arg_life);
|
||||
tcg_reg_alloc_mov(s, op);
|
||||
break;
|
||||
case INDEX_op_movi_i32:
|
||||
case INDEX_op_movi_i64:
|
||||
tcg_reg_alloc_movi(s, op->args, arg_life);
|
||||
tcg_reg_alloc_movi(s, op);
|
||||
break;
|
||||
case INDEX_op_insn_start:
|
||||
if (num_insns >= 0) {
|
||||
@ -2878,7 +2877,7 @@ int tcg_gen_code(TCGContext *s, TranslationBlock *tb)
|
||||
tcg_out_label(s, arg_label(op->args[0]), s->code_ptr);
|
||||
break;
|
||||
case INDEX_op_call:
|
||||
tcg_reg_alloc_call(s, op->callo, op->calli, op->args, arg_life);
|
||||
tcg_reg_alloc_call(s, op);
|
||||
break;
|
||||
default:
|
||||
/* Sanity check that we've not introduced any unhandled opcodes. */
|
||||
@ -2886,7 +2885,7 @@ int tcg_gen_code(TCGContext *s, TranslationBlock *tb)
|
||||
/* Note: in order to speed up the code, it would be much
|
||||
faster to have specialized register allocator functions for
|
||||
some common argument patterns */
|
||||
tcg_reg_alloc_op(s, def, opc, op->args, arg_life);
|
||||
tcg_reg_alloc_op(s, op);
|
||||
break;
|
||||
}
|
||||
#ifdef CONFIG_DEBUG_TCG
|
||||
|
Loading…
x
Reference in New Issue
Block a user