On 09/27/2012 10:15 AM, Aurelien Jarno wrote:
> @@ -1706,11 +1718,9 @@ static void tcg_reg_alloc_mov(TCGContext *s, const
> TCGOpDef *def,
> if (!ots->mem_allocated) {
> temp_allocate_frame(s, args[0]);
> }
> - if (ts->val_type == TEMP_VAL_REG) {
> - tcg_out_st(s, ots->type, ts->reg, ots->mem_reg, ots->mem_offset);
> - if (IS_DEAD_ARG(1)) {
> - temp_dead(s, args[1]);
> - }
> + tcg_out_st(s, ots->type, ts->reg, ots->mem_reg, ots->mem_offset);
> + if (IS_DEAD_ARG(1)) {
> + temp_dead(s, args[1]);
> }
> temp_dead(s, args[0]);
> } else if (ts->val_type == TEMP_VAL_CONST) {
Did this hunk belong to a different patch? It seems like it belongs
with the tcg_reg_alloc_mov rewrite.
If it actually depends on patches 7-8, then perhaps a reorder is better.
r~
