RFR: 8344665: Refactor PartialArrayState allocation for reuse [v4]

Kim Barrett kbarrett at openjdk.org
Tue Dec 3 14:43:43 UTC 2024


On Tue, 3 Dec 2024 14:23:14 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

>> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove phase invariant checks
>
> src/hotspot/share/gc/shared/partialArrayState.cpp line 110:
> 
>> 108:     _max_allocators(max_allocators),
>> 109:     _registered_allocators(0),
>> 110:     _released_allocators(0)
> 
> `_released_allocators` is a debug only variable, should fail in release build

Well spotted.  It seems I haven't done a release build since my final touch-up to make it debug-only.
I'll push an update once it's been through our CI.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22287#discussion_r1867847941


More information about the hotspot-gc-dev mailing list