RFR: 8367759: G1: Move G1UpdateRegionLivenessAndSelectForRebuildTask into its own file

Thomas Schatzl tschatzl at openjdk.org
Fri Sep 19 11:47:51 UTC 2025


On Fri, 19 Sep 2025 07:39:06 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this change to move `G1UpdateRegionLivenessAndSelectForRebuildTask` into a separate file. I would like to merge this class and G1BuildCandidateRegionsTask in the near future because the latter mostly repeats the work of the former anyway, but together (with helper classes) they are quite large (if not large already by themselves). This should help make `g1ConcurrentMark.cpp` a little more manageable.
>> 
>> There is no change in the code, just moving and fixing includes/adding forwards.
>> 
>> Testing: local compilation, gha
>> 
>> Thanks,
>>   Thomas
>
> Marked as reviewed by ayang (Reviewer).

Thanks @albertnetymk @walulyai for your reviews,
  Thomas

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

PR Comment: https://git.openjdk.org/jdk/pull/27314#issuecomment-3311885040


More information about the hotspot-gc-dev mailing list