Skip to content
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

Prevent collection of uncollectable allocations #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jacob-hughes
Copy link

This modifies the collector so that allocations with the uncollectable flag set are never considered garbage during sweeping.

This allows for switching between collectable and uncollectable allocations during runtime.

This modifies the collector so that allocations with the uncollectable
flag set are never considered garbage during sweeping.

This allows for switching between collectable and uncollectable
allocations during runtime.
@ltratt ltratt added this pull request to the merge queue Sep 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 16, 2024
@jacob-hughes
Copy link
Author

Looks like I was running these tests locally without one of the -D flags, let me see what's going on.

@ltratt
Copy link
Member

ltratt commented Sep 17, 2024

Any luck?

@jacob-hughes
Copy link
Author

I've got a new PR that I'm going to raise that addresses this and includes the logic around moving between allocation types during finaliser elision. Without this, some invariants don't hold and the assertions don't make sense.

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.

2 participants