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

Pin more references and expose some globally rooted symbols #64

Merged
merged 10 commits into from
Oct 11, 2024

Conversation

qinsoon
Copy link
Member

@qinsoon qinsoon commented Aug 30, 2024

This PR adds a few more pining for types in the native heap, and exposes globally rooted symbols so we can trace them in the MMTk binding. With this PR, we can run moving Immix without transitively pinning global_roots_table. See mmtk/mmtk-julia#170.

Copy link

@udesou udesou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@qinsoon qinsoon changed the title Pin modules Pin modules and expose some globally rooted symbols Sep 2, 2024
@qinsoon qinsoon changed the title Pin modules and expose some globally rooted symbols Pin more references and expose some globally rooted symbols Sep 25, 2024
src/builtins.c Show resolved Hide resolved
src/builtins.c Outdated Show resolved Hide resolved
src/jl_uv.c Outdated Show resolved Hide resolved
@qinsoon qinsoon merged commit 9709741 into mmtk:v1.9.2+RAI Oct 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants