tcg/optimize: Return true from fold_qemu_st, fold_tcg_st
Stores have no output operands, and so need no further work. Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
parent
6813be9b9b
commit
082b3ef919
@ -2143,7 +2143,7 @@ static bool fold_qemu_st(OptContext *ctx, TCGOp *op)
|
|||||||
{
|
{
|
||||||
/* Opcodes that touch guest memory stop the mb optimization. */
|
/* Opcodes that touch guest memory stop the mb optimization. */
|
||||||
ctx->prev_mb = NULL;
|
ctx->prev_mb = NULL;
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static bool fold_remainder(OptContext *ctx, TCGOp *op)
|
static bool fold_remainder(OptContext *ctx, TCGOp *op)
|
||||||
@ -2706,7 +2706,7 @@ static bool fold_tcg_st(OptContext *ctx, TCGOp *op)
|
|||||||
|
|
||||||
if (op->args[1] != tcgv_ptr_arg(tcg_env)) {
|
if (op->args[1] != tcgv_ptr_arg(tcg_env)) {
|
||||||
remove_mem_copy_all(ctx);
|
remove_mem_copy_all(ctx);
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
switch (op->opc) {
|
switch (op->opc) {
|
||||||
@ -2730,7 +2730,7 @@ static bool fold_tcg_st(OptContext *ctx, TCGOp *op)
|
|||||||
g_assert_not_reached();
|
g_assert_not_reached();
|
||||||
}
|
}
|
||||||
remove_mem_copy_in(ctx, ofs, ofs + lm1);
|
remove_mem_copy_in(ctx, ofs, ofs + lm1);
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static bool fold_tcg_st_memcopy(OptContext *ctx, TCGOp *op)
|
static bool fold_tcg_st_memcopy(OptContext *ctx, TCGOp *op)
|
||||||
@ -2740,8 +2740,7 @@ static bool fold_tcg_st_memcopy(OptContext *ctx, TCGOp *op)
|
|||||||
TCGType type;
|
TCGType type;
|
||||||
|
|
||||||
if (op->args[1] != tcgv_ptr_arg(tcg_env)) {
|
if (op->args[1] != tcgv_ptr_arg(tcg_env)) {
|
||||||
fold_tcg_st(ctx, op);
|
return fold_tcg_st(ctx, op);
|
||||||
return false;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
src = arg_temp(op->args[0]);
|
src = arg_temp(op->args[0]);
|
||||||
@ -2763,7 +2762,7 @@ static bool fold_tcg_st_memcopy(OptContext *ctx, TCGOp *op)
|
|||||||
last = ofs + tcg_type_size(type) - 1;
|
last = ofs + tcg_type_size(type) - 1;
|
||||||
remove_mem_copy_in(ctx, ofs, last);
|
remove_mem_copy_in(ctx, ofs, last);
|
||||||
record_mem_copy(ctx, type, src, ofs, last);
|
record_mem_copy(ctx, type, src, ofs, last);
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static bool fold_xor(OptContext *ctx, TCGOp *op)
|
static bool fold_xor(OptContext *ctx, TCGOp *op)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user