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

[Joseph Huber via Phabricator via cfe-commits]

jhuber6 added a comment.

In D143306#4145482 <https://reviews.llvm.org/D143306#4145482>, @jdoerfert wrote:

> @jhuber6 Can you look into the last part, creating such a clang.cfg as part of the build/install process?
>
> The fact that rpath is always set is a difference but I doubt it's that bad. We effectively point only to the llvm runtimes, which should be what users want anyway.
> If this is really a problem, maybe we can/should separate the OpenMP runtimes into a subfolder and rpath (-L) that one. Users get by default the OpenMP runtimes associated with the clang, but we would not pollute anything else.

If we do this as part of LLVM, what's the different between the config file and just setting `rpath` anyway? I figured that MaskRay was suggesting that each user curates their own and the LLVM install itself doesn't stipulate the `rpath` since it would lead to the same behavior.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143306