RFR: 8325808: GenShen: Move generational mode code out of shFullGC.cpp
Y. Srinivas Ramakrishna
ysr at openjdk.org
Mon Feb 19 18:33:10 UTC 2024
On Mon, 19 Feb 2024 15:38:39 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:
>> This change reduces the differences from the upstream branch by moving large chunks of generational mode code into separate files.
>
> src/hotspot/share/gc/shenandoah/shenandoahScanRemembered.cpp line 35:
>
>> 33:
>> 34:
>> 35: class ShenandoahSetRememberedCardsToDirtyClosure : public BasicOopIterateClosure {
>
> Nit: I realize this is the existing name of the closure, and you just moved it here, but consider the following renaming suggestion: `ShenandoahDirtyRememberedSetClosure`
Nit: may be place a block comment for the closure:
// A closure that takes an oop in the old generation and, if it's
// pointing into the young generation, dirties the corresponding
// remembered set entry.
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/398#discussion_r1494755034
More information about the shenandoah-dev
mailing list