[PATCH] D143306: [Driver] Default to -fno-openmp-implicit-rpath

Wed Feb 22 13:12:27 PST 2023

tstellar added a comment.

In D143306#4145432 <https://reviews.llvm.org/D143306#4145432>, @MaskRay wrote:

> This is the point. Specifying a driver option to use libc++/libc++abi/libunwind doesn't magically change `DT_RUNPATH`. This is exactly the behavior a user wants for a system Clang.
> It does make users with a non-system Clang inconvenient but that's the point that such users should specify rpath by themselves.
> openmp should not diverge from libc++/libc++abi/libunwind in this regard.

To me this is a really strong argument in favor of this change.  Why does libomp need to be different here?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143306