Revert D118493 "Set rpath on openmp executables"
authorFangrui Song <i@maskray.me>
Wed, 8 Mar 2023 18:18:40 +0000 (10:18 -0800)
committerFangrui Song <i@maskray.me>
Wed, 8 Mar 2023 18:18:40 +0000 (10:18 -0800)
commit555b572e3f407ac48b5f30fc06760cc4d0549977
treed3208c9b4198f9f5a843b5e85ee8552fe346337e
parenta66bc1c4a30ca958fee094bd7dcb6bc4b9cdd49f
Revert D118493 "Set rpath on openmp executables"

This reverts commit 9b9d08111b618d74574ba03e5cc3d752ecc56f55.
(Accepted by Jon https://reviews.llvm.org/D118493#4178250)

libc++, libc++abi, libunwind, and compiler-rt don't add the extra DT_RUNPATH,
it's strange for OpenMP to diverge.
Some build systems want to handle DT_RUNPATH themselves (e.g.
CMAKE_INSTALL_RPATH). Some distributions (e.g. Fedora) have policies against
DT_RUNPATH and the default DT_RUNPATH for OpenMP is causing trouble.

For users who don't want to specify rpath by themselves,
https://clang.llvm.org/docs/UsersManual.html#configuration-files
can be used to specify the default rpath, e.g.
specify -frtlib-add-rpath or -Wl,-rpath in bin/clang.cfg
clang/include/clang/Driver/Options.td
clang/lib/Driver/ToolChains/CommonArgs.cpp
clang/lib/Driver/ToolChains/CommonArgs.h
clang/test/OpenMP/Inputs/libomp.a [deleted file]
openmp/libomptarget/test/lit.cfg