RFR: 8345314: Add a red–black tree as a utility data structure
Amit Kumar
amitkumar at openjdk.org
Tue Dec 3 09:37:40 UTC 2024
On Mon, 25 Nov 2024 13:11:52 GMT, Casper Norrbin <cnorrbin at openjdk.org> wrote:
> Hi everyone,
>
> This effort began as an exploration of replacing the current NMT treap with a red-black tree. Along the way, I discovered that others were also interested in having a general-purpose tree structure available within HotSpot.
>
> The red-black tree is designed to serve as a drop-in replacement for the existing NMT treap, keeping a nearly identical interface. However, I’ve also added a few additional requested features, such as an iterator.
>
> Testing builds off the treap tests, adding a few extra that inserts/removes and checks that the tree is correct. Testing uses the function `verify_self`, which iterates over the tree and checks that all red-black tree properties hold. Additionally, the tree has been tested in vmatree instead of the treap without any errors.
>
> For those who may want to revisit the fundamentals of red-black trees, [Wikipedia](https://en.wikipedia.org/wiki/Red%E2%80%93black_tree) offers a great summary with tables covering the various balancing cases. Alternatively, your favorite data structure book could provide even more insight.
src/hotspot/share/utilities/rbTree.hpp line 86:
> 84: void color_red() {
> 85: _color = RED;
> 86: }
Suggestion:
void set_color_black() {
_color = BLACK;
}
void set_color_red() {
_color = RED;
}
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22360#discussion_r1867356788
More information about the hotspot-dev
mailing list