RFR: 8367626: Parallel: Remove ParallelCompactData::summarize_dense_prefix

Wed Sep 17 14:42:26 UTC 2025

On Wed, 17 Sep 2025 14:04:09 GMT, Guoxiong Li <gli at openjdk.org> wrote:

>> Remove the unnecessary summary operation on dense-prefix, because the summary operation is only needed for compaction, but dense-prefix doesn't participate in compaction.
>> 
>> Test: tier1-5
>
> src/hotspot/share/gc/parallel/psParallelCompact.cpp line 1552:
> 
>> 1550:     HeapWord* dense_prefix_addr = dense_prefix(SpaceId(id));
>> 1551:     HeapWord* top = sp->top();
>> 1552:     HeapWord* cur_addr = dense_prefix_addr;
> 
> These changes just refactor the code which is not related to the title `Remove ParallelCompactData::summarize_dense_prefix`. Is my thought right?

Yes; thought they are "trivial" enough, so included them here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27286#discussion_r2355763115