PR bootstrap/83446 - Bootstrap failed on i686
authorMartin Sebor <msebor@redhat.com>
Mon, 18 Dec 2017 03:31:05 +0000 (03:31 +0000)
committerMartin Sebor <msebor@gcc.gnu.org>
Mon, 18 Dec 2017 03:31:05 +0000 (20:31 -0700)
gcc/testsuite/ChangeLog:

PR bootstrap/83446
* c-c++-common/Warray-bounds-3.c: Adjust.
* gcc.dg/Warray-bounds-25.c: New test.

gcc/ChangeLog:

PR bootstrap/83446
* gimple-ssa-warn-restrict.c
(builtin_memref::offset_out_of_bounds): Correct the handling of
anti-ranges.

From-SVN: r255772

gcc/ChangeLog
gcc/gimple-ssa-warn-restrict.c
gcc/testsuite/c-c++-common/Warray-bounds-3.c
gcc/testsuite/gcc.dg/Warray-bounds-25.c [new file with mode: 0644]

index 5b6b6772e46f789b8314b2ecc3ed42fddd7032ca..13b8cc70671e18ef91ffbe242fa1120e9054269c 100644 (file)
@@ -1,3 +1,10 @@
+2017-12-17  Martin Sebor  <msebor@redhat.com>
+
+       PR bootstrap/83446
+       * gimple-ssa-warn-restrict.c
+       (builtin_memref::offset_out_of_bounds): Correct the handling of
+       anti-ranges.
+
 2017-12-17  Sandra Loosemore  <sandra@codesourcery.com>
 
        * doc/invoke.texi (ARC Options): Add missing -mlra entry.
index f524e1dc3d53a732efe5fa246e2995f09456f33b..4d424735d2ac25684d9880d310d34316fc98aaed 100644 (file)
@@ -413,7 +413,9 @@ builtin_memref::offset_out_of_bounds (int strict, offset_int ooboff[2]) const
 
   if (DECL_P (base) && TREE_CODE (TREE_TYPE (base)) == ARRAY_TYPE)
     {
-      if (offrng[1] < offrng[0])
+      /* Check for offset in an anti-range with a negative lower bound.
+        For such a range, consider only the non-negative subrange.  */
+      if (offrng[1] < offrng[0] && offrng[1] < 0)
        offrng[1] = maxobjsize;
     }
 
index 8bb4f1c1b3e1f451f6f539664a965980c0a49439..85c8f6a6d0295b7f42c0894729fb0fa7502fa7a1 100644 (file)
@@ -123,8 +123,11 @@ void test_memcpy_bounds_anti_range (char *d, const char *s, size_t n)
      (yet).  */
   T (char, 9, a, a + SAR ( 1,  6), 3);   /* { dg-warning "forming offset \\\[9, 0] is out of the bounds \\\[0, 9] of object " "memcpy" { xfail *-*-* } } */
 
-  T (char, 9, a, a + SAR ( 2,  6), 3);   /* { dg-warning "forming offset 10 is out of the bounds \\\[0, 9] of object " "memcpy" } */
-  T (char, 9, a, a + SAR ( 3,  6), 3);   /* { dg-warning "forming offset 10 is out of the bounds \\\[0, 9] of object " "memcpy" } */
+  /* The range of offsets is the union of [0, 1] and [7, PTRDIFF_MAX]
+     of which the first subrange is valid and thus no warming for memcpy
+     is issued.  Similarly for the next test.  */
+  T (char, 9, a, a + SAR ( 2,  6), 3);
+  T (char, 9, a, a + SAR ( 3,  6), 3);
 
   T (char, 9, a, a + SAR (-1,  7), 3);   /* { dg-warning "forming offset \\\[10, 11] is out of the bounds \\\[0, 9] of object " "memcpy" } */
   T (char, 9, a, a + SAR (-2,  8), 3);   /* { dg-warning "forming offset \\\[10, 12] is out of the bounds \\\[0, 9] of object " "memcpy" } */
diff --git a/gcc/testsuite/gcc.dg/Warray-bounds-25.c b/gcc/testsuite/gcc.dg/Warray-bounds-25.c
new file mode 100644 (file)
index 0000000..37fb9cc
--- /dev/null
@@ -0,0 +1,33 @@
+/* PR tree-optimization/83446 - Bootstrap failed on i686
+   { dg-do compile }
+   { dg-options "-O2 -Warray-bounds" } */
+
+char a[4];
+
+void f0i (void *d, int n)
+{
+  if (n < 0) n = 0;
+
+  __builtin_memcpy (d, a + sizeof a - n, n);
+}
+
+void f0L (void *d, long n)
+{
+  if (n < 0) n = 0;
+
+  __builtin_memcpy (d, a + sizeof a - n, n);
+}
+
+void f0u (void *d, unsigned n)
+{
+  if (n < 0) n = 1;
+
+  __builtin_memcpy (d, a + sizeof a - n, n);   /* { dg-bogus "\\\[-Warray-bounds" } */
+}
+
+void f1lu (void *d, unsigned long n)
+{
+  if (n < 1) n = 1;
+
+  __builtin_memcpy (d, a + sizeof a - n, n);   /* { dg-bogus "\\\[-Warray-bounds" } */
+}