RFR: 8340103: Add internal set_flag function to VMATree [v17]
Johan Sjölen
jsjolen at openjdk.org
Thu Nov 7 10:17:20 UTC 2024
On Wed, 6 Nov 2024 10:29:07 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> Johan Sjölen has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Use size type and size argument name uniformly
>> - Avoid style discussion by adopting an already accepted style :D
>
> Tests: very nice. Comments inline.
@tstuefe , alright I've cleaned up a bit and fixed the tests. For address holes, the solution is to skip released ranges. I think I want another test for that case, so I'll make sure to get that up. Other than that, the code is pretty good. I guess I could write a small explanation of the algorithm used in `set_tag`, if you find it unintuitive?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20994#issuecomment-2461830167
More information about the hotspot-runtime-dev
mailing list