[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.

  rOMP OpenMP


More information about the Openmp-commits mailing list