Skip to content

Commit

Permalink
Add Windows PGO builder (#561)
Browse files Browse the repository at this point in the history
Along with some minor cleanup of dead code/comments.
  • Loading branch information
zware authored Nov 23, 2024
1 parent 9f84063 commit 864027c
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
9 changes: 6 additions & 3 deletions master/custom/builders.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
Windows64Build,
Windows64BigmemBuild,
Windows64NoGilBuild,
Windows64PGOBuild,
Windows64RefleakBuild,
Windows64ReleaseBuild,
MacOSArmWithBrewBuild,
Expand Down Expand Up @@ -219,6 +220,9 @@
("AMD64 CentOS9 FIPS No Builtin Hashes", "cstratak-CentOS9-fips-x86_64", CentOS9NoBuiltinHashesUnixBuild),

("AMD64 Arch Linux Valgrind", "pablogsal-arch-x86_64", ValgrindBuild),

# Windows MSVC
("AMD64 Windows PGO", "bolen-windows10", Windows64PGOBuild),
]


Expand Down Expand Up @@ -324,12 +328,11 @@ def get_builders(settings):


# Match builder name (excluding the branch name) of builders that should only
# run on the main and "custom" branches.
# run on the main and PR branches.
ONLY_MAIN_BRANCH = (
"Alpine Linux",
# Cygwin is not supported on 2.7, 3.6, 3.7
"Cygwin",
"ARM64 Windows",
"Windows PGO",
"AMD64 Arch Linux Perf",
"AMD64 Arch Linux Valgrind",
)
7 changes: 7 additions & 0 deletions master/custom/factories.py
Original file line number Diff line number Diff line change
Expand Up @@ -666,6 +666,13 @@ class Windows64ReleaseBuild(Windows64Build):
factory_tags = ["win64", "nondebug"]


class Windows64PGOBuild(Windows64ReleaseBuild):
buildersuffix = ".pgo"
buildFlags = Windows64Build.buildFlags + ["--pgo"]
testFlags = [*Windows64Build.testFlags, "+d"]
factory_tags = ["win64", "nondebug", "pgo"]


class Windows64NoGilBuild(Windows64Build):
buildersuffix = '.x64.nogil'
buildFlags = Windows64Build.buildFlags + ["--disable-gil"]
Expand Down
6 changes: 0 additions & 6 deletions master/master.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -342,12 +342,6 @@ stable_pull_request_builders = []
all_pull_request_builders = []

for name, worker_name, buildfactory, stability, tier in BUILDERS:
if "Windows XP" in name or "VS9.0" in name:
continue
# bpo-39911: Python 3.9 dropped Windows 7 support
if "Windows7" in name:
continue

buildername = f"{name} PR"
all_pull_request_builders.append(buildername)
if stability == STABLE:
Expand Down

0 comments on commit 864027c

Please sign in to comment.