forked from llvm/llvm-project
-
Notifications
You must be signed in to change notification settings - Fork 11
Open
Labels
claccRelated to the Clacc project (OpenACC support for C/C++)Related to the Clacc project (OpenACC support for C/C++)clangRelated to the Clang front endRelated to the Clang front endupstreamableIdentifies work that is ready to be considered for LLVM upstreamIdentifies work that is ready to be considered for LLVM upstream
Description
As documented in the acc_on_device comments in Clacc's openmp/libacc2omp/src/include/openacc.h.var, Clacc's implementation of acc_on_device depends on OpenMP enum variants. While upstream Clang currently supports OpenMP enum variants sufficiently for Clacc's purposes, it is not clear whether the OpenMP spec requires this support.
We need to consider whether to redesign Clacc's acc_on_device implementation or to introduce wording into the Clang documentation and the OpenMP spec to ensure support is universal. While there's no OpenMP feature to upstream from Clacc, it might be worthwhile to discuss this issue further with the upstream LLVM/OpenMP community.
Metadata
Metadata
Assignees
Labels
claccRelated to the Clacc project (OpenACC support for C/C++)Related to the Clacc project (OpenACC support for C/C++)clangRelated to the Clang front endRelated to the Clang front endupstreamableIdentifies work that is ready to be considered for LLVM upstreamIdentifies work that is ready to be considered for LLVM upstream