-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
ensure objects beyond bump allocated region are inserted into the object pool freelist #50357
Conversation
1ad31ca
to
53fe3e3
Compare
Please link the PR & comments in question directly, to make it easier for others to follow the train of thought. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Any idea if this fixes the recent memory leak claimed?
Test with
Seems unchanged, problems still exist. |
I believe the possible memory leak @vtjnash is referring to is not related to #50345, but discussed in this slack thread https://julialang.slack.com/archives/C0309QBEZNH/p1687914365229929. |
80c95d4
to
673f8f6
Compare
Any chance we can still backport this to alpha2 @KristofferC? (forgot to put the backport 1.10 label). |
NVM, this PR already made it to alpha1... |
x-ref #50137