RFR: 8357306: G1: Remove _gc_succeeded from VM_G1CollectForAllocation because it is always true [v2]

Thomas Schatzl tschatzl at openjdk.org
Fri May 23 12:03:57 UTC 2025


On Thu, 22 May 2025 06:58:52 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - Merge branch 'master' into 8357306-remove-gc-succeeded-in-g1-vm-operations
>>  - * remove comment
>>  - * fix try-concurrent-...
>>  - * some minor refactoring
>>  - 8357306
>>    
>>    Hi all,
>>    
>>      please review this refactoring of G1 VM GC operations to remove the _gc_succeeded members because they are not necessary any more - GC operations themselves (i.e. the doit() part) always succeed.
>>    
>>    Testing: tier1-3, gha
>>    
>>    Thanks,
>>      Thomas
>
> Marked as reviewed by ayang (Reviewer).

Thanks @albertnetymk @kstefanj  for your reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/25320#issuecomment-2904196630


More information about the hotspot-gc-dev mailing list