[Openmp-commits] [PATCH] D30015: Add arch-specific directory to search path
Michał Górny via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Fri Feb 17 02:12:05 PST 2017
mgorny added inline comments.
Comment at: lib/Driver/Tools.cpp:3267
+ if (llvm::sys::fs::is_directory(CandidateLibPath))
+ CmdArgs.push_back(Args.MakeArgString("-L" + CandidateLibPath));
> mgorny wrote:
> > Don't you also need rpath for it? Or is this purely for static runtime?
> I am doing this for a cross-compiling toolchain (Android NDK) where the actual rpath is not valid at runtime. The runtime is packaged with the application and made available to the loader behind the scenes.
> That said, I don't think providing an rpath doesn't hurt. I'll wait for input from @cbergstrom and OpenMP folks to see if this'd be useful.
@cbergstrom says: 'let them know that your comments are equivalent (or better) than mine'. But yeah, PathScale's adding rpaths for the libraries on our end, so this is consistent with what we do.
More information about the Openmp-commits