Skip to content

utils: remove option to control the allocator #78998

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

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

compnerd
Copy link
Member

The evolution proposal for enabling the memory allocator for the toolchain on Windows has been accepted. Always use mimalloc for the Windows toolchain and reduce the complexity. The last piece that remains is the ARM64 support that will universally enable the allocator for the toolchain.

@compnerd
Copy link
Member Author

@swift-ci please test

The evolution proposal for enabling the memory allocator for the
toolchain on Windows has been accepted. Always use mimalloc for the
Windows toolchain and reduce the complexity.

Clean up and simplify some of the logic for the MSBuild based build of
mimalloc. Avoid building in the source tree at a cost of complexity.
Reduce the amount of building by selectively building projects instead
of the solution.
@compnerd
Copy link
Member Author

Please test with following PRs:
swiftlang/swift-installer-scripts#371

@swift-ci please build toolchain windows platform

@compnerd
Copy link
Member Author

Please test with following PRs:
swiftlang/swift-installer-scripts#371

@swift-ci please smoke test

@compnerd compnerd merged commit 7c9f47e into swiftlang:main Jan 31, 2025
4 checks passed
@compnerd compnerd deleted the your-allocator branch January 31, 2025 16:49
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