On Thu, Nov 07, 2013 at 08:17:13AM -0700, Aldy Hernandez wrote:
> But as discussed on IRC, I wonder whether we can do without the
> following in the attached patch:
> 
> +                 tree repl = make_ssa_name (TREE_TYPE (retval), NULL);
> +                 stmt = gimple_build_assign (repl, retval);
> +                 gsi_insert_before (&gsi, stmt, GSI_SAME_STMT);
> +                 stmt = gimple_build_assign (ref, repl);
> 
> ...and unconditionally do:
> 
> +               stmt = gimple_build_assign (ref, retval);

This should be sufficient.
> 
> ...since it seems all the GIMPLE_RETURNs I see can be replaced by
> ``retval_array[iter] = whatever'' without creating something
> non-gimple (thus avoiding an SSA variable).
> 
> Either way, I'm ok.  Let me know.

Thanks.

        Jakub

Reply via email to