OSDN Git Service

2009-04-10 Paolo Bonzini <bonzini@gnu.org>
[pf3gnuchains/gcc-fork.git] / gcc / tree-ssa.c
index 74968bf..ba0c6ab 100644 (file)
@@ -1,6 +1,6 @@
 /* Miscellaneous SSA utility functions.
-   Copyright (C) 2001, 2002, 2003, 2004, 2005, 2007, 2008 Free Software
-   Foundation, Inc.
+   Copyright (C) 2001, 2002, 2003, 2004, 2005, 2007, 2008, 2009
+   Free Software Foundation, Inc.
 
 This file is part of GCC.
 
@@ -112,7 +112,8 @@ redirect_edge_var_map_clear (edge e)
 void
 redirect_edge_var_map_dup (edge newe, edge olde)
 {
-  void **new_slot, **old_slot; edge_var_map_vector head;
+  void **new_slot, **old_slot;
+  edge_var_map_vector head;
 
   if (!edge_var_maps)
     return;
@@ -149,6 +150,17 @@ redirect_edge_var_map_vector (edge e)
   return (edge_var_map_vector) *slot;
 }
 
+/* Used by redirect_edge_var_map_destroy to free all memory.  */
+
+static bool
+free_var_map_entry (const void *key ATTRIBUTE_UNUSED,
+                   void **value,
+                   void *data ATTRIBUTE_UNUSED)
+{
+  edge_var_map_vector head = (edge_var_map_vector) *value;
+  VEC_free (edge_var_map, heap, head);
+  return true;
+}
 
 /* Clear the edge variable mappings.  */
 
@@ -157,6 +169,7 @@ redirect_edge_var_map_destroy (void)
 {
   if (edge_var_maps)
     {
+      pointer_map_traverse (edge_var_maps, free_var_map_entry, NULL);
       pointer_map_destroy (edge_var_maps);
       edge_var_maps = NULL;
     }
@@ -258,6 +271,12 @@ verify_ssa_name (tree ssa_name, bool is_virtual)
       return true;
     }
 
+  if (is_virtual && SSA_NAME_VAR (ssa_name) != gimple_vop (cfun))
+    {
+      error ("virtual SSA name for non-VOP decl");
+      return true;
+    }
+
   if (!is_virtual && !is_gimple_reg (ssa_name))
     {
       error ("found a real definition for a non-register");
@@ -465,6 +484,21 @@ verify_phi_args (gimple phi, basic_block bb, basic_block *definition_block)
                             op_p, phi, e->flags & EDGE_ABNORMAL, NULL);
        }
 
+      if (TREE_CODE (op) == ADDR_EXPR)
+       {
+         tree base = TREE_OPERAND (op, 0);
+         while (handled_component_p (base))
+           base = TREE_OPERAND (base, 0);
+         if ((TREE_CODE (base) == VAR_DECL
+              || TREE_CODE (base) == PARM_DECL
+              || TREE_CODE (base) == RESULT_DECL)
+             && !TREE_ADDRESSABLE (base))
+           {
+             error ("address taken, but ADDRESSABLE bit not set");
+             err = true;
+           }
+       }
+
       if (e->dest != bb)
        {
          error ("wrong edge %d->%d for PHI argument",
@@ -492,232 +526,6 @@ error:
 }
 
 
-static void
-verify_flow_insensitive_alias_info (void)
-{
-  tree var;
-  referenced_var_iterator rvi;
-
-  FOR_EACH_REFERENCED_VAR (var, rvi)
-    {
-      unsigned int j;
-      bitmap aliases;
-      tree alias;
-      bitmap_iterator bi;
-
-      if (!MTAG_P (var) || !MTAG_ALIASES (var))
-       continue;
-      
-      aliases = MTAG_ALIASES (var);
-
-      EXECUTE_IF_SET_IN_BITMAP (aliases, 0, j, bi)
-       {
-         alias = referenced_var (j);
-
-         if (TREE_CODE (alias) != MEMORY_PARTITION_TAG
-             && !may_be_aliased (alias))
-           {
-             error ("non-addressable variable inside an alias set");
-             debug_variable (alias);
-             goto err;
-           }
-       }
-    }
-
-  return;
-
-err:
-  debug_variable (var);
-  internal_error ("verify_flow_insensitive_alias_info failed");
-}
-
-
-static void
-verify_flow_sensitive_alias_info (void)
-{
-  size_t i;
-  tree ptr;
-
-  for (i = 1; i < num_ssa_names; i++)
-    {
-      tree var;
-      var_ann_t ann;
-      struct ptr_info_def *pi;
-
-      ptr = ssa_name (i);
-      if (!ptr)
-       continue;
-
-      /* We only care for pointers that are actually referenced in the
-        program.  */
-      if (!POINTER_TYPE_P (TREE_TYPE (ptr)) || !TREE_VISITED (ptr))
-       continue;
-
-      /* RESULT_DECL is special.  If it's a GIMPLE register, then it
-        is only written-to only once in the return statement.
-        Otherwise, aggregate RESULT_DECLs may be written-to more than
-        once in virtual operands.  */
-      var = SSA_NAME_VAR (ptr);
-      if (TREE_CODE (var) == RESULT_DECL
-         && is_gimple_reg (ptr))
-       continue;
-
-      pi = SSA_NAME_PTR_INFO (ptr);
-      if (pi == NULL)
-       continue;
-
-      ann = var_ann (var);
-      if (pi->memory_tag_needed && !pi->name_mem_tag && !ann->symbol_mem_tag)
-       {
-         error ("dereferenced pointers should have a name or a symbol tag");
-         goto err;
-       }
-
-      if (pi->name_mem_tag
-         && (pi->pt_vars == NULL || bitmap_empty_p (pi->pt_vars)))
-       {
-         error ("pointers with a memory tag, should have points-to sets");
-         goto err;
-       }
-
-      if (pi->value_escapes_p
-         && pi->escape_mask & ~ESCAPE_TO_RETURN
-         && pi->name_mem_tag)
-       {
-         tree t = memory_partition (pi->name_mem_tag);
-         if (t == NULL_TREE)
-           t = pi->name_mem_tag;
-         
-         if (!is_call_clobbered (t))
-           {
-             error ("pointer escapes but its name tag is not call-clobbered");
-             goto err;
-           }
-       }
-    }
-
-  return;
-
-err:
-  debug_variable (ptr);
-  internal_error ("verify_flow_sensitive_alias_info failed");
-}
-
-
-/* Verify the consistency of call clobbering information.  */
-
-static void
-verify_call_clobbering (void)
-{
-  unsigned int i;
-  bitmap_iterator bi;
-  tree var;
-  referenced_var_iterator rvi;
-
-  /* At all times, the result of the call_clobbered flag should
-     match the result of the call_clobbered_vars bitmap.  Verify both
-     that everything in call_clobbered_vars is marked
-     call_clobbered, and that everything marked
-     call_clobbered is in call_clobbered_vars.  */
-  EXECUTE_IF_SET_IN_BITMAP (gimple_call_clobbered_vars (cfun), 0, i, bi)
-    {
-      var = referenced_var (i);
-
-      if (memory_partition (var))
-       var = memory_partition (var);
-
-      if (!MTAG_P (var) && !var_ann (var)->call_clobbered)
-       {
-         error ("variable in call_clobbered_vars but not marked "
-                "call_clobbered");
-         debug_variable (var);
-         goto err;
-       }
-    }
-
-  FOR_EACH_REFERENCED_VAR (var, rvi)
-    {
-      if (is_gimple_reg (var))
-       continue;
-
-      if (memory_partition (var))
-       var = memory_partition (var);
-
-      if (!MTAG_P (var)
-         && var_ann (var)->call_clobbered
-         && !bitmap_bit_p (gimple_call_clobbered_vars (cfun), DECL_UID (var)))
-       {
-         error ("variable marked call_clobbered but not in "
-                "call_clobbered_vars bitmap.");
-         debug_variable (var);
-         goto err;
-       }
-    }
-
-  return;
-
- err:
-    internal_error ("verify_call_clobbering failed");
-}
-
-
-/* Verify invariants in memory partitions.  */
-
-static void
-verify_memory_partitions (void)
-{
-  unsigned i;
-  tree mpt;
-  VEC(tree,heap) *mpt_table = gimple_ssa_operands (cfun)->mpt_table;
-  struct pointer_set_t *partitioned_syms = pointer_set_create ();
-
-  for (i = 0; VEC_iterate (tree, mpt_table, i, mpt); i++)
-    {
-      unsigned j;
-      bitmap_iterator bj;
-
-      if (MPT_SYMBOLS (mpt) == NULL)
-       {
-         error ("Memory partitions should have at least one symbol");
-         debug_variable (mpt);
-         goto err;
-       }
-
-      EXECUTE_IF_SET_IN_BITMAP (MPT_SYMBOLS (mpt), 0, j, bj)
-       {
-         tree var = referenced_var (j);
-         if (pointer_set_insert (partitioned_syms, var))
-           {
-             error ("Partitioned symbols should belong to exactly one "
-                    "partition");
-             debug_variable (var);
-             goto err;
-           }
-       }
-    }
-
-  pointer_set_destroy (partitioned_syms);
-
-  return;
-
-err:
-  internal_error ("verify_memory_partitions failed");
-}
-
-
-/* Verify the consistency of aliasing information.  */
-
-static void
-verify_alias_info (void)
-{
-  verify_flow_sensitive_alias_info ();
-  verify_call_clobbering ();
-  verify_flow_insensitive_alias_info ();
-  verify_memory_partitions ();
-}
-
-
 /* Verify common invariants in the SSA web.
    TODO: verify the variable annotations.  */
 
@@ -732,7 +540,7 @@ verify_ssa (bool check_modified_stmt)
   enum dom_state orig_dom_state = dom_info_state (CDI_DOMINATORS);
   bitmap names_defined_in_bb = BITMAP_ALLOC (NULL);
 
-  gcc_assert (!need_ssa_update_p ());
+  gcc_assert (!need_ssa_update_p (cfun));
 
   verify_stmts ();
 
@@ -796,6 +604,7 @@ verify_ssa (bool check_modified_stmt)
        {
          gimple stmt = gsi_stmt (gsi);
          use_operand_p use_p;
+         bool has_err;
 
          if (check_modified_stmt && gimple_modified_p (stmt))
            {
@@ -814,10 +623,9 @@ verify_ssa (bool check_modified_stmt)
              base_address = get_base_address (lhs);
 
              if (base_address
-                 && gimple_aliases_computed_p (cfun)
                  && SSA_VAR_P (base_address)
-                 && !gimple_has_volatile_ops (stmt)
-                 && ZERO_SSA_OPERANDS (stmt, SSA_OP_VDEF))
+                 && !gimple_vdef (stmt)
+                 && optimize > 0)
                {
                  error ("statement makes a memory store, but has no VDEFS");
                  print_gimple_stmt (stderr, stmt, 0, TDF_VOPS);
@@ -825,14 +633,42 @@ verify_ssa (bool check_modified_stmt)
                }
            }
 
-         FOR_EACH_SSA_TREE_OPERAND (op, stmt, iter, SSA_OP_ALL_VIRTUALS)
+         /* Verify the single virtual operand and its constraints.  */
+         has_err = false;
+         if (gimple_vdef (stmt))
            {
-             if (verify_ssa_name (op, true))
+             if (gimple_vdef_op (stmt) == NULL_DEF_OPERAND_P)
                {
-                 error ("in statement");
-                 print_gimple_stmt (stderr, stmt, 0, TDF_VOPS|TDF_MEMSYMS);
-                 goto err;
+                 error ("statement has VDEF operand not in defs list");
+                 has_err = true;
+               }
+             if (!gimple_vuse (stmt))
+               {
+                 error ("statement has VDEF but no VUSE operand");
+                 has_err = true;
+               }
+             else if (SSA_NAME_VAR (gimple_vdef (stmt))
+                      != SSA_NAME_VAR (gimple_vuse (stmt)))
+               {
+                 error ("VDEF and VUSE do not use the same symbol");
+                 has_err = true;
                }
+             has_err |= verify_ssa_name (gimple_vdef (stmt), true);
+           }
+         if (gimple_vuse (stmt))
+           {
+             if  (gimple_vuse_op (stmt) == NULL_USE_OPERAND_P)
+               {
+                 error ("statement has VUSE operand not in uses list");
+                 has_err = true;
+               }
+             has_err |= verify_ssa_name (gimple_vuse (stmt), true);
+           }
+         if (has_err)
+           {
+             error ("in statement");
+             print_gimple_stmt (stderr, stmt, 0, TDF_VOPS|TDF_MEMSYMS);
+             goto err;
            }
 
          FOR_EACH_SSA_TREE_OPERAND (op, stmt, iter, SSA_OP_USE|SSA_OP_DEF)
@@ -854,16 +690,24 @@ verify_ssa (bool check_modified_stmt)
            }
 
          FOR_EACH_SSA_TREE_OPERAND (op, stmt, iter, SSA_OP_ALL_DEFS)
-           bitmap_set_bit (names_defined_in_bb, SSA_NAME_VERSION (op));
+           {
+             if (SSA_NAME_DEF_STMT (op) != stmt)
+               {
+                 error ("SSA_NAME_DEF_STMT is wrong");
+                 fprintf (stderr, "Expected definition statement:\n");
+                 print_gimple_stmt (stderr, stmt, 4, TDF_VOPS);
+                 fprintf (stderr, "\nActual definition statement:\n");
+                 print_gimple_stmt (stderr, SSA_NAME_DEF_STMT (op),
+                                    4, TDF_VOPS);
+                 goto err;
+               }
+             bitmap_set_bit (names_defined_in_bb, SSA_NAME_VERSION (op));
+           }
        }
 
       bitmap_clear (names_defined_in_bb);
     }
 
-  /* Finally, verify alias information.  */
-  if (gimple_aliases_computed_p (cfun))
-    verify_alias_info ();
-
   free (definition_block);
 
   /* Restore the dominance information to its prior known state, so
@@ -946,9 +790,8 @@ init_tree_ssa (struct function *fn)
                                                    uid_decl_map_eq, NULL);
   fn->gimple_df->default_defs = htab_create_ggc (20, uid_ssaname_map_hash, 
                                                 uid_ssaname_map_eq, NULL);
-  fn->gimple_df->call_clobbered_vars = BITMAP_GGC_ALLOC ();
-  fn->gimple_df->call_used_vars = BITMAP_GGC_ALLOC ();
-  fn->gimple_df->addressable_vars = BITMAP_GGC_ALLOC ();
+  pt_solution_reset (&fn->gimple_df->escaped);
+  pt_solution_reset (&fn->gimple_df->callused);
   init_ssanames (fn, 0);
   init_phinodes ();
 }
@@ -996,10 +839,8 @@ delete_tree_ssa (void)
 
          if (gimple_has_mem_ops (stmt))
            {
-             gimple_set_vdef_ops (stmt, NULL);
-             gimple_set_vuse_ops (stmt, NULL);
-             BITMAP_FREE (stmt->gsmem.membase.stores);
-             BITMAP_FREE (stmt->gsmem.membase.loads);
+             gimple_set_vdef (stmt, NULL_TREE);
+             gimple_set_vuse (stmt, NULL_TREE);
            }
 
          gimple_set_modified (stmt, true);
@@ -1010,13 +851,8 @@ delete_tree_ssa (void)
   /* Remove annotations from every referenced local variable.  */
   FOR_EACH_REFERENCED_VAR (var, rvi)
     {
-      if (!MTAG_P (var)
-         && (TREE_STATIC (var) || DECL_EXTERNAL (var)))
-       {
-         var_ann (var)->mpt = NULL_TREE;
-         var_ann (var)->symbol_mem_tag = NULL_TREE;
-         continue;
-       }
+      if (is_global_var (var))
+       continue;
       if (var->base.ann)
         ggc_free (var->base.ann);
       var->base.ann = NULL;
@@ -1031,22 +867,13 @@ delete_tree_ssa (void)
   if (ssa_operands_active ())
     fini_ssa_operands ();
 
-  cfun->gimple_df->global_var = NULL_TREE;
-  
+  delete_alias_heapvars ();
+
   htab_delete (cfun->gimple_df->default_defs);
   cfun->gimple_df->default_defs = NULL;
-  cfun->gimple_df->call_clobbered_vars = NULL;
-  cfun->gimple_df->call_used_vars = NULL;
-  cfun->gimple_df->addressable_vars = NULL;
+  pt_solution_reset (&cfun->gimple_df->escaped);
+  pt_solution_reset (&cfun->gimple_df->callused);
   cfun->gimple_df->modified_noreturn_calls = NULL;
-  if (gimple_aliases_computed_p (cfun))
-    {
-      delete_alias_heapvars ();
-      gcc_assert (!need_ssa_update_p ());
-    }
-  cfun->gimple_df->aliases_computed_p = false;
-  delete_mem_ref_stats (cfun);
-
   cfun->gimple_df = NULL;
 
   /* We no longer need the edge variable maps.  */
@@ -1058,7 +885,18 @@ delete_tree_ssa (void)
 static bool
 useless_type_conversion_p_1 (tree outer_type, tree inner_type)
 {
-  /* Qualifiers on value types do not matter.  */
+  /* Do the following before stripping toplevel qualifiers.  */
+  if (POINTER_TYPE_P (inner_type)
+      && POINTER_TYPE_P (outer_type))
+    {
+      /* Do not lose casts to restrict qualified pointers.  */
+      if ((TYPE_RESTRICT (outer_type)
+          != TYPE_RESTRICT (inner_type))
+         && TYPE_RESTRICT (outer_type))
+       return false;
+    }
+
+  /* From now on qualifiers on value types do not matter.  */
   inner_type = TYPE_MAIN_VARIANT (inner_type);
   outer_type = TYPE_MAIN_VARIANT (outer_type);
 
@@ -1112,9 +950,14 @@ useless_type_conversion_p_1 (tree outer_type, tree inner_type)
     {
       /* Don't lose casts between pointers to volatile and non-volatile
         qualified types.  Doing so would result in changing the semantics
-        of later accesses.  */
-      if ((TYPE_VOLATILE (TREE_TYPE (outer_type))
-          != TYPE_VOLATILE (TREE_TYPE (inner_type)))
+        of later accesses.  For function types the volatile qualifier
+        is used to indicate noreturn functions.  */
+      if (TREE_CODE (TREE_TYPE (outer_type)) != FUNCTION_TYPE
+         && TREE_CODE (TREE_TYPE (outer_type)) != METHOD_TYPE
+         && TREE_CODE (TREE_TYPE (inner_type)) != FUNCTION_TYPE
+         && TREE_CODE (TREE_TYPE (inner_type)) != METHOD_TYPE
+         && (TYPE_VOLATILE (TREE_TYPE (outer_type))
+             != TYPE_VOLATILE (TREE_TYPE (inner_type)))
          && TYPE_VOLATILE (TREE_TYPE (outer_type)))
        return false;
 
@@ -1129,12 +972,6 @@ useless_type_conversion_p_1 (tree outer_type, tree inner_type)
       /* We do not care for const qualification of the pointed-to types
         as const qualification has no semantic value to the middle-end.  */
 
-      /* Do not lose casts to restrict qualified pointers.  */
-      if ((TYPE_RESTRICT (outer_type)
-          != TYPE_RESTRICT (inner_type))
-         && TYPE_RESTRICT (outer_type))
-       return false;
-
       /* Otherwise pointers/references are equivalent if their pointed
         to types are effectively the same.  We can strip qualifiers
         on pointed-to types for further comparison, which is done in
@@ -1146,15 +983,15 @@ useless_type_conversion_p_1 (tree outer_type, tree inner_type)
   /* Recurse for complex types.  */
   else if (TREE_CODE (inner_type) == COMPLEX_TYPE
           && TREE_CODE (outer_type) == COMPLEX_TYPE)
-    return useless_type_conversion_p_1 (TREE_TYPE (outer_type),
-                                       TREE_TYPE (inner_type));
+    return useless_type_conversion_p (TREE_TYPE (outer_type),
+                                     TREE_TYPE (inner_type));
 
   /* Recurse for vector types with the same number of subparts.  */
   else if (TREE_CODE (inner_type) == VECTOR_TYPE
           && TREE_CODE (outer_type) == VECTOR_TYPE
           && TYPE_PRECISION (inner_type) == TYPE_PRECISION (outer_type))
-    return useless_type_conversion_p_1 (TREE_TYPE (outer_type),
-                                       TREE_TYPE (inner_type));
+    return useless_type_conversion_p (TREE_TYPE (outer_type),
+                                     TREE_TYPE (inner_type));
 
   /* For aggregates we may need to fall back to structural equality
      checks.  */
@@ -1411,6 +1248,10 @@ warn_uninit (tree t, const char *gmsgid, void *data)
   if (TREE_NO_WARNING (var))
     return;
 
+  /* Do not warn if it can be initialized outside this module.  */
+  if (is_global_var (var))
+    return;
+  
   location = (context != NULL && gimple_has_location (context))
             ? gimple_location (context)
             : DECL_SOURCE_LOCATION (var);
@@ -1423,7 +1264,7 @@ warn_uninit (tree t, const char *gmsgid, void *data)
       if (xloc.file != floc.file
          || xloc.line < floc.line
          || xloc.line > LOCATION_LINE (cfun->function_end_locus))
-       inform ("%J%qD was declared here", var, var);
+       inform (input_location, "%J%qD was declared here", var, var);
     }
 }
 
@@ -1443,8 +1284,49 @@ warn_uninitialized_var (tree *tp, int *walk_subtrees, void *data_)
   struct walk_data *data = (struct walk_data *) wi->info;
   tree t = *tp;
 
+  /* We do not care about LHS.  */
+  if (wi->is_lhs)
+    return NULL_TREE;
+
   switch (TREE_CODE (t))
     {
+    case ADDR_EXPR:
+      /* Taking the address of an uninitialized variable does not
+        count as using it.  */
+      *walk_subtrees = 0;
+      break;
+
+    case VAR_DECL:
+      {
+       /* A VAR_DECL in the RHS of a gimple statement may mean that
+          this variable is loaded from memory.  */
+       use_operand_p vuse;
+       tree op;
+
+       /* If there is not gimple stmt, 
+          or alias information has not been computed,
+          then we cannot check VUSE ops.  */
+       if (data->stmt == NULL)
+         return NULL_TREE;
+
+       /* If the load happens as part of a call do not warn about it.  */
+       if (is_gimple_call (data->stmt))
+         return NULL_TREE;
+
+       vuse = gimple_vuse_op (data->stmt);
+       if (vuse == NULL_USE_OPERAND_P)
+         return NULL_TREE;
+
+       op = USE_FROM_PTR (vuse);
+       if (t != SSA_NAME_VAR (op) 
+           || !SSA_NAME_IS_DEFAULT_DEF (op))
+         return NULL_TREE;
+       /* If this is a VUSE of t and it is the default definition,
+          then warn about op.  */
+       t = op;
+       /* Fall through into SSA_NAME.  */
+      }
+
     case SSA_NAME:
       /* We only do data flow with SSA_NAMEs, so that's all we
         can warn about.  */
@@ -1520,6 +1402,11 @@ warn_uninitialized_vars (bool warn_possibly_uninitialized)
          walk_gimple_op (gsi_stmt (gsi), warn_uninitialized_var, &wi);
        }
     }
+
+  /* Post-dominator information can not be reliably updated. Free it
+     after the use.  */
+
+  free_dominance_info (CDI_POST_DOMINATORS);
   return 0;
 }
 
@@ -1598,17 +1485,17 @@ struct gimple_opt_pass pass_late_warn_uninitialized =
  }
 };
 
-/* Compute TREE_ADDRESSABLE for local variables.  */
+/* Compute TREE_ADDRESSABLE and DECL_GIMPLE_REG_P for local variables.  */
 
-static unsigned int
-execute_update_addresses_taken (void)
+void
+execute_update_addresses_taken (bool do_optimize)
 {
   tree var;
   referenced_var_iterator rvi;
   gimple_stmt_iterator gsi;
   basic_block bb;
   bitmap addresses_taken = BITMAP_ALLOC (NULL);
-  bitmap vars_updated = BITMAP_ALLOC (NULL);
+  bitmap not_reg_needs = BITMAP_ALLOC (NULL);
   bool update_vops = false;
 
   /* Collect into ADDRESSES_TAKEN all variables whose address is taken within
@@ -1617,9 +1504,26 @@ execute_update_addresses_taken (void)
     {
       for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
        {
-         bitmap taken = gimple_addresses_taken (gsi_stmt (gsi));
+         const_gimple stmt = gsi_stmt (gsi);
+         enum gimple_code code = gimple_code (stmt);
+         bitmap taken = gimple_addresses_taken (stmt);
+         
          if (taken)
            bitmap_ior_into (addresses_taken, taken);
+         
+         /* If we have a call or an assignment, see if the lhs contains
+            a local decl that requires not to be a gimple register.  */
+         if (code == GIMPLE_ASSIGN || code == GIMPLE_CALL)
+           {
+             tree lhs = gimple_get_lhs (stmt);
+             /* A plain decl does not need it set.  */
+             if (lhs && handled_component_p (lhs))
+               {
+                 var = get_base_address (lhs);
+                 if (DECL_P (var))
+                   bitmap_set_bit (not_reg_needs, DECL_UID (var));
+               }
+           }
        }
 
       for (gsi = gsi_start_phis (bb); !gsi_end_p (gsi); gsi_next (&gsi))
@@ -1638,44 +1542,71 @@ execute_update_addresses_taken (void)
        }
     }
 
-  /* When possible, clear ADDRESSABLE bit and mark variable for conversion into
-     SSA.  */
-  FOR_EACH_REFERENCED_VAR (var, rvi)
-    if (!is_global_var (var)
-       && TREE_CODE (var) != RESULT_DECL
-       && TREE_ADDRESSABLE (var)
-       && !bitmap_bit_p (addresses_taken, DECL_UID (var)))
+  /* When possible, clear ADDRESSABLE bit or set the REGISTER bit
+     and mark variable for conversion into SSA.  */
+  if (optimize && do_optimize)
+    FOR_EACH_REFERENCED_VAR (var, rvi)
       {
-        TREE_ADDRESSABLE (var) = 0;
-       if (is_gimple_reg (var))
-         mark_sym_for_renaming (var);
-       update_vops = true;
-       bitmap_set_bit (vars_updated, DECL_UID (var));
-       if (dump_file)
+       /* Global Variables, result decls cannot be changed.  */
+       if (is_global_var (var)
+           || TREE_CODE (var) == RESULT_DECL
+           || bitmap_bit_p (addresses_taken, DECL_UID (var)))
+         continue;
+
+       if (TREE_ADDRESSABLE (var)
+           /* Do not change TREE_ADDRESSABLE if we need to preserve var as
+              a non-register.  Otherwise we are confused and forget to
+              add virtual operands for it.  */
+           && (!is_gimple_reg_type (TREE_TYPE (var))
+               || !bitmap_bit_p (not_reg_needs, DECL_UID (var))))
          {
-           fprintf (dump_file, "No longer having address taken ");
-           print_generic_expr (dump_file, var, 0);
-           fprintf (dump_file, "\n");
+           TREE_ADDRESSABLE (var) = 0;
+           if (is_gimple_reg (var))
+             mark_sym_for_renaming (var);
+           update_vops = true;
+           if (dump_file)
+             {
+               fprintf (dump_file, "No longer having address taken ");
+               print_generic_expr (dump_file, var, 0);
+               fprintf (dump_file, "\n");
+             }
+         }
+       if (!DECL_GIMPLE_REG_P (var)
+           && !bitmap_bit_p (not_reg_needs, DECL_UID (var))
+           && (TREE_CODE (TREE_TYPE (var)) == COMPLEX_TYPE
+               || TREE_CODE (TREE_TYPE (var)) == VECTOR_TYPE))
+         {
+           DECL_GIMPLE_REG_P (var) = 1;
+           mark_sym_for_renaming (var);
+           update_vops = true;
+           if (dump_file)
+             {
+               fprintf (dump_file, "Decl is now a gimple register ");
+               print_generic_expr (dump_file, var, 0);
+               fprintf (dump_file, "\n");
+             }
          }
       }
 
   /* Operand caches needs to be recomputed for operands referencing the updated
      variables.  */
   if (update_vops)
-    FOR_EACH_BB (bb)
-      for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
-       {
-         gimple stmt = gsi_stmt (gsi);
+    {
+      FOR_EACH_BB (bb)
+         for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
+           {
+             gimple stmt = gsi_stmt (gsi);
 
-         if ((gimple_loaded_syms (stmt)
-              && bitmap_intersect_p (gimple_loaded_syms (stmt), vars_updated))
-             || (gimple_stored_syms (stmt)
-                 && bitmap_intersect_p (gimple_stored_syms (stmt), vars_updated)))
-           update_stmt (stmt);
-       }
+             if (gimple_references_memory_p (stmt))
+               update_stmt (stmt);
+           }
+
+      /* Update SSA form here, we are called as non-pass as well.  */
+      update_ssa (TODO_update_ssa);
+    }
+
+  BITMAP_FREE (not_reg_needs);
   BITMAP_FREE (addresses_taken);
-  BITMAP_FREE (vars_updated);
-  return 0;
 }
 
 struct gimple_opt_pass pass_update_address_taken =
@@ -1684,7 +1615,7 @@ struct gimple_opt_pass pass_update_address_taken =
   GIMPLE_PASS,
   "addressables",                      /* name */
   NULL,                                        /* gate */
-  execute_update_addresses_taken,      /* execute */
+  NULL,                                        /* execute */
   NULL,                                        /* sub */
   NULL,                                        /* next */
   0,                                   /* static_pass_number */
@@ -1693,6 +1624,7 @@ struct gimple_opt_pass pass_update_address_taken =
   0,                                   /* properties_provided */
   0,                                   /* properties_destroyed */
   0,                                   /* todo_flags_start */
-  TODO_update_ssa                       /* todo_flags_finish */
+  TODO_update_address_taken
+  | TODO_dump_func                     /* todo_flags_finish */
  }
 };