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

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added a comment.

In https://reviews.llvm.org/D44992#1057094, @gregrodgers wrote:

> So , will the deviceRTLs/nvptx change?   Instead of `extern __shared__`, what will it use for those data structures?


I don't question the flag, as I said `libomptarget-nvptx` was the main reason to implement that functionality. What's bad about this is adding this flag unconditionally, the build system needs to properly detect if the compiler supports it.

If you can't work on that right now, I hereby request this change to be reverted as it results in a build regression.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D44992