Skip to content

Commit d69c33a

Browse files
committed
Auto merge of #103569 - RalfJung:miri-test-macos, r=Mark-Simulacrum
fix and (re-)enable Miri cross-target checks on macOS and Windows Fixes #103519 r? `@Mark-Simulacrum`
2 parents 68f7729 + 397e5bb commit d69c33a

File tree

2 files changed

+20
-7
lines changed

2 files changed

+20
-7
lines changed

src/bootstrap/sanity.rs

+16
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,14 @@ than building it.
155155
continue;
156156
}
157157

158+
// Some environments don't want or need these tools, such as when testing Miri.
159+
// FIXME: it would be better to refactor this code to split necessary setup from pure sanity
160+
// checks, and have a regular flag for skipping the latter. Also see
161+
// <https://github.com/rust-lang/rust/pull/103569#discussion_r1008741742>.
162+
if env::var_os("BOOTSTRAP_SKIP_TARGET_SANITY").is_some() {
163+
continue;
164+
}
165+
158166
if !build.config.dry_run {
159167
cmd_finder.must_have(build.cc(*target));
160168
if let Some(ar) = build.ar(*target) {
@@ -212,6 +220,14 @@ than building it.
212220
}
213221
}
214222

223+
// Some environments don't want or need these tools, such as when testing Miri.
224+
// FIXME: it would be better to refactor this code to split necessary setup from pure sanity
225+
// checks, and have a regular flag for skipping the latter. Also see
226+
// <https://github.com/rust-lang/rust/pull/103569#discussion_r1008741742>.
227+
if env::var_os("BOOTSTRAP_SKIP_TARGET_SANITY").is_some() {
228+
continue;
229+
}
230+
215231
if need_cmake && target.contains("msvc") {
216232
// There are three builds of cmake on windows: MSVC, MinGW, and
217233
// Cygwin. The Cygwin build does not have generators for Visual

src/ci/docker/host-x86_64/x86_64-gnu-tools/checktools.sh

+4-7
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,7 @@ python3 "$X_PY" test --stage 2 src/tools/rustfmt
2727
python3 "$X_PY" test --stage 2 src/tools/miri
2828
# We natively run this script on x86_64-unknown-linux-gnu and x86_64-pc-windows-msvc.
2929
# Also cover some other targets (on both of these hosts) via cross-testing.
30-
#
31-
# Currently disabled -- we end up pulling in a cross-compile of LLVM (maybe
32-
# just overly eager sanity checks), but in any case this won't work when
33-
# building LLVM as of this comment.
34-
#python3 "$X_PY" test --stage 2 src/tools/miri --target i686-pc-windows-msvc
35-
#FIXME(https://github.com/rust-lang/rust/issues/103519): macOS testing is currently disabled
36-
# python3 "$X_PY" test --stage 2 src/tools/miri --target aarch64-apple-darwin
30+
export BOOTSTRAP_SKIP_TARGET_SANITY=1 # we don't need `cc` for these targets
31+
python3 "$X_PY" test --stage 2 src/tools/miri --target i686-pc-windows-msvc
32+
python3 "$X_PY" test --stage 2 src/tools/miri --target aarch64-apple-darwin
33+
unset BOOTSTRAP_SKIP_TARGET_SANITY

0 commit comments

Comments
 (0)