[PATCH] D122592: [OpenMP] Fix library path missing when using OpenMP

Mon Mar 28 10:27:41 PDT 2022

jdoerfert added a comment.

In D122592#3411812 <https://reviews.llvm.org/D122592#3411812>, @MaskRay wrote:

> In D122592#3411795 <https://reviews.llvm.org/D122592#3411795>, @ye-luo wrote:
>
>>> using the multiarch directory
>>
>> If we can cross compile libomp and libomptarget to the target system. We may have
>> lib/x86_64-unknown-linux-gnu/libomp.so
>> lib/aarch64-unknown-linux-gnu/libomp.so
>> Compile clang once but compile runtime library for multiple architectures.
>
> Yes, mostly for deambiguating cross compiling and multilib style -m32/-m64.
> In addition, I think all LLVM_ENABLE_RUNTIMES projects but openmp switched to lib/$triple/xxx.so for runtime libraries, e.g. libc++.so, libc++abi.so, libunwind.a.
> By switching openmp we will improve consistency. Downstream distributions can use the same way to install runtime libraries, instead of doing different things for openmp and non-openmp.

I'm all for it ;). Assuming @jhuber6 is not looking at that right now we need an issue to track this.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122592/new/

https://reviews.llvm.org/D122592