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