re PR tree-optimization/87117 (ICE in…

re PR tree-optimization/87117 (ICE in eliminate_dom_walker::eliminate_cleanup(bool) at gcc/gcc/tree-ssa-sccvn.c:5431 since r263875)

2018-08-29  Richard Biener  <rguenther@suse.de>

	PR tree-optimization/87117
	* tree-ssa-sccvn.c (eliminate_dom_walker::eliminate_stmt): Only
	re-value-number released SSA VDEFs.

	* gfortran.dg/pr87117.f90: New testcase.

From-SVN: r263945

gcc/ChangeLog

 2018-08-29  Richard Biener  <rguenther@suse.de>
 
+	PR tree-optimization/87117
+	* tree-ssa-sccvn.c (eliminate_dom_walker::eliminate_stmt): Only
+	re-value-number released SSA VDEFs.
+
+2018-08-29  Richard Biener  <rguenther@suse.de>
+
 	PR tree-optimization/87126
 	* tree-ssa-sccvn.c (vn_reference_insert): Remove assert.
 

gcc/testsuite/ChangeLog

 2018-08-29  Richard Biener  <rguenther@suse.de>
 
+	PR tree-optimization/87117
+	* gfortran.dg/pr87117.f90: New testcase.
+
+2018-08-29  Richard Biener  <rguenther@suse.de>
+
 	PR tree-optimization/87126
 	* gcc.dg/tree-ssa/pr87126.c: New testcase.
 

gcc/testsuite/gfortran.dg/pr87117.f90

+! { dg-do compile }
+! { dg-options "-O" }
+program p
+   real(4) :: a, b
+   integer(4) :: n, m
+   equivalence (a, n)
+   a = 1024.0
+   m = 8
+   a = 1024.0
+   b = set_exponent(a, m)
+   n = 8
+   a = f(a, n)
+   b = set_exponent(a, m)
+end

gcc/tree-ssa-sccvn.c

@@ -4989,8 +4989,14 @@ eliminate_dom_walker::eliminate_stmt (basic_block b, gimple_stmt_iterator *gsi)
 	  propagate_tree_value_into_stmt (gsi, sprime);
 	  stmt = gsi_stmt (*gsi);
 	  update_stmt (stmt);
+	  /* In case the VDEF on the original stmt was released, value-number
+	     it to the VUSE.  This is to make vuse_ssa_val able to skip
+	     released virtual operands.  */
 	  if (vdef != gimple_vdef (stmt))
-	    VN_INFO (vdef)->valnum = vuse;
+	    {
+	      gcc_assert (SSA_NAME_IN_FREE_LIST (vdef));
+	      VN_INFO (vdef)->valnum = vuse;
+	    }
 
 	  /* If we removed EH side-effects from the statement, clean
 	     its EH information.  */
@@ -5268,7 +5274,10 @@ eliminate_dom_walker::eliminate_stmt (basic_block b, gimple_stmt_iterator *gsi)
 	    fprintf (dump_file, "  Removed AB side-effects.\n");
 	}
       update_stmt (stmt);
-      if (vdef != gimple_vdef (stmt))
+      /* In case the VDEF on the original stmt was released, value-number
+         it to the VUSE.  This is to make vuse_ssa_val able to skip
+	 released virtual operands.  */
+      if (vdef && SSA_NAME_IN_FREE_LIST (vdef))
 	VN_INFO (vdef)->valnum = vuse;
     }