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