tcg/optimize: Copy mask writeback to fold_masks
Use of fold_masks should be restricted to those opcodes that can reliably make use of it -- those with a single output, and from higher-level folders that set up the masks. Prepare for conversion of each folder in turn. Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
parent
045ace35a8
commit
56e06ecfa5
@ -1048,6 +1048,12 @@ static bool fold_masks(OptContext *ctx, TCGOp *op)
|
|||||||
{
|
{
|
||||||
uint64_t z_mask = ctx->z_mask;
|
uint64_t z_mask = ctx->z_mask;
|
||||||
uint64_t s_mask = ctx->s_mask;
|
uint64_t s_mask = ctx->s_mask;
|
||||||
|
const TCGOpDef *def = &tcg_op_defs[op->opc];
|
||||||
|
TCGTemp *ts;
|
||||||
|
TempOptInfo *ti;
|
||||||
|
|
||||||
|
/* Only single-output opcodes are supported here. */
|
||||||
|
tcg_debug_assert(def->nb_oargs == 1);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* 32-bit ops generate 32-bit results, which for the purpose of
|
* 32-bit ops generate 32-bit results, which for the purpose of
|
||||||
@ -1059,14 +1065,19 @@ static bool fold_masks(OptContext *ctx, TCGOp *op)
|
|||||||
if (ctx->type == TCG_TYPE_I32) {
|
if (ctx->type == TCG_TYPE_I32) {
|
||||||
z_mask = (int32_t)z_mask;
|
z_mask = (int32_t)z_mask;
|
||||||
s_mask |= MAKE_64BIT_MASK(32, 32);
|
s_mask |= MAKE_64BIT_MASK(32, 32);
|
||||||
ctx->z_mask = z_mask;
|
|
||||||
ctx->s_mask = s_mask;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (z_mask == 0) {
|
if (z_mask == 0) {
|
||||||
return tcg_opt_gen_movi(ctx, op, op->args[0], 0);
|
return tcg_opt_gen_movi(ctx, op, op->args[0], 0);
|
||||||
}
|
}
|
||||||
return false;
|
|
||||||
|
ts = arg_temp(op->args[0]);
|
||||||
|
reset_ts(ctx, ts);
|
||||||
|
|
||||||
|
ti = ts_info(ts);
|
||||||
|
ti->z_mask = z_mask;
|
||||||
|
ti->s_mask = s_mask;
|
||||||
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
Loading…
x
Reference in New Issue
Block a user