re PR tree-optimization/63445 (request: make -Wstrict-overflow avoid a class of false...
authorRichard Biener <rguenther@suse.de>
Thu, 9 Oct 2014 08:20:53 +0000 (08:20 +0000)
committerRichard Biener <rguenth@gcc.gnu.org>
Thu, 9 Oct 2014 08:20:53 +0000 (08:20 +0000)
2014-10-09  Richard Biener  <rguenther@suse.de>

PR tree-optimization/63445
* tree-vrp.c (simplify_cond_using_ranges): Only warn about
overflow for non-equality compares.

* gcc.dg/Wstrict-overflow-26.c: New testcase.

From-SVN: r216028

gcc/ChangeLog
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.dg/Wstrict-overflow-26.c [new file with mode: 0644]
gcc/tree-vrp.c

index 85bc3619af5b2bd773f64be4ee019b40b0c640bf..b8b9f3019a802d44b337f3215fce93dc39173beb 100644 (file)
@@ -1,3 +1,9 @@
+2014-10-09  Richard Biener  <rguenther@suse.de>
+
+       PR tree-optimization/63445
+       * tree-vrp.c (simplify_cond_using_ranges): Only warn about
+       overflow for non-equality compares.
+
 2014-10-09  Uros Bizjak  <ubizjak@gmail.com>
 
        PR rtl-optimization/57003
index b282bc7a4299a9ff574e32610f73fec8e6ca1732..a8870ea3de711dd7a11bf77663189f80544739b1 100644 (file)
@@ -1,3 +1,8 @@
+2014-10-09  Richard Biener  <rguenther@suse.de>
+
+       PR tree-optimization/63445
+       * gcc.dg/Wstrict-overflow-26.c: New testcase.
+
 2014-10-08  Pat Haugen  <pthaugen@us.ibm.com>
 
        * gcc.dg/vmx/3c-01a.c: Add default options from vmx.exp.
diff --git a/gcc/testsuite/gcc.dg/Wstrict-overflow-26.c b/gcc/testsuite/gcc.dg/Wstrict-overflow-26.c
new file mode 100644 (file)
index 0000000..ef805b5
--- /dev/null
@@ -0,0 +1,16 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -Wstrict-overflow" } */
+
+int
+f (int i, int j)
+{
+  unsigned int c = 0;                                                          
+  if (i < j)
+    {
+      unsigned int n = j - i;
+      unsigned int i;
+      for (i = 0; i < n; i++) /* { dg-bogus "signed overflow" } */
+       c++;
+    }
+  return c;
+}
index 011db78d7344e196852c8b61bf6b9e4c7e2572b7..7ca05284ede099a4cb3aa1f6117f6e2910a78e9e 100644 (file)
@@ -9189,8 +9189,9 @@ simplify_cond_using_ranges (gimple stmt)
              /* If the range overflowed and the user has asked for warnings
                 when strict overflow semantics were used to optimize code,
                 issue an appropriate warning.  */
-             if ((is_negative_overflow_infinity (vr->min)
-                  || is_positive_overflow_infinity (vr->max))
+             if (cond_code != EQ_EXPR && cond_code != NE_EXPR
+                 && (is_negative_overflow_infinity (vr->min)
+                     || is_positive_overflow_infinity (vr->max))
                  && issue_strict_overflow_warning (WARN_STRICT_OVERFLOW_CONDITIONAL))
                {
                  location_t location;