]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix OpenMP's target update directive in templated code.
authorThomas Schwinge <thomas@codesourcery.com>
Wed, 29 Apr 2015 09:10:13 +0000 (11:10 +0200)
committerThomas Schwinge <tschwinge@gcc.gnu.org>
Wed, 29 Apr 2015 09:10:13 +0000 (11:10 +0200)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++98 (internal compiler error)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++98 (test for excess errors)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++11 (internal compiler error)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++11 (test for excess errors)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++14 (internal compiler error)
    FAIL: g++.dg/gomp/tpl-target-update.C  -std=c++14 (test for excess errors)

    [...]/source-gcc/gcc/testsuite/g++.dg/gomp/tpl-target-update.C: In instantiation of 'void f(T, T) [with T = int]':
    [...]/source-gcc/gcc/testsuite/g++.dg/gomp/tpl-target-update.C:19:9:   required from here
    [...]/source-gcc/gcc/testsuite/g++.dg/gomp/tpl-target-update.C:10:9: internal compiler error: tree check: expected oacc_parallel or oacc_kernels or oacc_data or oacc_host_data or omp_parallel or omp_task or omp_for or omp_simd or cilk_simd or cilk_for or omp_distribute or oacc_loop or omp_teams or omp_target_data or omp_target or omp_sections or omp_single, have omp_target_update in tsubst_expr, at cp/pt.c:14209
    0xf5aae1 tree_range_check_failed(tree_node const*, char const*, int, char const*, tree_code, tree_code)
            [...]/source-gcc/gcc/tree.c:9384
    0x66e201 tree_range_check
            [...]/source-gcc/gcc/tree.h:2979
    0x66e201 tsubst_expr
            [...]/source-gcc/gcc/cp/pt.c:14209
    0x6695e3 tsubst_expr
            [...]/source-gcc/gcc/cp/pt.c:13752
    0x66ac07 tsubst_expr
            [...]/source-gcc/gcc/cp/pt.c:13938
    0x667c41 instantiate_decl(tree_node*, int, bool)
            [...]/source-gcc/gcc/cp/pt.c:20367
    0x6ae386 instantiate_pending_templates(int)
            [...]/source-gcc/gcc/cp/pt.c:20484
    0x6edc3d cp_write_global_declarations()
            [...]/source-gcc/gcc/cp/decl2.c:4456

Backport from trunk r222564:

gcc/cp/
* pt.c (tsubst_expr) <OMP_TARGET_UPDATE>: Use
OMP_TARGET_UPDATE_CLAUSES instead of OMP_CLAUSES.
gcc/testsuite/
* g++.dg/gomp/tpl-target-update.C: New file.

From-SVN: r222566

gcc/cp/ChangeLog
gcc/cp/pt.c
gcc/testsuite/ChangeLog
gcc/testsuite/g++.dg/gomp/tpl-target-update.C [new file with mode: 0644]

index 1f029ad09e845dc4f2681445a32927d077360260..8232a3371a493999f763926c50d0816139569b93 100644 (file)
@@ -1,3 +1,12 @@
+2015-04-29  Thomas Schwinge  <thomas@codesourcery.com>
+
+       Backport from trunk r222564:
+
+       2015-04-29  Thomas Schwinge  <thomas@codesourcery.com>
+
+       * pt.c (tsubst_expr) <OMP_TARGET_UPDATE>: Use
+       OMP_TARGET_UPDATE_CLAUSES instead of OMP_CLAUSES.
+
 2015-04-23  Marek Polacek  <polacek@redhat.com>
 
        PR c++/65727
index fc8300ae2a7b06e8942e47bf69d241bb69c365fd..60e96710eb638327258baade207f58a849633de7 100644 (file)
@@ -13912,7 +13912,7 @@ tsubst_expr (tree t, tree args, tsubst_flags_t complain, tree in_decl,
       tmp = tsubst_omp_clauses (OMP_TARGET_UPDATE_CLAUSES (t), false,
                                args, complain, in_decl);
       t = copy_node (t);
-      OMP_CLAUSES (t) = tmp;
+      OMP_TARGET_UPDATE_CLAUSES (t) = tmp;
       add_stmt (t);
       break;
 
index 09d3521ea7f66868f3b91605761e1a9a9d80b56c..8eba2c2274b05d6e860f60706b206cfd50b927cb 100644 (file)
@@ -1,3 +1,11 @@
+2015-04-29  Thomas Schwinge  <thomas@codesourcery.com>
+
+       Backport from trunk r222564:
+
+       2015-04-29  Thomas Schwinge  <thomas@codesourcery.com>
+
+       * g++.dg/gomp/tpl-target-update.C: New file.
+
 2015-04-28  Tejas Belagod  <tejas.belagod@arm.com>
 
        Backport from mainline
diff --git a/gcc/testsuite/g++.dg/gomp/tpl-target-update.C b/gcc/testsuite/g++.dg/gomp/tpl-target-update.C
new file mode 100644 (file)
index 0000000..6226ebf
--- /dev/null
@@ -0,0 +1,20 @@
+// { dg-do compile }
+
+template <typename T>
+void f(T A, T B)
+{
+  extern int *v;
+  T a = 2;
+  T b = 4;
+
+#pragma omp target update to(v[a:b])
+  v[a] = 0;
+
+#pragma omp target update to(v[A:B])
+  v[a] = 0;
+}
+
+void g()
+{
+  f(1, 5);
+}