[clang] fix clang linker selection (PR #160017)

via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 21 15:53:40 PDT 2025


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


More information about the cfe-commits mailing list