[PATCH] D143306: [Driver] Default to -fno-openmp-implicit-rpath
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 22 11:36:12 PST 2023
MaskRay added a comment.
In D143306#4144518 <https://reviews.llvm.org/D143306#4144518>, @JonChesterfield wrote:
> Marking this as "no" because as far as I can tell it'll stop anyone running openmp built from source which constitutes a severe regression and I am struggling to find information on what Fedora are doing here. This link https://fedoraproject.org/wiki/Changes/Broken_RPATH_will_fail_rpmbuild suggests changing clang to not set rpath when it would be aiming at a "system directory", which is unfortunately platform specific magic strings, would be fine. That is, maybe Fedora is OK with setting RPATH as long as it isn't set to /usr/lib64 or possibly other unspecified strings.
>
> The use case I want to preserve is people running clang -fopenmp from a local install and without setting environment variables. That means the binary needs to find the shared libraries from that local install and not unrelated files with the same name that happen to be under /usr somewhere.
Perhaps we simply fundamentally disagree with each other. My opinion is that D118493 <https://reviews.llvm.org/D118493> should be reverted.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143306/new/
https://reviews.llvm.org/D143306
More information about the cfe-commits
mailing list