Merged
Conversation
…-debug feature more robust (but more memory hungry)
Pull Request Test Coverage Report for Build 21778821045Details
💛 - Coveralls |
599df40 to
a741578
Compare
richardkiss
approved these changes
Feb 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use 32 bits for version in
allocator-debugfeature. This allows 2^32 uses ofrestore_checkpoint()on theAllocatorwhen debugging.Note
Low Risk
Debug-only behavior changes plus fuzz configuration; no changes to non-debug allocator semantics, but could affect assertions/panics when
allocator-debugis enabled.Overview
Improves
allocator-debugsafety checks by separatingAllocatorReferenceinto a 24-bit-ishfingerprintplus a full 32-bitversion, removing the previous 8-bit version packing limit and allowing far morerestore_checkpoint()cycles before debug invalidation tracking breaks.Updates
NodePtrcreation andvalidate_node()to use the new explicitversionfield, and enables theallocator-debugfeature in the fuzz crate to exercise these debug checks during fuzzing.Written by Cursor Bugbot for commit a741578. This will update automatically on new commits. Configure here.