[Openmp-commits] [PATCH] D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user

[George Rokos via Phabricator via Openmp-commits]

grokos added a comment.

In https://reviews.llvm.org/D45220#1058805, @guansong wrote:

> This is only for compiler and linker used for building bclib. It is not the default build compiler.
>
> With this patch, if a user does not specify compiler and linker for the bc library, but uses  -DLIBOMPTARGET_NVPTX_ENABLE_BCLIB=1, the cmake script will try to locate if the build compiler is clang and can be used for building the bc lib.


We had discussed this exact issue when upstreaming the nvptx RTL and the concesus was that we don't want to use the just-built compiler for the BC lib. If the installed compiler cannot build the BC lib, then the user must install the just-built compiler first and then use it to produce the BC lib.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D45220