Skip to content

Commit 39103ce

Browse files
committed
Fix run-make tests running when LLVM is disabled
1 parent ffe5288 commit 39103ce

File tree

1 file changed

+15
-11
lines changed

1 file changed

+15
-11
lines changed

src/bootstrap/test.rs

+15-11
Original file line numberDiff line numberDiff line change
@@ -1203,17 +1203,6 @@ note: if you're sure you want to do this, please open an issue as to why. In the
12031203
// Only pass correct values for these flags for the `run-make` suite as it
12041204
// requires that a C++ compiler was configured which isn't always the case.
12051205
if !builder.config.dry_run && matches!(suite, "run-make" | "run-make-fulldeps") {
1206-
cmd.arg("--cc")
1207-
.arg(builder.cc(target))
1208-
.arg("--cxx")
1209-
.arg(builder.cxx(target).unwrap())
1210-
.arg("--cflags")
1211-
.arg(builder.cflags(target, GitRepo::Rustc).join(" "));
1212-
copts_passed = true;
1213-
if let Some(ar) = builder.ar(target) {
1214-
cmd.arg("--ar").arg(ar);
1215-
}
1216-
12171206
// The llvm/bin directory contains many useful cross-platform
12181207
// tools. Pass the path to run-make tests so they can use them.
12191208
let llvm_bin_path = llvm_config
@@ -1239,6 +1228,21 @@ note: if you're sure you want to do this, please open an issue as to why. In the
12391228
}
12401229
}
12411230

1231+
// Only pass correct values for these flags for the `run-make` suite as it
1232+
// requires that a C++ compiler was configured which isn't always the case.
1233+
if !builder.config.dry_run && matches!(suite, "run-make" | "run-make-fulldeps") {
1234+
cmd.arg("--cc")
1235+
.arg(builder.cc(target))
1236+
.arg("--cxx")
1237+
.arg(builder.cxx(target).unwrap())
1238+
.arg("--cflags")
1239+
.arg(builder.cflags(target, GitRepo::Rustc).join(" "));
1240+
copts_passed = true;
1241+
if let Some(ar) = builder.ar(target) {
1242+
cmd.arg("--ar").arg(ar);
1243+
}
1244+
}
1245+
12421246
if !llvm_components_passed {
12431247
cmd.arg("--llvm-components").arg("");
12441248
}

0 commit comments

Comments
 (0)