[gomp4] rewrite simd clone argument adjustment to avoid regimplification

Jakub Jelinek jakub@redhat.com
Thu Nov 7 15:48:00 GMT 2013


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



More information about the Gcc-patches mailing list