RFR: 8345732: Provide helpers for using PartialArrayState [v2]

Kim Barrett kbarrett at openjdk.org
Fri Dec 13 19:37:39 UTC 2024


On Fri, 13 Dec 2024 09:31:10 GMT, Ivan Walulya <iwalulya at openjdk.org> wrote:

>> Kim Barrett has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Merge branch 'master' into pa-splitter
>>  - parallel uses PartialArraySplitter
>>  - g1 uses PartialArraySplitter
>>  - add PartialArraySplitter
>>  - add PartialArrayTaskStats
>
> src/hotspot/share/gc/shared/partialArraySplitter.inline.hpp line 62:
> 
>> 60: template<typename Queue>
>> 61: PartialArraySplitter::Step
>> 62: PartialArraySplitter::step(PartialArrayState* state, Queue* queue, bool stolen) {
> 
> Probably easier to read if we rename to `claim`,  step is used as noun in many other places

I like the suggested name change.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22622#discussion_r1884408963


More information about the hotspot-gc-dev mailing list