[clang] [clang][AIX] Fix -print-runtime-dir on AIX (PR #104806)

via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 19 08:47:55 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-driver

@llvm/pr-subscribers-clang

Author: Jake Egan (jakeegan)

<details>
<summary>Changes</summary>

Currently the option prints a path to a nonexistent directory with the full triple, `lib/powerpc64-ibm-aix7.2.0.0`. It should only be `lib/aix`. 

---
Full diff: https://github.com/llvm/llvm-project/pull/104806.diff


1 Files Affected:

- (modified) clang/lib/Driver/ToolChain.cpp (+2-2) 


``````````diff
diff --git a/clang/lib/Driver/ToolChain.cpp b/clang/lib/Driver/ToolChain.cpp
index 0e8577b1115e3..4059730fa3665 100644
--- a/clang/lib/Driver/ToolChain.cpp
+++ b/clang/lib/Driver/ToolChain.cpp
@@ -831,8 +831,8 @@ std::optional<std::string> ToolChain::getRuntimePath() const {
   llvm::sys::path::append(P, "lib");
   if (auto Ret = getTargetSubDirPath(P))
     return Ret;
-  // Darwin does not use per-target runtime directory.
-  if (Triple.isOSDarwin())
+  // Darwin and AIX does not use per-target runtime directory.
+  if (Triple.isOSDarwin() || Triple.isOSAIX())
     return {};
   llvm::sys::path::append(P, Triple.str());
   return std::string(P);

``````````

</details>


https://github.com/llvm/llvm-project/pull/104806


More information about the cfe-commits mailing list