llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: None (Un1q32) <details> <summary>Changes</summary> Makes more sense to choose the linker based on binary format instead of OS --- Full diff: https://github.com/llvm/llvm-project/pull/160017.diff 1 Files Affected: - (modified) clang/lib/Driver/ToolChain.cpp (+1-1) ``````````diff diff --git a/clang/lib/Driver/ToolChain.cpp b/clang/lib/Driver/ToolChain.cpp index a9041d26c7ba4..66902fd8f2621 100644 --- a/clang/lib/Driver/ToolChain.cpp +++ b/clang/lib/Driver/ToolChain.cpp @@ -1149,7 +1149,7 @@ std::string ToolChain::GetLinkerPath(bool *LinkerIsLLD) const { return std::string(UseLinker); } else { llvm::SmallString<8> LinkerName; - if (Triple.isOSDarwin()) + if (Triple.isOSBinFormatMachO()) LinkerName.append("ld64."); else LinkerName.append("ld."); `````````` </details> https://github.com/llvm/llvm-project/pull/160017 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
