RFR: 8340103: Add internal set_flag function to VMATree [v11]
Thomas Stuefe
stuefe at openjdk.org
Tue Nov 5 15:19:31 UTC 2024
On Tue, 5 Nov 2024 09:17:10 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
>> test/hotspot/gtest/nmt/test_vmatree.cpp line 296:
>>
>>> 294: VMATree tree;
>>> 295:
>>> 296: VMATree::SummaryDiff result = tree.reserve_mapping(0, 500, rd);
>>
>> How does using '0' gel with your find_enclosing_range, where NULL for start or end is "did not find"?
>
> Sorry, I don't see what the relevance is for this line of code with the rest of the PR :-). Of course, `nullptr` is a pointer type so can't be passed in as a numerical argument, as opposed to `NULL`.
Ah, my mistake. Range.end/start is null, so the TreapNode* itself is null, not the address the TreapNode* represents.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20994#discussion_r1829539869
More information about the hotspot-runtime-dev
mailing list