[Openmp-commits] [PATCH] D44992: [OpenMP] enable bc file compilation using the latest clang
Jonas Hahnfeld via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Wed Apr 4 10:01:04 PDT 2018
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.
More information about the Openmp-commits