[Openmp-commits] [PATCH] D44992: [OpenMP] enable bc file compilation using the latest clang

[George Rokos via Phabricator via Openmp-commits]

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.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D44992