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

[CN-Test-Gen] Rewrite ownership and allocation checking #674

Merged

Conversation

ZippeyKeys12
Copy link
Collaborator

@ZippeyKeys12 ZippeyKeys12 commented Oct 30, 2024

Previously, we had a slightly unsound approach that piggy-backed off the executable spec work. Now there's a fully sound version that provides checkpoints, without sacrificing too much speed.

This means not using the CN-Exec allocator. With the CN allocator's only use during generation being for the values constructed, we can implement a similar checkpoint system in order to no longer leak memory when backtracking.

Now separated from the CN-Exec allocator
@ZippeyKeys12 ZippeyKeys12 self-assigned this Oct 30, 2024
@ZippeyKeys12 ZippeyKeys12 merged commit 4340544 into rems-project:master Oct 30, 2024
2 checks passed
@ZippeyKeys12 ZippeyKeys12 deleted the cn-test-gen-split-ownership branch October 30, 2024 08:54
ZippeyKeys12 added a commit to ZippeyKeys12/cerberus that referenced this pull request Nov 18, 2024
ZippeyKeys12 added a commit that referenced this pull request Nov 19, 2024
Already generalized in the runtime in #674
vzaliva pushed a commit to vzaliva/cerberus that referenced this pull request Dec 4, 2024
…#674)

No longer connected to CN-Exec ownership tracking or the CN-Exec allocator
vzaliva pushed a commit to vzaliva/cerberus that referenced this pull request Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant