llvm-doe-org / llvm-project

The LLVM DOE Fork is a fork of upstream LLVM (https://github.com/llvm/llvm-project/) that hosts multiple DOE-funded projects. Contact information and other details for each project are described in the wiki.
https://github.com/llvm-doe-org/llvm-project/wiki/
Other
22 stars 13 forks source link

OpenMP enum variant support #20

Open jdenny-ornl opened 3 years ago

jdenny-ornl commented 3 years ago

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.