[Openmp-commits] [PATCH] D44992: [OpenMP] enable bc file compilation using the latest clang
George Rokos via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Thu Mar 29 08:27:06 PDT 2018
grokos added a comment.
Right, I remember this issue. It is caused by the fact that clang does not support `__shared__` variables to be `extern`. This restriction was explicitly introduced here: https://reviews.llvm.org/D25125.
When upstreaming the nvptx RTL, this issue was discussed here: https://reviews.llvm.org/D14254?id=121993#inline-347991. That is why building the bc lib has been disabled by default. Back then you had mentioned that you had a fix for clang. What is the status of that fix?
I am really hesitant to accept the proposed solution because `rdc` is known to slow down CUDA code considerably. A clang-side fix would be much more preferable.
More information about the Openmp-commits