[PATCH] D30015: [OpenMP] Add arch-specific directory to search path

[Pirama Arumuga Nainar via Phabricator via cfe-commits]

pirama added a subscriber: openmp-commits.
pirama marked an inline comment as done.
pirama 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.


https://reviews.llvm.org/D30015