[llvm-branch-commits] [NFC][RISCV] Simplify the dynamic linker construction logic (PR #97383)
via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Mon Jul 1 22:35:11 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-driver
Author: Pengcheng Wang (wangpc-pp)
<details>
<summary>Changes</summary>
The format of dynamic linker is `ld-linux-{arch}-{abi}.so.1`, so
we can just get the arch name from arch type.
---
Full diff: https://github.com/llvm/llvm-project/pull/97383.diff
1 Files Affected:
- (modified) clang/lib/Driver/ToolChains/Linux.cpp (+3-7)
``````````diff
diff --git a/clang/lib/Driver/ToolChains/Linux.cpp b/clang/lib/Driver/ToolChains/Linux.cpp
index 49e029e7c9ab7..98a878e1d764d 100644
--- a/clang/lib/Driver/ToolChains/Linux.cpp
+++ b/clang/lib/Driver/ToolChains/Linux.cpp
@@ -568,16 +568,12 @@ std::string Linux::getDynamicLinker(const ArgList &Args) const {
Loader =
(tools::ppc::hasPPCAbiArg(Args, "elfv1")) ? "ld64.so.1" : "ld64.so.2";
break;
- case llvm::Triple::riscv32: {
- StringRef ABIName = tools::riscv::getRISCVABI(Args, Triple);
- LibDir = "lib";
- Loader = ("ld-linux-riscv32-" + ABIName + ".so.1").str();
- break;
- }
+ case llvm::Triple::riscv32:
case llvm::Triple::riscv64: {
+ StringRef ArchName = llvm::Triple::getArchTypeName(Arch);
StringRef ABIName = tools::riscv::getRISCVABI(Args, Triple);
LibDir = "lib";
- Loader = ("ld-linux-riscv64-" + ABIName + ".so.1").str();
+ Loader = ("ld-linux-" + ArchName + "-" + ABIName + ".so.1").str();
break;
}
case llvm::Triple::sparc:
``````````
</details>
https://github.com/llvm/llvm-project/pull/97383
More information about the llvm-branch-commits
mailing list