beanz added a comment.

This looks like it is in the direction I would expect. Please let me know once 
the final patch is ready.



================
Comment at: lib/Driver/Tools.cpp:2021
   std::string Plugin =
-      ToolChain.getDriver().Dir + "/../lib" CLANG_LIBDIR_SUFFIX "/LLVMgold.so";
+      ToolChain.getDriver().Dir + "/../lib" LLVM_RUNTIMES_LIBDIR_SUFFIX 
"/LLVMgold.so";
   CmdArgs.push_back(Args.MakeArgString(Plugin));
----------------
mgorny wrote:
> Note: I don't know if it's better to use LLVM_RUNTIMES_LIBDIR_SUFFIX here, or 
> just #define LLVM_LIBDIR_SUFFIX in addition to that.
This should be `LLVM_LIBDIR_SUFFIX` as per Chandler's comments.


https://reviews.llvm.org/D23752



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to