OSDN Git Service

* tree-iterator.c (EXPR_LAST_BODY): Remove.
[pf3gnuchains/gcc-fork.git] / gcc / tree-ssa-dse.c
index d7453dd..315b550 100644 (file)
@@ -1,5 +1,6 @@
 /* Dead store elimination
-   Copyright (C) 2004, 2005, 2006, 2007 Free Software Foundation, Inc.
+   Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009
+   Free Software Foundation, Inc.
 
 This file is part of GCC.
 
@@ -33,6 +34,7 @@ along with GCC; see the file COPYING3.  If not see
 #include "tree-dump.h"
 #include "domwalk.h"
 #include "flags.h"
+#include "langhooks.h"
 
 /* This file implements dead store elimination.
 
@@ -62,7 +64,7 @@ along with GCC; see the file COPYING3.  If not see
    relationship between dead store and redundant load elimination.  In
    fact, they are the same transformation applied to different views of
    the CFG.  */
-   
+
 
 struct dse_global_data
 {
@@ -82,13 +84,6 @@ struct dse_block_local_data
   bitmap stores;
 };
 
-/* Basic blocks of the potentially dead store and the following
-   store, for memory_address_same.  */
-struct address_walk_data
-{
-  basic_block store1_bb, store2_bb;
-};
-
 static bool gate_dse (void);
 static unsigned int tree_ssa_dse (void);
 static void dse_initialize_block_local_data (struct dom_walk_data *,
@@ -96,24 +91,21 @@ static void dse_initialize_block_local_data (struct dom_walk_data *,
                                             bool);
 static void dse_optimize_stmt (struct dom_walk_data *,
                               basic_block,
-                              block_stmt_iterator);
+                              gimple_stmt_iterator);
 static void dse_record_phis (struct dom_walk_data *, basic_block);
 static void dse_finalize_block (struct dom_walk_data *, basic_block);
 static void record_voperand_set (bitmap, bitmap *, unsigned int);
 
-static unsigned max_stmt_uid;  /* Maximal uid of a statement.  Uids to phi
-                                  nodes are assigned using the versions of
-                                  ssa names they define.  */
-
 /* Returns uid of statement STMT.  */
 
 static unsigned
-get_stmt_uid (tree stmt)
+get_stmt_uid (gimple stmt)
 {
-  if (TREE_CODE (stmt) == PHI_NODE)
-    return SSA_NAME_VERSION (PHI_RESULT (stmt)) + max_stmt_uid;
+  if (gimple_code (stmt) == GIMPLE_PHI)
+    return SSA_NAME_VERSION (gimple_phi_result (stmt))
+           + gimple_stmt_max_uid (cfun);
 
-  return stmt_ann (stmt)->uid;
+  return gimple_uid (stmt);
 }
 
 /* Set bit UID in bitmaps GLOBAL and *LOCAL, creating *LOCAL as needed.  */
@@ -152,236 +144,112 @@ dse_initialize_block_local_data (struct dom_walk_data *walk_data,
     }
 }
 
-/* Helper function for memory_address_same via walk_tree.  Returns
-   non-NULL if it finds an SSA_NAME which is part of the address,
-   such that the definition of the SSA_NAME post-dominates the store
-   we want to delete but not the store that we believe makes it
-   redundant.  This indicates that the address may change between
-   the two stores.  */
-
-static tree
-memory_ssa_name_same (tree *expr_p, int *walk_subtrees ATTRIBUTE_UNUSED,
-                     void *data)
-{
-  struct address_walk_data *walk_data = (struct address_walk_data *) data;
-  tree expr = *expr_p;
-  tree def_stmt;
-  basic_block def_bb;
-
-  if (TREE_CODE (expr) != SSA_NAME)
-    return NULL_TREE;
-
-  /* If we've found a default definition, then there's no problem.  Both
-     stores will post-dominate it.  And def_bb will be NULL.  */
-  if (SSA_NAME_IS_DEFAULT_DEF (expr))
-    return NULL_TREE;
-
-  def_stmt = SSA_NAME_DEF_STMT (expr);
-  def_bb = bb_for_stmt (def_stmt);
-
-  /* DEF_STMT must dominate both stores.  So if it is in the same
-     basic block as one, it does not post-dominate that store.  */
-  if (walk_data->store1_bb != def_bb
-      && dominated_by_p (CDI_POST_DOMINATORS, walk_data->store1_bb, def_bb))
-    {
-      if (walk_data->store2_bb == def_bb
-         || !dominated_by_p (CDI_POST_DOMINATORS, walk_data->store2_bb,
-                             def_bb))
-       /* Return non-NULL to stop the walk.  */
-       return def_stmt;
-    }
-
-  return NULL_TREE;
-}
-
-/* Return TRUE if the destination memory address in STORE1 and STORE2
-   might be modified after STORE1, before control reaches STORE2.  */
-
-static bool
-memory_address_same (tree store1, tree store2)
-{
-  struct address_walk_data walk_data;
-
-  walk_data.store1_bb = bb_for_stmt (store1);
-  walk_data.store2_bb = bb_for_stmt (store2);
-
-  return (walk_tree (&GIMPLE_STMT_OPERAND (store1, 0), memory_ssa_name_same,
-                    &walk_data, NULL)
-         == NULL);
-}
-
-/* Return the use stmt for the lhs of STMT following the virtual
-   def-use chains.  Returns the MODIFY_EXPR stmt which lhs is equal to
-   the lhs of STMT or NULL_TREE if no such stmt can be found.  */
-static tree 
-get_use_of_stmt_lhs (tree stmt,
-                    use_operand_p * first_use_p,
-                    use_operand_p * use_p, tree * use_stmt)
-{
-  tree usevar, lhs;
-  def_operand_p def_p;
-
-  if (TREE_CODE (stmt) != GIMPLE_MODIFY_STMT)
-    return NULL_TREE;
-
-  lhs = GIMPLE_STMT_OPERAND (stmt, 0);
-
-  /* The stmt must have a single VDEF.  */
-  def_p = SINGLE_SSA_DEF_OPERAND (stmt, SSA_OP_VDEF);
-  if (def_p == NULL_DEF_OPERAND_P)
-    return NULL_TREE;
-
-  if (!has_single_use (DEF_FROM_PTR (def_p)))
-    return NULL_TREE;
-  /* Get the immediate use of the def.  */
-  single_imm_use (DEF_FROM_PTR (def_p), use_p, use_stmt);
-  gcc_assert (*use_p != NULL_USE_OPERAND_P);
-  first_use_p = use_p;
-
-  /* If the use is not simple, give up.  */
-  if (TREE_CODE (*use_stmt) != GIMPLE_MODIFY_STMT
-      || get_call_expr_in (*use_stmt))
-    return NULL_TREE;
-
-  do
-    {
-      /* Look at the use stmt and see if it's LHS matches
-         stmt's lhs SSA_NAME.  */
-      def_p = SINGLE_SSA_DEF_OPERAND (*use_stmt, SSA_OP_VDEF);
-      if (def_p == NULL_DEF_OPERAND_P)
-       return NULL_TREE;
-
-      usevar = GIMPLE_STMT_OPERAND (*use_stmt, 0);
-      if (operand_equal_p (usevar, lhs, 0))
-       return *use_stmt;
-
-      if (!has_single_use (DEF_FROM_PTR (def_p)))
-       return NULL_TREE;
-      single_imm_use (DEF_FROM_PTR (def_p), use_p, use_stmt);
-      gcc_assert (*use_p != NULL_USE_OPERAND_P);
-      if (TREE_CODE (*use_stmt) != GIMPLE_MODIFY_STMT
-         || get_call_expr_in (*use_stmt))
-       return NULL_TREE;
-    }
-  while (1);
-
-  return NULL_TREE;
-}
-
 /* A helper of dse_optimize_stmt.
-   Given a GIMPLE_MODIFY_STMT in STMT, check that each VDEF has one
-   use, and that one use is another VDEF clobbering the first one.
-
+   Given a GIMPLE_ASSIGN in STMT, find a candidate statement *USE_STMT that
+   may prove STMT to be dead.
    Return TRUE if the above conditions are met, otherwise FALSE.  */
 
 static bool
-dse_possible_dead_store_p (tree stmt,
-                          use_operand_p *first_use_p,
-                          use_operand_p *use_p,
-                          tree *use_stmt,
-                          struct dse_global_data *dse_gd,
-                          struct dse_block_local_data *bd)
+dse_possible_dead_store_p (gimple stmt, gimple *use_stmt)
 {
-  ssa_op_iter op_iter;
-  bool fail = false;
-  def_operand_p var1;
-  vuse_vec_p vv;
-  tree defvar = NULL_TREE, temp;
-  tree prev_defvar = NULL_TREE;
-  stmt_ann_t ann = stmt_ann (stmt);
-
-  /* We want to verify that each virtual definition in STMT has
-     precisely one use and that all the virtual definitions are
-     used by the same single statement.  When complete, we
-     want USE_STMT to refer to the one statement which uses
-     all of the virtual definitions from STMT.  */
-  *use_stmt = NULL;
-  FOR_EACH_SSA_VDEF_OPERAND (var1, vv, stmt, op_iter)
-    {
-      defvar = DEF_FROM_PTR (var1);
-
-      /* If this virtual def does not have precisely one use, then
-        we will not be able to eliminate STMT.  */
-      if (!has_single_use (defvar))
-       {
-         fail = true;
-         break;
-       }
-
-      /* Get the one and only immediate use of DEFVAR.  */
-      single_imm_use (defvar, use_p, &temp);
-      gcc_assert (*use_p != NULL_USE_OPERAND_P);
-      *first_use_p = *use_p;
+  gimple temp;
+  unsigned cnt = 0;
 
-      /* In the case of memory partitions, we may get:
+  *use_stmt = NULL;
 
-          # MPT.764_162 = VDEF <MPT.764_161(D)>
-          x = {};
-          # MPT.764_167 = VDEF <MPT.764_162>
-          y = {};
+  /* Find the first dominated statement that clobbers (part of) the
+     memory stmt stores to with no intermediate statement that may use
+     part of the memory stmt stores.  That is, find a store that may
+     prove stmt to be a dead store.  */
+  temp = stmt;
+  do
+    {
+      gimple prev, use_stmt;
+      imm_use_iterator ui;
+      bool fail = false;
+      tree defvar;
+
+      /* Limit stmt walking to be linear in the number of possibly
+         dead stores.  */
+      if (++cnt > 256)
+       return false;
 
-          So we must make sure we're talking about the same LHS.
-      */
-      if (TREE_CODE (temp) == GIMPLE_MODIFY_STMT)
+      if (gimple_code (temp) == GIMPLE_PHI)
+       defvar = PHI_RESULT (temp);
+      else
+       defvar = gimple_vdef (temp);
+      prev = temp;
+      temp = NULL;
+      FOR_EACH_IMM_USE_STMT (use_stmt, ui, defvar)
        {
-         tree base1 = get_base_address (GIMPLE_STMT_OPERAND (stmt, 0));
-         tree base2 =  get_base_address (GIMPLE_STMT_OPERAND (temp, 0));
-
-         while (base1 && INDIRECT_REF_P (base1))
-           base1 = TREE_OPERAND (base1, 0);
-         while (base2 && INDIRECT_REF_P (base2))
-           base2 = TREE_OPERAND (base2, 0);
+         cnt++;
 
-         if (base1 != base2)
+         /* In simple cases we can look through PHI nodes, but we
+            have to be careful with loops and with memory references
+            containing operands that are also operands of PHI nodes.
+            See gcc.c-torture/execute/20051110-*.c.  */
+         if (gimple_code (use_stmt) == GIMPLE_PHI)
+           {
+             if (temp
+                 /* We can look through PHIs to post-dominated regions
+                    without worrying if the use not also dominates prev
+                    (in which case it would be a loop PHI with the use
+                    in a latch block).  */
+                 || gimple_bb (prev) == gimple_bb (use_stmt)
+                 || !dominated_by_p (CDI_POST_DOMINATORS,
+                                     gimple_bb (prev), gimple_bb (use_stmt))
+                 || dominated_by_p (CDI_DOMINATORS,
+                                    gimple_bb (prev), gimple_bb (use_stmt)))
+               {
+                 fail = true;
+                 BREAK_FROM_IMM_USE_STMT (ui);
+               }
+             temp = use_stmt;
+           }
+         /* If the statement is a use the store is not dead.  */
+         else if (ref_maybe_used_by_stmt_p (use_stmt,
+                                            gimple_assign_lhs (stmt)))
            {
              fail = true;
-             break;
+             BREAK_FROM_IMM_USE_STMT (ui);
+           }
+         /* If this is a store, remember it or bail out if we have
+            multiple ones (the will be in different CFG parts then).  */
+         else if (gimple_vdef (use_stmt))
+           {
+             if (temp)
+               {
+                 fail = true;
+                 BREAK_FROM_IMM_USE_STMT (ui);
+               }
+             temp = use_stmt;
            }
        }
 
-      /* If the immediate use of DEF_VAR is not the same as the
-        previously find immediate uses, then we will not be able
-        to eliminate STMT.  */
-      if (*use_stmt == NULL)
-       {
-         *use_stmt = temp;
-         prev_defvar = defvar;
-       }
-      else if (temp != *use_stmt)
+      if (fail)
+       return false;
+
+      /* If we didn't find any definition this means the store is dead
+         if it isn't a store to global reachable memory.  In this case
+        just pretend the stmt makes itself dead.  Otherwise fail.  */
+      if (!temp)
        {
-         fail = true;
+         if (is_hidden_global_store (stmt))
+           return false;
+
+         temp = stmt;
          break;
        }
     }
+  /* We deliberately stop on clobbering statements and not only on
+     killing ones to make walking cheaper.  Otherwise we can just
+     continue walking until both stores have equal reference trees.  */
+  while (!stmt_may_clobber_ref_p (temp, gimple_assign_lhs (stmt)));
 
-  if (fail)
-    {
-      record_voperand_set (dse_gd->stores, &bd->stores, ann->uid);
-      return false;
-    }
-
-  /* Skip through any PHI nodes we have already seen if the PHI
-     represents the only use of this store.
+  if (!is_gimple_assign (temp))
+    return false;
 
-     Note this does not handle the case where the store has
-     multiple VDEFs which all reach a set of PHI nodes in the same block.  */
-  while (*use_p != NULL_USE_OPERAND_P
-        && TREE_CODE (*use_stmt) == PHI_NODE
-        && bitmap_bit_p (dse_gd->stores, get_stmt_uid (*use_stmt)))
-    {
-      /* A PHI node can both define and use the same SSA_NAME if
-        the PHI is at the top of a loop and the PHI_RESULT is
-        a loop invariant and copies have not been fully propagated.
-
-        The safe thing to do is exit assuming no optimization is
-        possible.  */
-      if (SSA_NAME_DEF_STMT (PHI_RESULT (*use_stmt)) == *use_stmt)
-       return false;
-
-      /* Skip past this PHI and loop again in case we had a PHI
-        chain.  */
-      single_imm_use (PHI_RESULT (*use_stmt), use_p, use_stmt);
-    }
+  *use_stmt = temp;
 
   return true;
 }
@@ -401,108 +269,82 @@ dse_possible_dead_store_p (tree stmt,
 static void
 dse_optimize_stmt (struct dom_walk_data *walk_data,
                   basic_block bb ATTRIBUTE_UNUSED,
-                  block_stmt_iterator bsi)
+                  gimple_stmt_iterator gsi)
 {
   struct dse_block_local_data *bd
     = (struct dse_block_local_data *)
        VEC_last (void_p, walk_data->block_data_stack);
   struct dse_global_data *dse_gd
     = (struct dse_global_data *) walk_data->global_data;
-  tree stmt = bsi_stmt (bsi);
-  stmt_ann_t ann = stmt_ann (stmt);
+  gimple stmt = gsi_stmt (gsi);
 
   /* If this statement has no virtual defs, then there is nothing
      to do.  */
-  if (ZERO_SSA_OPERANDS (stmt, SSA_OP_VDEF))
+  if (!gimple_vdef (stmt))
     return;
 
-  /* We know we have virtual definitions.  If this is a GIMPLE_MODIFY_STMT
+  /* We know we have virtual definitions.  If this is a GIMPLE_ASSIGN
      that's not also a function call, then record it into our table.  */
-  if (get_call_expr_in (stmt))
+  if (is_gimple_call (stmt) && gimple_call_fndecl (stmt))
     return;
 
-  if (ann->has_volatile_ops)
+  if (gimple_has_volatile_ops (stmt))
     return;
 
-  if (TREE_CODE (stmt) == GIMPLE_MODIFY_STMT)
+  if (is_gimple_assign (stmt))
     {
-      use_operand_p first_use_p = NULL_USE_OPERAND_P;
-      use_operand_p use_p = NULL;
-      tree use_stmt;
+      gimple use_stmt;
 
-      if (!dse_possible_dead_store_p (stmt, &first_use_p, &use_p, &use_stmt,
-                                     dse_gd, bd))
-       return;
+      record_voperand_set (dse_gd->stores, &bd->stores, gimple_uid (stmt));
 
-      /* If we have precisely one immediate use at this point, then we may
-        have found redundant store.  Make sure that the stores are to
-        the same memory location.  This includes checking that any
-        SSA-form variables in the address will have the same values.  */
-      if (use_p != NULL_USE_OPERAND_P
-          && bitmap_bit_p (dse_gd->stores, get_stmt_uid (use_stmt))
-          && !operand_equal_p (GIMPLE_STMT_OPERAND (stmt, 0),
-                               GIMPLE_STMT_OPERAND (use_stmt, 0), 0)
-          && memory_address_same (stmt, use_stmt))
-        {
-          /* If we have precisely one immediate use at this point, but
-             the stores are not to the same memory location then walk the
-             virtual def-use chain to get the stmt which stores to that same
-             memory location.  */
-          if (get_use_of_stmt_lhs (stmt, &first_use_p, &use_p, &use_stmt) ==
-              NULL_TREE)
-            {
-              record_voperand_set (dse_gd->stores, &bd->stores, ann->uid);
-              return;
-            }
-        }
+      if (!dse_possible_dead_store_p (stmt, &use_stmt))
+       return;
 
       /* If we have precisely one immediate use at this point and the
         stores are to the same memory location or there is a chain of
         virtual uses from stmt and the stmt which stores to that same
         memory location, then we may have found redundant store.  */
-      if (use_p != NULL_USE_OPERAND_P
-         && bitmap_bit_p (dse_gd->stores, get_stmt_uid (use_stmt))
-         && operand_equal_p (GIMPLE_STMT_OPERAND (stmt, 0),
-                             GIMPLE_STMT_OPERAND (use_stmt, 0), 0)
-         && memory_address_same (stmt, use_stmt))
+      if (bitmap_bit_p (dse_gd->stores, get_stmt_uid (use_stmt))
+         && operand_equal_p (gimple_assign_lhs (stmt),
+                             gimple_assign_lhs (use_stmt), 0))
        {
-         ssa_op_iter op_iter;
-         def_operand_p var1;
-         vuse_vec_p vv;
-         tree stmt_lhs;
+         /* If use_stmt is or might be a nop assignment, e.g. for
+            struct { ... } S a, b, *p; ...
+            b = a; b = b;
+            or
+            b = a; b = *p; where p might be &b,
+            or
+            *p = a; *p = b; where p might be &b,
+            or
+            *p = *u; *p = *v; where p might be v, then USE_STMT
+            acts as a use as well as definition, so store in STMT
+            is not dead.  */
+         if (stmt != use_stmt
+             && !is_gimple_reg (gimple_assign_rhs1 (use_stmt))
+             && !is_gimple_min_invariant (gimple_assign_rhs1 (use_stmt))
+             /* ???  Should {} be invariant?  */
+             && gimple_assign_rhs_code (use_stmt) != CONSTRUCTOR
+             && refs_may_alias_p (gimple_assign_lhs (use_stmt),
+                                  gimple_assign_rhs1 (use_stmt)))
+           return;
 
          if (dump_file && (dump_flags & TDF_DETAILS))
             {
               fprintf (dump_file, "  Deleted dead store '");
-              print_generic_expr (dump_file, bsi_stmt (bsi), dump_flags);
+              print_gimple_stmt (dump_file, gsi_stmt (gsi), dump_flags, 0);
               fprintf (dump_file, "'\n");
             }
 
          /* Then we need to fix the operand of the consuming stmt.  */
-         stmt_lhs = USE_FROM_PTR (first_use_p);
-         FOR_EACH_SSA_VDEF_OPERAND (var1, vv, stmt, op_iter)
-           {
-             tree usevar, temp;
-
-             single_imm_use (DEF_FROM_PTR (var1), &use_p, &temp);
-             gcc_assert (VUSE_VECT_NUM_ELEM (*vv) == 1);
-             usevar = VUSE_ELEMENT_VAR (*vv, 0);
-             SET_USE (use_p, usevar);
-
-             /* Make sure we propagate the ABNORMAL bit setting.  */
-             if (SSA_NAME_OCCURS_IN_ABNORMAL_PHI (stmt_lhs))
-               SSA_NAME_OCCURS_IN_ABNORMAL_PHI (usevar) = 1;
-           }
+         unlink_stmt_vdef (stmt);
 
          /* Remove the dead store.  */
-         bsi_remove (&bsi, true);
+         gsi_remove (&gsi, true);
 
          /* And release any SSA_NAMEs set in this statement back to the
             SSA_NAME manager.  */
          release_defs (stmt);
        }
-
-      record_voperand_set (dse_gd->stores, &bd->stores, ann->uid);
     }
 }
 
@@ -516,13 +358,15 @@ dse_record_phis (struct dom_walk_data *walk_data, basic_block bb)
        VEC_last (void_p, walk_data->block_data_stack);
   struct dse_global_data *dse_gd
     = (struct dse_global_data *) walk_data->global_data;
-  tree phi;
+  gimple phi;
+  gimple_stmt_iterator gsi;
 
-  for (phi = phi_nodes (bb); phi; phi = PHI_CHAIN (phi))
-    if (!is_gimple_reg (PHI_RESULT (phi)))
-      record_voperand_set (dse_gd->stores,
-                          &bd->stores,
-                          get_stmt_uid (phi));
+  for (gsi = gsi_start_phis (bb); !gsi_end_p (gsi); gsi_next (&gsi))
+    {
+      phi = gsi_stmt (gsi);
+      if (!is_gimple_reg (gimple_phi_result (phi)))
+       record_voperand_set (dse_gd->stores, &bd->stores, get_stmt_uid (phi));
+    }
 }
 
 static void
@@ -553,24 +397,15 @@ tree_ssa_dse (void)
 {
   struct dom_walk_data walk_data;
   struct dse_global_data dse_gd;
-  basic_block bb;
 
-  /* Create a UID for each statement in the function.  Ordering of the
-     UIDs is not important for this pass.  */
-  max_stmt_uid = 0;
-  FOR_EACH_BB (bb)
-    {
-      block_stmt_iterator bsi;
-
-      for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi))
-       stmt_ann (bsi_stmt (bsi))->uid = max_stmt_uid++;
-    }
+  renumber_gimple_stmt_uids ();
 
   /* We might consider making this a property of each pass so that it
      can be [re]computed on an as-needed basis.  Particularly since
      this pass could be seen as an extension of DCE which needs post
      dominators.  */
   calculate_dominance_info (CDI_POST_DOMINATORS);
+  calculate_dominance_info (CDI_DOMINATORS);
 
   /* Dead store elimination is fundamentally a walk of the post-dominator
      tree and a backwards walk of statements within each block.  */
@@ -614,7 +449,10 @@ gate_dse (void)
   return flag_tree_dse != 0;
 }
 
-struct tree_opt_pass pass_dse = {
+struct gimple_opt_pass pass_dse = 
+{
+ {
+  GIMPLE_PASS,
   "dse",                       /* name */
   gate_dse,                    /* gate */
   tree_ssa_dse,                        /* execute */
@@ -630,140 +468,7 @@ struct tree_opt_pass pass_dse = {
   0,                           /* todo_flags_start */
   TODO_dump_func
     | TODO_ggc_collect
-    | TODO_verify_ssa,         /* todo_flags_finish */
-  0                            /* letter */
+    | TODO_verify_ssa          /* todo_flags_finish */
+ }
 };
 
-/* A very simple dead store pass eliminating write only local variables.
-   The pass does not require alias information and thus can be run before
-   inlining to quickly eliminate artifacts of some common C++ constructs.  */
-
-static unsigned int
-execute_simple_dse (void)
-{
-  block_stmt_iterator bsi;
-  basic_block bb;
-  bitmap variables_loaded = BITMAP_ALLOC (NULL);
-  unsigned int todo = 0;
-
-  /* Collect into VARIABLES LOADED all variables that are read in function
-     body.  */
-  FOR_EACH_BB (bb)
-    for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi))
-      if (LOADED_SYMS (bsi_stmt (bsi)))
-       bitmap_ior_into (variables_loaded,
-                        LOADED_SYMS (bsi_stmt (bsi)));
-
-  /* Look for statements writting into the write only variables.
-     And try to remove them.  */
-
-  FOR_EACH_BB (bb)
-    for (bsi = bsi_start (bb); !bsi_end_p (bsi);)
-      {
-       tree stmt = bsi_stmt (bsi), op;
-       bool removed = false;
-        ssa_op_iter iter;
-
-       if (STORED_SYMS (stmt) && TREE_CODE (stmt) == GIMPLE_MODIFY_STMT
-           && TREE_CODE (stmt) != RETURN_EXPR
-           && !bitmap_intersect_p (STORED_SYMS (stmt), variables_loaded))
-         {
-           unsigned int i;
-           bitmap_iterator bi;
-           bool dead = true;
-
-
-
-           /* See if STMT only stores to write-only variables and
-              verify that there are no volatile operands.  tree-ssa-operands
-              sets has_volatile_ops flag for all statements involving
-              reads and writes when aliases are not built to prevent passes
-              from removing them as dead.  The flag thus has no use for us
-              and we need to look into all operands.  */
-             
-           EXECUTE_IF_SET_IN_BITMAP (STORED_SYMS (stmt), 0, i, bi)
-             {
-               tree var = referenced_var_lookup (i);
-               if (TREE_ADDRESSABLE (var)
-                   || is_global_var (var)
-                   || TREE_THIS_VOLATILE (var))
-                 dead = false;
-             }
-
-           if (dead && LOADED_SYMS (stmt))
-             EXECUTE_IF_SET_IN_BITMAP (LOADED_SYMS (stmt), 0, i, bi)
-               if (TREE_THIS_VOLATILE (referenced_var_lookup (i)))
-                 dead = false;
-
-           if (dead)
-             FOR_EACH_SSA_TREE_OPERAND (op, stmt, iter, SSA_OP_ALL_OPERANDS)
-               if (TREE_THIS_VOLATILE (op))
-                 dead = false;
-
-           /* Look for possible occurence var = indirect_ref (...) where
-              indirect_ref itself is volatile.  */
-
-           if (dead && TREE_THIS_VOLATILE (GIMPLE_STMT_OPERAND (stmt, 1)))
-             dead = false;
-
-           if (dead)
-             {
-               tree call = get_call_expr_in (stmt);
-
-               /* When LHS of var = call (); is dead, simplify it into
-                  call (); saving one operand.  */
-               if (TREE_CODE (stmt) == GIMPLE_MODIFY_STMT
-                   && call
-                   && TREE_SIDE_EFFECTS (call))
-                 {
-                   if (dump_file && (dump_flags & TDF_DETAILS))
-                     {
-                       fprintf (dump_file, "Deleted LHS of call: ");
-                       print_generic_stmt (dump_file, stmt, TDF_SLIM);
-                       fprintf (dump_file, "\n");
-                     }
-                   push_stmt_changes (bsi_stmt_ptr (bsi));
-                   TREE_BLOCK (call) = TREE_BLOCK (stmt);
-                   bsi_replace (&bsi, call, false);
-                   maybe_clean_or_replace_eh_stmt (stmt, call);
-                   mark_symbols_for_renaming (call);
-                   pop_stmt_changes (bsi_stmt_ptr (bsi));
-                 }
-               else
-                 {
-                   if (dump_file && (dump_flags & TDF_DETAILS))
-                     {
-                       fprintf (dump_file, "  Deleted dead store '");
-                       print_generic_expr (dump_file, stmt, dump_flags);
-                       fprintf (dump_file, "'\n");
-                     }
-                   removed = true;
-                   bsi_remove (&bsi, true);
-                   todo |= TODO_cleanup_cfg;
-                 }
-               todo |= TODO_remove_unused_locals | TODO_ggc_collect;
-             }
-         }
-       if (!removed)
-         bsi_next (&bsi);
-      }
-  BITMAP_FREE (variables_loaded);
-  return todo;
-}
-
-struct tree_opt_pass pass_simple_dse =
-{
-  "sdse",                              /* name */
-  NULL,                                        /* gate */
-  execute_simple_dse,                  /* execute */
-  NULL,                                        /* sub */
-  NULL,                                        /* next */
-  0,                                   /* static_pass_number */
-  0,                                   /* tv_id */
-  PROP_ssa,                            /* properties_required */
-  0,                                   /* properties_provided */
-  0,                                   /* properties_destroyed */
-  0,                                   /* todo_flags_start */
-  TODO_dump_func,                      /* todo_flags_finish */
-  0                                    /* letter */
-};