Skip to content

Use Satori GC for x64 and arm64 package #17487

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hez2010
Copy link
Member

@hez2010 hez2010 commented Aug 23, 2025

This can improve the overall performance, memory footprint and responsiveness.
Note: this only affects the package. It won't kick in during the build process.

I'm maintaining the Satori GC build in a separate repository.

Also, bump the Windows SDK version to 26100.

@0x5bfa
Copy link
Member

0x5bfa commented Aug 24, 2025

Will we set latency mode for the compaction?

@hez2010
Copy link
Member Author

hez2010 commented Aug 24, 2025

Will we set latency mode for the compaction?

Maybe not. For the desktop app, we prefer a smaller memory footprint, especially when the latency of interactive mode is already low.

@hez2010
Copy link
Member Author

hez2010 commented Aug 24, 2025

Hmmm based on my observation it seems that it would be better to enable LowLatency mode. Need to verify the memory consumption.

@hez2010
Copy link
Member Author

hez2010 commented Aug 24, 2025

With LowLatency mode the memory grows very aggressively.
I think we need to stick with the default interactive mode.

It's slightly better than WKS GC btw, but not that observable.

@yaira2
Copy link
Member

yaira2 commented Aug 24, 2025

The package size is 1mb smaller.

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.

3 participants