re PR tree-optimization/61383 (wrong code at -O2 and -O3 on x86_64-linux-gnu)

2014-06-04  Richard Biener  <rguenther@suse.de>

	PR tree-optimization/61383
	* tree-ssa-ifcombine.c (bb_no_side_effects_p): Make sure
	stmts can't trap.

	* gcc.dg/torture/pr61383-1.c: New testcase.

From-SVN: r211232

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 343050c0d0a1febf010aaf1773659ce59913e715..a6eeb63d852a1b6fef39f5e50560d5d7fd982955 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2014-06-04  Richard Biener  <rguenther@suse.de>
+
+       PR tree-optimization/61383
+       * tree-ssa-ifcombine.c (bb_no_side_effects_p): Make sure
+       stmts can't trap.
+
 2014-06-03  Andrey Belevantsev  <abel@ispras.ru>
 
        Backport from mainline

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0bba1d5680191bba2f8c1b80747921f0b353442c..b66a7aa40f970d488a6f57607de92034c665adbe 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2014-06-04  Richard Biener  <rguenther@suse.de>
+
+       PR tree-optimization/61383
+       * gcc.dg/torture/pr61383-1.c: New testcase.
+
 2014-06-03  Andrey Belevantsev  <abel@ispras.ru>
 
        Backport from mainline

diff --git a/gcc/testsuite/gcc.dg/torture/pr61383-1.c b/gcc/testsuite/gcc.dg/torture/pr61383-1.c
new file mode 100644 (file)
index 0000000..d9a0a0b
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/torture/pr61383-1.c
@@ -0,0 +1,35 @@
+/* { dg-do run } */
+
+int a, b = 1, c, d, e, f, g;
+
+int
+fn1 ()
+{
+  int h;
+  for (;;)
+    {
+      g = b;
+      g = g ? 0 : 1 % g;
+      e = a + 1;
+      for (; d < 1; d = e)
+       {
+         if (f == 0)
+           h = 0;
+         else
+           h = 1 % f;
+         if (f < 1)
+           c = 0;
+         else if (h)
+           break;
+       }
+      if (b)
+       return 0;
+    }
+}
+
+int
+main ()
+{
+  fn1 ();
+  return 0;
+}

diff --git a/gcc/tree-ssa-ifcombine.c b/gcc/tree-ssa-ifcombine.c
index cc06ca13b1e280635374895af7aa51e9fecd637a..ed6ea8261d0758b0c77c1e1f08eb9dc66268b045 100644 (file)
--- a/gcc/tree-ssa-ifcombine.c
+++ b/gcc/tree-ssa-ifcombine.c
@@ -105,7 +105,11 @@ bb_no_side_effects_p (basic_block bb)
     {
       gimple stmt = gsi_stmt (gsi);
 
+      if (is_gimple_debug (stmt))
+       continue;
+
       if (gimple_has_side_effects (stmt)
+         || gimple_could_trap_p (stmt)
          || gimple_vuse (stmt))
        return false;
     }