[Openmp-commits] [PATCH] D129875: [Libomptarget] Make libomptarget an LLVM library

Ye Luo via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jul 15 14:55:20 PDT 2022

ye-luo added a comment.

In D129875#3656431 <https://reviews.llvm.org/D129875#3656431>, @jhuber6 wrote:

> In D129875#3656403 <https://reviews.llvm.org/D129875#3656403>, @ye-luo wrote:
>> Prefer additional dependency remains optional.
> The only downsides I see from this approach is that user code will depend on LLVM libraries if LLVM is build using shared libraries. But I'm pretty sure the default build method in LLVM is to use static libraries so that shouldn't propagate to users just wishing to build `libomptarget` alone. We may have some vendors complain about the extra dependency, but I'm personally not aware of anyone using Clang/LLVM for OpenMP offloading that isn't building it with LLVM or getting it through a release that was build with LLVM.

That is exactly my concern. Won't be happy to see llvm libraries on the application side. If it is internal to the libomptarget, that is fine. It seems all the llvm libraries are static in my builds. Then it is likely OK.

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list