Skip to content

Commit 4af94f8

Browse files
committed
Implement RunMake test suite using the test! macro
1 parent 807f6ff commit 4af94f8

File tree

1 file changed

+4
-34
lines changed
  • src/bootstrap/src/core/build_steps

1 file changed

+4
-34
lines changed

Diff for: src/bootstrap/src/core/build_steps/test.rs

+4-34
Original file line numberDiff line numberDiff line change
@@ -1381,40 +1381,7 @@ test!(Pretty {
13811381
only_hosts: true,
13821382
});
13831383

1384-
/// Special-handling is needed for `run-make`, so don't use `test!` for defining `RunMake`
1385-
/// tests.
1386-
#[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)]
1387-
pub struct RunMake {
1388-
pub compiler: Compiler,
1389-
pub target: TargetSelection,
1390-
}
1391-
1392-
impl Step for RunMake {
1393-
type Output = ();
1394-
const DEFAULT: bool = true;
1395-
const ONLY_HOSTS: bool = false;
1396-
1397-
fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {
1398-
run.suite_path("tests/run-make")
1399-
}
1400-
1401-
fn make_run(run: RunConfig<'_>) {
1402-
let compiler = run.builder.compiler(run.builder.top_stage, run.build_triple());
1403-
run.builder.ensure(RunMakeSupport { compiler, target: run.build_triple() });
1404-
run.builder.ensure(RunMake { compiler, target: run.target });
1405-
}
1406-
1407-
fn run(self, builder: &Builder<'_>) {
1408-
builder.ensure(Compiletest {
1409-
compiler: self.compiler,
1410-
target: self.target,
1411-
mode: "run-make",
1412-
suite: "run-make",
1413-
path: "tests/run-make",
1414-
compare_mode: None,
1415-
});
1416-
}
1417-
}
1384+
test!(RunMake { path: "tests/run-make", mode: "run-make", suite: "run-make", default: true });
14181385

14191386
test!(Assembly { path: "tests/assembly", mode: "assembly", suite: "assembly", default: true });
14201387

@@ -1657,6 +1624,9 @@ NOTE: if you're sure you want to do this, please open an issue as to why. In the
16571624
host: target,
16581625
});
16591626
}
1627+
if suite == "run-make" {
1628+
builder.tool_exe(Tool::RunMakeSupport);
1629+
}
16601630

16611631
// Also provide `rust_test_helpers` for the host.
16621632
builder.ensure(TestHelpers { target: compiler.host });

0 commit comments

Comments
 (0)