On Sun, 18 May 2025 15:36:03 GMT, Guoxiong Li <[email protected]> wrote:
>> Albert Mingkun Yang has updated the pull request with a new target base due
>> to a merge or a rebase. The pull request now contains five commits:
>>
>> - review
>> - Merge branch 'master' into pgc-size-policy
>> - review
>> - Merge branch 'master' into pgc-size-policy
>> - pgc-size-policy
>
> src/hotspot/share/gc/parallel/psYoungGen.cpp line 268:
>
>> 266: size_t original_committed_size = virtual_space()->committed_size();
>> 267:
>> 268: while (true) {
>
> The `while` statement only runs once. May we find a better way to refactor
> the code?
@lgxbslgx I did some restructuring based on some offline discussion --
hopefully, the new style is more readable.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25000#discussion_r2154053199