[CLANG][OpenMP] Add support for OpenMP6.0 transparent clause #166810
Conversation
|
✅ With the latest revision this PR passed the C/C++ code formatter. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
|
Do we have support for this in runtime library? If not, this definitely requires initial support in runtime at first |
No. Should the runtime change be a preliminary PR to this one? I am assuming that a |
This comment was marked as outdated.
This comment was marked as outdated.
@alexey-bataev Would adding the flag in #168873 be enough for this PR to be merged? |
…168873) Clang is adding support for the new `OpenMP transparent` clause on `task` and `taskloop` directives. The parsing and semantic handling for this clause is introduced in #166810 . To allow the compiler to communicate this clause to the `OpenMP` runtime, a dedicated bit in `kmp_tasking_flags` is required. This patch adds a new compiler-reserved bit `transparent` to the` kmp_tasking_flags` structure.
Tht's is correct, it takes an "expression of OpenMP impex type" (not a keyword), which is an enum value. void func(omp_impex_t arg) {
#pragma omp task transparent(arg)
... |
…ing_flags (#168873) Clang is adding support for the new `OpenMP transparent` clause on `task` and `taskloop` directives. The parsing and semantic handling for this clause is introduced in llvm/llvm-project#166810 . To allow the compiler to communicate this clause to the `OpenMP` runtime, a dedicated bit in `kmp_tasking_flags` is required. This patch adds a new compiler-reserved bit `transparent` to the` kmp_tasking_flags` structure.
4 participants
Add basic parsing and semantic support for
transparentclause fortaskandtaskloopdirectives described inSection 17.9.6of https://www.openmp.org/wp-content/uploads/OpenMP-API-Specification-6-0.pdf .