RFR: 8367626: Parallel: Remove ParallelCompactData::summarize_dense_prefix

Guoxiong Li gli at openjdk.org
Wed Sep 17 14:07:56 UTC 2025


On Mon, 15 Sep 2025 08:22:40 GMT, Albert Mingkun Yang <ayang 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

Looks good. One question.

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?

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

Marked as reviewed by gli (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27286#pullrequestreview-3234641841
PR Review Comment: https://git.openjdk.org/jdk/pull/27286#discussion_r2355645890


More information about the hotspot-gc-dev mailing list