Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[build] Disable WebGPU for Windows non-release builds #2988

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fhanau
Copy link
Collaborator

@fhanau fhanau commented Oct 23, 2024

The Windows build is currently much slower than the macOS/Linux builds on CI, especially on mostly cached builds.
Based on profiling on #2981, this is caused by the repo mapping stage for dawn taking several minutes of Windows. Disable the experimental WebGPU feature on Windows to better meet compile time constraints. Release builds remain unaffected.

@fhanau fhanau force-pushed the felix/102324-windows-no-webgpu branch from 997bf14 to 7ec4270 Compare October 23, 2024 17:31
@fhanau
Copy link
Collaborator Author

fhanau commented Oct 23, 2024

This does not seem to be as impactful as expected – will take some more work

@fhanau fhanau force-pushed the felix/102324-windows-no-webgpu branch 4 times, most recently from 9683707 to feab6aa Compare October 26, 2024 18:02
@fhanau fhanau force-pushed the felix/102324-windows-no-webgpu branch 5 times, most recently from eae12ea to 13c0d0b Compare November 8, 2024 04:14
The Windows build is currently much slower than the macOS/Linux builds on CI,
especially on mostly cached builds.
Based on profiling on #2981, this is caused by the repo mapping stage for dawn
taking several minutes of Windows. Disable the experimental WebGPU feature on
Windows to better meet compile time constraints. Release builds remain
unaffected.
Fetching WPT also contributes here. Only fetch/use WPT on Unix and only
configure targets based on globs as we start using them, Bazel configures
90,000 targets for it otherwise.
@fhanau fhanau force-pushed the felix/102324-windows-no-webgpu branch from 13c0d0b to 0ed26d3 Compare November 8, 2024 04:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant