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

Adjust to Memory{T} changes #28

Closed
topolarity opened this issue Oct 30, 2023 · 1 comment · Fixed by #32
Closed

Adjust to Memory{T} changes #28

topolarity opened this issue Oct 30, 2023 · 1 comment · Fixed by #32
Milestone

Comments

@topolarity
Copy link
Member

Now that JuliaLang/julia#51319 we need to update AllocCheck to support the changed C API.

There's also a chance that this might have made some array-like optimizations more opaque to us. We should check some basic array allocations and see what the situation is.

@topolarity topolarity added this to the Public MVP milestone Oct 30, 2023
@topolarity
Copy link
Member Author

@gbaraldi Can you start to scope this one out?

Specifically, which runtime functions have changed and how might we chunk up the change to adapt to new codegen, etc?

@topolarity topolarity linked a pull request Nov 12, 2023 that will close this issue
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 a pull request may close this issue.

1 participant