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
fix: use win_clang_x64 binary for x86 extract symbols #35078
Conversation
Merging given the release builds are green 👏 |
Release Notes Persisted
|
I was unable to backport this PR to "18-x-y" cleanly; |
I have automatically backported this PR to "19-x-y", please check out #35090 |
I have automatically backported this PR to "20-x-y", please check out #35091 |
@VerteDinde has manually backported this PR to "18-x-y", please check out #35096 |
fix: use win_clang_x64 for x86 extract symbols
fix: use win_clang_x64 for x86 extract symbols
Description of Change
Addresses #34413
We're currently experiencing build failures on main, 20-x-y and 18-x-y because the ia32 symbol generation is running out of virtual memory space. This PR uses the win_clang_x64 binary to extract/generate electron symbols for Windows x86.
Note: This issue isn't currently affecting 19-x-y, but it was introduced into 18-x-y by a backported Chromium roll. Adding this change to 19-x-y will protect us against a potential future breakage in that branch.
Checklist
npm test
passesRelease Notes
Notes: Fixed symbol generation on 32-bit Windows release builds.