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
correctly track freed bytes in jl_genericmemory_to_string #54331
Conversation
Ended up making some more major changes here in reviewing this, as it turns out we appear to have also been undercounting the |
8f992c7
to
eb20906
Compare
eb20906
to
b85c508
Compare
Merge? |
Some tests would be good, but otherwise lgtm. |
Followed up in #54371. (Sorry, the test should have been added in this PR). |
Master version of #54309.
Should fix #54275.