Auto merge of #85959 - RalfJung:miri-stage-0, r=Mark-Simulacrum

fix testing Miri with --stage 0

Fixes https://github.com/rust-lang/rust/issues/78778 for Miri.

The issue remains open for error_index_generator, which has its own dylib logic:
903e369c83/src/bootstrap/tool.rs (L396-L400)
`@jyn514` I could just copy the logic from `add_rustc_lib_path`, but that does not seem great. Any other suggestions?

Also I wonder if maybe `prepare_tool_cargo` should already call `add_rustc_lib_path`.
This commit is contained in:
bors 2021-06-04 07:10:55 +00:00
commit c4c2ab57a4
3 changed files with 12 additions and 3 deletions

View File

@ -709,7 +709,15 @@ impl<'a> Builder<'a> {
return; return;
} }
add_dylib_path(vec![self.rustc_libdir(compiler)], cmd); let mut dylib_dirs = vec![self.rustc_libdir(compiler)];
// Ensure that the downloaded LLVM libraries can be found.
if self.config.llvm_from_ci {
let ci_llvm_lib = self.out.join(&*compiler.host.triple).join("ci-llvm").join("lib");
dylib_dirs.push(ci_llvm_lib);
}
add_dylib_path(dylib_dirs, cmd);
} }
/// Gets a path to the compiler specified. /// Gets a path to the compiler specified.

View File

@ -449,6 +449,7 @@ impl Step for Miri {
SourceType::Submodule, SourceType::Submodule,
&[], &[],
); );
cargo.add_rustc_lib_path(builder, compiler);
cargo.arg("--").arg("miri").arg("setup"); cargo.arg("--").arg("miri").arg("setup");
// Tell `cargo miri setup` where to find the sources. // Tell `cargo miri setup` where to find the sources.
@ -500,6 +501,7 @@ impl Step for Miri {
SourceType::Submodule, SourceType::Submodule,
&[], &[],
); );
cargo.add_rustc_lib_path(builder, compiler);
// miri tests need to know about the stage sysroot // miri tests need to know about the stage sysroot
cargo.env("MIRI_SYSROOT", miri_sysroot); cargo.env("MIRI_SYSROOT", miri_sysroot);
@ -508,8 +510,6 @@ impl Step for Miri {
cargo.arg("--").args(builder.config.cmd.test_args()); cargo.arg("--").args(builder.config.cmd.test_args());
cargo.add_rustc_lib_path(builder, compiler);
let mut cargo = Command::from(cargo); let mut cargo = Command::from(cargo);
if !try_run(builder, &mut cargo) { if !try_run(builder, &mut cargo) {
return; return;

View File

@ -45,6 +45,7 @@ pub fn libdir(target: TargetSelection) -> &'static str {
} }
/// Adds a list of lookup paths to `cmd`'s dynamic library lookup path. /// Adds a list of lookup paths to `cmd`'s dynamic library lookup path.
/// If The dylib_path_par is already set for this cmd, the old value will be overwritten!
pub fn add_dylib_path(path: Vec<PathBuf>, cmd: &mut Command) { pub fn add_dylib_path(path: Vec<PathBuf>, cmd: &mut Command) {
let mut list = dylib_path(); let mut list = dylib_path();
for path in path { for path in path {