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

[Thomas Schatzl]

> 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

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

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

Changes: https://git.openjdk.org/jdk/pull/25320/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25320&range=01
  Stats: 55 lines in 4 files changed: 2 ins; 22 del; 31 mod
  Patch: https://git.openjdk.org/jdk/pull/25320.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25320/head:pull/25320

PR: https://git.openjdk.org/jdk/pull/25320