diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2018-03-05 09:47:54 -0800 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2018-03-09 07:29:08 -0800 |
| commit | be902e7168505954b85e1bbb35322f8df8a29c19 (patch) | |
| tree | 1c68f40b0ae7d2c2b8a6e51064e18ab1dc601ca1 /src/bootstrap/lib.rs | |
| parent | fedce67cd21dc08ece5a484fe1a060346acac98a (diff) | |
rustbuild: Fix MSBuild location of `llvm-config.exe`
For LLD integration the path to `llvm-config` needed to change to inside the build directory itself (for whatever reason) but the build directory is different on MSBuild than it is on `ninja` for MSVC builds, so the path to `llvm-config.exe` was actually wrong and not working! This commit removes the `Build::llvm_config` function in favor of the source of truth, the `Llvm` build step itself. The build step was then updated to find the right build directory for MSBuild as well as `ninja` for where `llvm-config.exe` is located. Closes #48749
Diffstat (limited to 'src/bootstrap/lib.rs')
| -rw-r--r-- | src/bootstrap/lib.rs | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/src/bootstrap/lib.rs b/src/bootstrap/lib.rs index f3d9246c6fc..43f0c947308 100644 --- a/src/bootstrap/lib.rs +++ b/src/bootstrap/lib.rs @@ -532,20 +532,6 @@ impl Build { } } - /// Returns the path to `llvm-config` for the specified target. - /// - /// If a custom `llvm-config` was specified for target then that's returned - /// instead. - fn llvm_config(&self, target: Interned<String>) -> PathBuf { - let target_config = self.config.target_config.get(&target); - if let Some(s) = target_config.and_then(|c| c.llvm_config.as_ref()) { - s.clone() - } else { - self.llvm_out(self.config.build).join("bin") - .join(exe("llvm-config", &*target)) - } - } - /// Returns the path to `FileCheck` binary for the specified target fn llvm_filecheck(&self, target: Interned<String>) -> PathBuf { let target_config = self.config.target_config.get(&target); |
