sidneym created this revision.
sidneym added reviewers: kparzysz, bcain, adasgupt.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Use the default libpath when the target is linux-musl


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D76582

Files:
  clang/lib/Driver/ToolChains/Hexagon.cpp


Index: clang/lib/Driver/ToolChains/Hexagon.cpp
===================================================================
--- clang/lib/Driver/ToolChains/Hexagon.cpp
+++ clang/lib/Driver/ToolChains/Hexagon.cpp
@@ -499,11 +499,10 @@
 
   ToolChain::path_list &LibPaths = getFilePaths();
 
-  // Remove paths added by Linux toolchain. Currently Hexagon_TC really targets
-  // 'elf' OS type, so the Linux paths are not appropriate. When we actually
-  // support 'linux' we'll need to fix this up
-  LibPaths.clear();
-  getHexagonLibraryPaths(Args, LibPaths);
+  if (!Triple.isMusl()) {
+    LibPaths.clear();
+    getHexagonLibraryPaths(Args, LibPaths);
+  }
 }
 
 HexagonToolChain::~HexagonToolChain() {}


Index: clang/lib/Driver/ToolChains/Hexagon.cpp
===================================================================
--- clang/lib/Driver/ToolChains/Hexagon.cpp
+++ clang/lib/Driver/ToolChains/Hexagon.cpp
@@ -499,11 +499,10 @@
 
   ToolChain::path_list &LibPaths = getFilePaths();
 
-  // Remove paths added by Linux toolchain. Currently Hexagon_TC really targets
-  // 'elf' OS type, so the Linux paths are not appropriate. When we actually
-  // support 'linux' we'll need to fix this up
-  LibPaths.clear();
-  getHexagonLibraryPaths(Args, LibPaths);
+  if (!Triple.isMusl()) {
+    LibPaths.clear();
+    getHexagonLibraryPaths(Args, LibPaths);
+  }
 }
 
 HexagonToolChain::~HexagonToolChain() {}
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D76582: [Hexagon] Don'... Sid Manning via Phabricator via cfe-commits

Reply via email to