Adding stock gc heap resizing heuristics to MMTk #57147
Open
+50
−16
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
udesou
force-pushed
the
adding-stock-gc-trigger
branch
2 times, most recently
from
1dd799e to
a9f4be1
Compare
|
I suspect you did a bunch of changes in the MMTk binding or MMTk core to incorporate the heuristic changes. If you did, could you please also link the relevant PRs here? |
|
Linked in the description. I might start pinging you in those PRs as well from now on if that’s okay. That way you’ll get more familiar with the binding code and it will be super helpful to have another set of eyes reviewing them (for the |
udesou
force-pushed
the
adding-stock-gc-trigger
branch
from
a9f4be1 to
0c816c5
Compare
udesou
force-pushed
the
adding-stock-gc-trigger
branch
from
0c816c5 to
2194778
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR sets MMTk to use the stock GC heuristics for heap resizing by default. It also updates the
mmtk-juliaversion to incorporate the changes that support the new gc triggering heuristics (see mmtk/mmtk-julia#222).