diff options
author | Jakub Jelinek <jakub@redhat.com> | 2023-01-05 11:57:30 +0100 |
---|---|---|
committer | Jakub Jelinek <jakub@redhat.com> | 2023-01-05 11:57:30 +0100 |
commit | 29c3218618ef6177dc33871b26c8fbd9b21eabe1 (patch) | |
tree | 0affb30a38270beec5af10756d87bc83e047db90 /libgomp | |
parent | 72ce780a497eb3e5efe7a79ea5f21f8dd6858f7f (diff) | |
download | gcc-29c3218618ef6177dc33871b26c8fbd9b21eabe1.zip gcc-29c3218618ef6177dc33871b26c8fbd9b21eabe1.tar.gz gcc-29c3218618ef6177dc33871b26c8fbd9b21eabe1.tar.bz2 |
openmp: Fix up finish_omp_target_clauses [PR108286]
The comment in the loop says that we shouldn't add a map clause if such
a clause exists already, but the loop was actually using OMP_CLAUSE_DECL
on any clause. Target construct can have various clauses which don't
have OMP_CLAUSE_DECL at all (e.g. nowait, device or if) or clause
where it means something different (e.g. privatization clauses, allocate,
depend).
So, only check OMP_CLAUSE_DECL on OMP_CLAUSE_MAP clauses.
2023-01-05 Jakub Jelinek <jakub@redhat.com>
PR c++/108286
* semantics.cc (finish_omp_target_clauses): Ignore clauses other than
OMP_CLAUSE_MAP.
* testsuite/libgomp.c++/pr108286.C: New test.
Diffstat (limited to 'libgomp')
-rw-r--r-- | libgomp/testsuite/libgomp.c++/pr108286.C | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/libgomp/testsuite/libgomp.c++/pr108286.C b/libgomp/testsuite/libgomp.c++/pr108286.C new file mode 100644 index 0000000..ee88c2f --- /dev/null +++ b/libgomp/testsuite/libgomp.c++/pr108286.C @@ -0,0 +1,29 @@ +// PR c++/108286 +// { dg-do run } + +struct S { + int + foo () + { + int res = 0; +#pragma omp target map(size, ptr[:size], res) nowait + res = ptr[size - 1]; +#pragma omp taskwait + return res; + } + + unsigned size; + int *ptr; +}; + +int +main () +{ + S s; + int buf[5]; + s.size = 5; + s.ptr = buf; + buf[4] = 42; + if (s.foo () != 42) + __builtin_abort (); +} |