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

Mon Mar 28 10:29:34 PDT 2022

jhuber6 added a comment.

In D122592#3411877 <https://reviews.llvm.org/D122592#3411877>, @jdoerfert wrote:

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

Sounds good, I won't look at it immediately but I could set up an issue to track the transition.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122592