diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2024-06-14 12:23:38 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-14 12:23:38 +0200 |
| commit | b7dcdf684c93a9691bc6ce1a7f5a71b56cd04374 (patch) | |
| tree | ceb14a6c3e7fc528e008a24a947c83119c8fb74c | |
| parent | b55cabe636d8cde27cff9d65e63ab320d9364dc3 (diff) | |
| parent | 58e3ac0877bc5daf53e42383fc51b8c8e2d2a7bb (diff) | |
| download | rust-b7dcdf684c93a9691bc6ce1a7f5a71b56cd04374.tar.gz rust-b7dcdf684c93a9691bc6ce1a7f5a71b56cd04374.zip | |
Rollup merge of #126399 - onur-ozkan:126156, r=albertlarsan68
extend the check for LLVM build We don't build LLVM when using the precompiled version from the CI builder. Closes #126156
| -rw-r--r-- | src/bootstrap/src/core/sanity.rs | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/src/bootstrap/src/core/sanity.rs b/src/bootstrap/src/core/sanity.rs index b5f17b9f54e..da80f10b7dd 100644 --- a/src/bootstrap/src/core/sanity.rs +++ b/src/bootstrap/src/core/sanity.rs @@ -137,19 +137,20 @@ pub fn check(build: &mut Build) { } // We need cmake, but only if we're actually building LLVM or sanitizers. - let building_llvm = build - .hosts - .iter() - .map(|host| { - build.config.llvm_enabled(*host) - && build - .config - .target_config - .get(host) - .map(|config| config.llvm_config.is_none()) - .unwrap_or(true) - }) - .any(|build_llvm_ourselves| build_llvm_ourselves); + let building_llvm = !build.config.llvm_from_ci + && build + .hosts + .iter() + .map(|host| { + build.config.llvm_enabled(*host) + && build + .config + .target_config + .get(host) + .map(|config| config.llvm_config.is_none()) + .unwrap_or(true) + }) + .any(|build_llvm_ourselves| build_llvm_ourselves); let need_cmake = building_llvm || build.config.any_sanitizers_to_build(); if need_cmake && cmd_finder.maybe_have("cmake").is_none() { |
