Skip to content

Commit

Permalink
Auto merge of #133632 - MarcoIeni:split-x86_64-msvc, r=<try>
Browse files Browse the repository at this point in the history
CI: split x86_64-msvc job
  • Loading branch information
bors committed Nov 29, 2024
2 parents 5bbbc09 + 02bf397 commit 42d03ef
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 10 deletions.
6 changes: 4 additions & 2 deletions src/bootstrap/mk/Makefile.in
Original file line number Diff line number Diff line change
Expand Up @@ -99,11 +99,13 @@ prepare:

## MSVC native builders

# Set of tests that should represent half of the time of the test suite.
# Used to split tests across multiple CI runners.
# this intentionally doesn't use `$(BOOTSTRAP)` so we can test the shebang on Windows
ci-msvc-py:
$(Q)$(CFG_SRC_DIR)/x.py test --stage 2 tidy
$(Q)$(CFG_SRC_DIR)/x.py test --stage 2 --skip=compiler --skip=src
ci-msvc-ps1:
$(Q)$(CFG_SRC_DIR)/x.ps1 test --stage 2 --skip tidy
$(Q)$(CFG_SRC_DIR)/x.ps1 test --stage 2 --skip=tests --skip=coverage-map --skip=coverage-run --skip=library --skip=tidyselftest
ci-msvc: ci-msvc-py ci-msvc-ps1

## MingW native builders
Expand Down
10 changes: 5 additions & 5 deletions src/bootstrap/src/core/builder/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -258,11 +258,11 @@ impl PathSet {

// internal use only
fn check(p: &TaskPath, needle: &Path, module: Kind) -> bool {
if let Some(p_kind) = &p.kind {
p.path.ends_with(needle) && *p_kind == module
} else {
p.path.ends_with(needle)
}
let check_path = || {
// This order is important for retro-compatibility, as `starts_with` was introduced later.
p.path.ends_with(needle) || p.path.starts_with(needle)
};
if let Some(p_kind) = &p.kind { check_path() && *p_kind == module } else { check_path() }
}

/// Return all `TaskPath`s in `Self` that contain any of the `needles`, removing the
Expand Down
13 changes: 10 additions & 3 deletions src/ci/github-actions/jobs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -372,11 +372,18 @@ auto:
# Windows Builders #
######################

- image: x86_64-msvc
# x86_64-msvc is split into two jobs to run tests in parallel.
- image: x86_64-msvc-1
env:
RUST_CONFIGURE_ARGS: --build=x86_64-pc-windows-msvc --enable-profiler
SCRIPT: make ci-msvc
<<: *job-windows-8c
SCRIPT: make ci-msvc-py
<<: *job-windows

- image: x86_64-msvc-2
env:
RUST_CONFIGURE_ARGS: --build=x86_64-pc-windows-msvc --enable-profiler
SCRIPT: make ci-msvc-ps1
<<: *job-windows

- image: i686-msvc
env:
Expand Down

0 comments on commit 42d03ef

Please sign in to comment.