[Openmp-commits] [PATCH] D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available

[Gheorghe-Teodor Bercea via Phabricator via Openmp-commits]

gtbercea added a comment.

In https://reviews.llvm.org/D46842#1098463, @grokos wrote:

> I agree with Jonas, the initial version of the nvptx RTL that we tried to upstream was somewhat like what you try to do here and the consensus was that we do not want to have these dependencies.
>
> If you want to use the just-built clang to compile the BC lib, then the proper (albeit longer) solution is to build clang without the BC lib, then re-run cmake with `LIBOMPTARGET_NVPTX_ENABLE_BCLIB` enabled and `LIBOMPTARGET_NVPTX_SELECTED_CUDA_COMPILER` pointing to the just-built clang and re-run ninja.


George, I am well aware of the long-way and that is exactly what I'm trying to avoid.

The just-built compiler is a reasonable default in my opinion. I don't see a downside to it.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D46842