re PR c++/86291 (OpenMP incorrect for-loop collapsing with iterators and at least...

	PR c++/86291
	* parser.c (cp_parser_omp_for_loop_init): Change for_block argument
	type from vec<tree, va_gc> * to vec<tree, va_gc> *&.

	* testsuite/libgomp.c++/pr86291.C: New test.

From-SVN: r262140

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 9c7127b35c89de229dfdfb1a7eea036230a8c79e..738b043732a33cd5442afe39a23ce66af83efb4f 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2018-06-26  Jakub Jelinek  <jakub@redhat.com>
+
+       PR c++/86291
+       * parser.c (cp_parser_omp_for_loop_init): Change for_block argument
+       type from vec<tree, va_gc> * to vec<tree, va_gc> *&.
+
 2018-06-25  Jakub Jelinek  <jakub@redhat.com>
 
        Backported from mainline

diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c
index 4a49a2c6d747d7b9995120e934731c77403f57d2..48e921adfdf4391f2f7a90f2a26bac89bed4d55f 100644 (file)
--- a/gcc/cp/parser.c
+++ b/gcc/cp/parser.c
@@ -33382,7 +33382,7 @@ static tree
 cp_parser_omp_for_loop_init (cp_parser *parser,
                             enum tree_code code,
                             tree &this_pre_body,
-                            vec<tree, va_gc> *for_block,
+                            vec<tree, va_gc> *&for_block,
                             tree &init,
                             tree &orig_init,
                             tree &decl,

diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog
index 53d06ce117de5e02319ac886eb0fd3e49e87221c..a1e2a3cb179657004364baad29462b2dc9807357 100644 (file)
--- a/libgomp/ChangeLog
+++ b/libgomp/ChangeLog
@@ -1,3 +1,8 @@
+2018-06-26  Jakub Jelinek  <jakub@redhat.com>
+
+       PR c++/86291
+       * testsuite/libgomp.c++/pr86291.C: New test.
+
 2018-06-25  Jakub Jelinek  <jakub@redhat.com>
 
        Backported from mainline

diff --git a/libgomp/testsuite/libgomp.c++/pr86291.C b/libgomp/testsuite/libgomp.c++/pr86291.C
new file mode 100644 (file)
index 0000000..89c4b2d
--- /dev/null
+++ b/libgomp/testsuite/libgomp.c++/pr86291.C
@@ -0,0 +1,51 @@
+// PR c++/86291
+// { dg-do run }
+// { dg-additional-options "-std=c++11" }
+
+extern "C" void abort ();
+
+struct I
+{
+  using size_type = __SIZE_TYPE__;
+  using difference_type = __PTRDIFF_TYPE__;
+  using value_type = int;
+  using reference = int &;
+  using pointer = int *;
+  static I begin () { return I{}; }
+  static I end () { I res; res.pos = res.num; return res; }
+  I &operator++ () { ++pos; return *this; }
+  reference operator* () const { return val; }
+  I &operator+= (size_type diff) { pos += diff; return *this; }
+  friend bool operator< (const I &a, const I &b) { return a.pos < b.pos; }
+  friend difference_type operator- (const I &a, const I &b) { return a.pos - b.pos; }
+  size_type pos = 0;
+  size_type num = 1;
+  mutable int val = 0;
+};
+
+int c;
+
+int
+main ()
+{
+#pragma omp parallel for collapse(10)
+  for (auto i = I::begin (); i < I::end (); ++i)
+    for (auto j = I::begin (); j < I::end (); ++j)
+      for (auto k = I::begin (); k < I::end (); ++k)
+       for (auto l = I::begin (); l < I::end (); ++l)
+         for (auto m = I::begin (); m < I::end (); ++m)
+           for (auto n = I::begin (); n < I::end (); ++n)
+             for (auto o = I::begin (); o < I::end (); ++o)
+               for (auto p = I::begin (); p < I::end (); ++p)
+                 for (auto q = I::begin (); q < I::end (); ++q)
+                   for (auto r = I::begin (); r < I::end (); ++r)
+                     {
+                       if (*i != 0 || *j != 0 || *k != 0 || *l != 0 || *m != 0
+                           || *n != 0 || *o != 0 || *p != 0 || *q != 0 || *r != 0)
+                         abort ();
+                       #pragma omp atomic
+                         c++;
+                     }
+  if (c != 1)
+    abort ();
+}