On Mon, Oct 17, 2011 at 08:49:34AM +0200, Richard Guenther wrote:
> On Sun, Oct 16, 2011 at 5:47 PM, Jakub Jelinek <ja...@redhat.com> wrote:
> I think this should be exactly the other way around, using create_tmp_var,
> copying TREE_ADDRESSABLE and setting DECL_GIMPLE_REG_P if it is not 
> addressable.
> 
> Ok with that change.

Here is what I've committed after bootstrap/regtest on x86_64-linux and
i686-linux:

2011-10-18  Jakub Jelinek  <ja...@redhat.com>

        PR tree-optimization/50735
        * function.c (gimplify_parameters): Use create_tmp_var instead of
        create_tmp_reg.  If parm is not TREE_ADDRESSABLE and type is complex
        or vector type, set DECL_GIMPLE_REG_P.

--- gcc/function.c.jj   2011-10-17 09:20:38.000000000 +0200
+++ gcc/function.c      2011-10-18 13:54:02.000000000 +0200
@@ -3617,7 +3617,7 @@ gimplify_parameters (void)
                       && compare_tree_int (DECL_SIZE_UNIT (parm),
                                            STACK_CHECK_MAX_VAR_SIZE) > 0))
                {
-                 local = create_tmp_reg (type, get_name (parm));
+                 local = create_tmp_var (type, get_name (parm));
                  DECL_IGNORED_P (local) = 0;
                  /* If PARM was addressable, move that flag over
                     to the local copy, as its address will be taken,
@@ -3625,6 +3625,9 @@ gimplify_parameters (void)
                     as we'll query that flag during gimplification.  */
                  if (TREE_ADDRESSABLE (parm))
                    TREE_ADDRESSABLE (local) = 1;
+                 else if (TREE_CODE (type) == COMPLEX_TYPE
+                          || TREE_CODE (type) == VECTOR_TYPE)
+                   DECL_GIMPLE_REG_P (local) = 1;
                }
              else
                {

        Jakub

Reply via email to