-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Implement the mimalloc option to speed up the compiler on Windows #76563
base: main
Are you sure you want to change the base?
Conversation
@swift-ci please test |
utils/build.ps1
Outdated
@@ -102,6 +102,9 @@ in batch file format instead of executing them. | |||
.PARAMETER HostArchName | |||
The architecture where the toolchain will execute. | |||
.PARAMETER EnableMimalloc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should generalise this to something like -allocator ...
to experiment with other allocators.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
[hashtable]$Arch | ||
) | ||
|
||
if ($Arch -eq $ArchX64) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we abort if a different architecture is passed? Or is this being allowed for some other reason? As this currently stands, it feels like we could silently fail to enable something :(.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think if we do that, the cross-compiling CI would likely fail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, why would it fail? The check should be for what host we are compiling for, not what the build is right?
This is disabled by default.
33168a0
to
cef5a5c
Compare
@swift-ci please test |
This is disabled by default.