On Tue, 7 Nov 2023 14:59:48 GMT, Roman Kennke <rken...@openjdk.org> wrote:
>> See JBS issue for details.
>>
>> Testing:
>> - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC
>> - [x] tier1 -XX:+UseParallelGC
>> - [ ] tier2 -XX:+UseParallelGC
>> - [ ] hotspot_gc
>
> Roman Kennke has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Review comments; expose self-forwarded objects
src/hotspot/share/gc/parallel/mutableSpace.cpp line 242:
> 240: // so we can encounter objects with "forwarded" markword.
> 241: // They are essentially dead, so skipping them
> 242: if (!obj->is_forwarded() || obj->forwardee() == obj) {
Self-forwarded ones should already be handled by
`PSPromotionManager::restore_preserved_marks`, so the second condition will
always be false.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16494#discussion_r1385076675
