RFR: 8367347: Serial: Refactor CLDScanClosure [v3]

Albert Mingkun Yang ayang at openjdk.org
Wed Sep 10 19:05:30 UTC 2025


On Wed, 10 Sep 2025 18:59:18 GMT, Francesco Andreuzzi <fandreuzzi at openjdk.org> wrote:

>> Refactor `CLDScanClosure` so that all CLD logic is encapsulated in the CLD closure, without polluting the oop closure.
>> 
>> Test: tier1-2, running tier3
>
> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   trigger

Marked as reviewed by ayang (Reviewer).

src/hotspot/share/gc/serial/defNewGeneration.cpp line 100:

> 98: 
> 99:     template <typename T>
> 100:     void do_oop_work(T* p) {

Pre-existing: since this method has only a single caller, I suggest inlining it.

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

PR Review: https://git.openjdk.org/jdk/pull/27199#pullrequestreview-3207464984
PR Review Comment: https://git.openjdk.org/jdk/pull/27199#discussion_r2337652575


More information about the hotspot-gc-dev mailing list