[clang] af744f0 - [LLD][COFF] Add LLVM toolchain library paths by default.
Tobias Hieta via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 14 05:38:12 PDT 2023
Author: Tobias Hieta
Date: 2023-07-14T14:37:24+02:00
New Revision: af744f0b84e2b6410be65277068b9033124c73b2
URL: https://github.com/llvm/llvm-project/commit/af744f0b84e2b6410be65277068b9033124c73b2
DIFF: https://github.com/llvm/llvm-project/commit/af744f0b84e2b6410be65277068b9033124c73b2.diff
LOG: [LLD][COFF] Add LLVM toolchain library paths by default.
We want lld-link to automatically find compiler-rt's and
libc++ when it's in the same directory as the rest of the
toolchain. This is because on Windows linking isn't done
via the clang driver - but instead invoked directly.
This prepends: <llvm>/lib <llvm>/lib/clang/XX/lib and
<llvm>/lib/clang/XX/lib/windows automatically to the library
search paths.
Related to #63827
Differential Revision: https://reviews.llvm.org/D151188
Added:
Modified:
clang/lib/Driver/Driver.cpp
lld/COFF/Driver.cpp
lld/COFF/Driver.h
lld/docs/ReleaseNotes.rst
lld/test/COFF/print-search-paths.s
Removed:
################################################################################
diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp
index f7d28ab46b680c..be8632b0954983 100644
--- a/clang/lib/Driver/Driver.cpp
+++ b/clang/lib/Driver/Driver.cpp
@@ -184,6 +184,8 @@ std::string Driver::GetResourcesPath(StringRef BinaryPath,
// path of the embedding binary, which for LLVM binaries will be in bin/.
// ../lib gets us to lib/ in both cases.
P = llvm::sys::path::parent_path(Dir);
+ // This search path is also created in the COFF driver of lld, so any
+ // changes here also needs to happen in lld/COFF/Driver.cpp
llvm::sys::path::append(P, CLANG_INSTALL_LIBDIR_BASENAME, "clang",
CLANG_VERSION_MAJOR_STRING);
}
diff --git a/lld/COFF/Driver.cpp b/lld/COFF/Driver.cpp
index 5f11d11ea72538..d7476e91e03e38 100644
--- a/lld/COFF/Driver.cpp
+++ b/lld/COFF/Driver.cpp
@@ -637,6 +637,31 @@ void LinkerDriver::detectWinSysRoot(const opt::InputArgList &Args) {
}
}
+void LinkerDriver::addClangLibSearchPaths(const std::string &argv0) {
+ std::string lldBinary = sys::fs::getMainExecutable(argv0.c_str(), nullptr);
+ SmallString<128> binDir(lldBinary);
+ sys::path::remove_filename(binDir); // remove lld-link.exe
+ StringRef rootDir = sys::path::parent_path(binDir); // remove 'bin'
+
+ SmallString<128> libDir(rootDir);
+ sys::path::append(libDir, "lib");
+ // We need to prepend the paths here in order to make sure that we always
+ // try to link the clang versions of the builtins over the ones supplied by MSVC.
+ searchPaths.insert(searchPaths.begin(), saver().save(libDir.str()));
+
+ // Add the resource dir library path
+ SmallString<128> runtimeLibDir(rootDir);
+ sys::path::append(runtimeLibDir, "lib", "clang", std::to_string(LLVM_VERSION_MAJOR), "lib");
+ searchPaths.insert(searchPaths.begin(), saver().save(runtimeLibDir.str()));
+
+ // Resource dir + osname, which is hardcoded to windows since we are in the
+ // COFF driver.
+ SmallString<128> runtimeLibDirWithOS(runtimeLibDir);
+ sys::path::append(runtimeLibDirWithOS, "windows");
+ searchPaths.insert(searchPaths.begin(), saver().save(runtimeLibDirWithOS.str()));
+
+}
+
void LinkerDriver::addWinSysRootLibSearchPaths() {
if (!diaPath.empty()) {
// The DIA SDK always uses the legacy vc arch, even in new MSVC versions.
@@ -1543,6 +1568,7 @@ void LinkerDriver::linkerMain(ArrayRef<const char *> argsArr) {
detectWinSysRoot(args);
if (!args.hasArg(OPT_lldignoreenv) && !args.hasArg(OPT_winsysroot))
addLibSearchPaths();
+ addClangLibSearchPaths(argsArr[0]);
// Handle /ignore
for (auto *arg : args.filtered(OPT_ignore)) {
diff --git a/lld/COFF/Driver.h b/lld/COFF/Driver.h
index 6fb3abf260d6cf..17d2e6a483dc33 100644
--- a/lld/COFF/Driver.h
+++ b/lld/COFF/Driver.h
@@ -84,6 +84,8 @@ class LinkerDriver {
// config->machine has been set.
void addWinSysRootLibSearchPaths();
+ void addClangLibSearchPaths(const std::string &argv0);
+
// Used by the resolver to parse .drectve section contents.
void parseDirectives(InputFile *file);
diff --git a/lld/docs/ReleaseNotes.rst b/lld/docs/ReleaseNotes.rst
index 4bd5a628d2b902..9435f23a7af616 100644
--- a/lld/docs/ReleaseNotes.rst
+++ b/lld/docs/ReleaseNotes.rst
@@ -42,6 +42,12 @@ COFF Improvements
current directory.
I.e. ``lld-link /libpath:c:\relative\root relative\path\my.lib`` where before
we would have to do ``lld-link /libpath:c:\relative\root\relative\path my.lib``
+* lld-link learned -print-search-paths that will print all the paths where it will
+ search for libraries.
+* By default lld-link will now search for libraries in the toolchain directories.
+ Specifically it will search:
+ ``<toolchain>/lib``, ``<toolchain>/lib/clang/<version>/lib`` and
+ ``<toolchain>/lib/clang/<version>/lib/windows``.
MinGW Improvements
------------------
diff --git a/lld/test/COFF/print-search-paths.s b/lld/test/COFF/print-search-paths.s
index a484ac5b0fb164..5c292aafc68a11 100644
--- a/lld/test/COFF/print-search-paths.s
+++ b/lld/test/COFF/print-search-paths.s
@@ -4,11 +4,17 @@
# RUN: llvm-mc -triple i686-windows-msvc %s -filetype=obj -o %t_32.obj
# RUN: lld-link -safeseh:no /dll /noentry /winsysroot:%t.dir/sysroot /vctoolsversion:1.1.1.1 /winsdkversion:10.0.1 %t_32.obj -print-search-paths | FileCheck -DSYSROOT=%t.dir -check-prefix=X86 %s
# CHECK: Library search paths:
+# CHECK: [[CPATH:.*]]lib{{[/\\]}}clang{{[/\\]}}{{[0-9]+}}{{[/\\]}}lib{{[/\\]}}windows
+# CHECK: [[CPATH]]lib{{[/\\]}}clang{{[/\\]}}{{[0-9]+}}{{[/\\]}}lib
+# CHECK: [[CPATH]]lib
# CHECK: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}VC{{[/\\]}}Tools{{[/\\]}}MSVC{{[/\\]}}1.1.1.1{{[/\\]}}lib{{[/\\]}}x64
# CHECK: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}VC{{[/\\]}}Tools{{[/\\]}}MSVC{{[/\\]}}1.1.1.1{{[/\\]}}atlmfc{{[/\\]}}lib{{[/\\]}}x64
# CHECK: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}Windows Kits{{[/\\]}}10{{[/\\]}}Lib{{[/\\]}}10.0.1{{[/\\]}}ucrt{{[/\\]}}x64
# CHECK: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}Windows Kits{{[/\\]}}10{{[/\\]}}Lib{{[/\\]}}10.0.1{{[/\\]}}um{{[/\\]}}x64
# X86: Library search paths:
+# X86: [[CPATH:.*]]lib{{[/\\]}}clang{{[/\\]}}{{[0-9]+}}{{[/\\]}}lib{{[/\\]}}windows
+# X86: [[CPATH]]lib{{[/\\]}}clang{{[/\\]}}{{[0-9]+}}{{[/\\]}}lib
+# X86: [[CPATH]]lib
# X86: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}VC{{[/\\]}}Tools{{[/\\]}}MSVC{{[/\\]}}1.1.1.1{{[/\\]}}lib{{[/\\]}}x86
# X86: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}VC{{[/\\]}}Tools{{[/\\]}}MSVC{{[/\\]}}1.1.1.1{{[/\\]}}atlmfc{{[/\\]}}lib{{[/\\]}}x86
# X86: [[SYSROOT]]{{[/\\]}}sysroot{{[/\\]}}Windows Kits{{[/\\]}}10{{[/\\]}}Lib{{[/\\]}}10.0.1{{[/\\]}}ucrt{{[/\\]}}x86
More information about the cfe-commits
mailing list