Skip to content

Commit f4d014c

Browse files
committed
Auto merge of #79074 - Mark-Simulacrum:fix-ci-llvm, r=jyn514
Install CI llvm into the library directory In other words, my concern in #78932 (comment) was perfectly justified by something we were already doing. For now just special case CI LLVM, but in the future we may want a more general fix. Fixes #79071. r? `@alexcrichton`
2 parents 603ab5b + 4feaa35 commit f4d014c

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/bootstrap/dist.rs

+4-1
Original file line numberDiff line numberDiff line change
@@ -2358,7 +2358,7 @@ fn maybe_install_llvm(builder: &Builder<'_>, target: TargetSelection, dst_libdir
23582358
}
23592359

23602360
if let Some(config) = builder.config.target_config.get(&target) {
2361-
if config.llvm_config.is_some() {
2361+
if config.llvm_config.is_some() && !builder.config.llvm_from_ci {
23622362
// If the LLVM was externally provided, then we don't currently copy
23632363
// artifacts into the sysroot. This is not necessarily the right
23642364
// choice (in particular, it will require the LLVM dylib to be in
@@ -2369,6 +2369,9 @@ fn maybe_install_llvm(builder: &Builder<'_>, target: TargetSelection, dst_libdir
23692369
// with the wrong files and isn't what distributions want.
23702370
//
23712371
// This behavior may be revisited in the future though.
2372+
//
2373+
// If the LLVM is coming from ourselves (just from CI) though, we
2374+
// still want to install it, as it otherwise won't be available.
23722375
return;
23732376
}
23742377
}

0 commit comments

Comments
 (0)