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