On Mon, 9 Sep 2024 10:16:24 GMT, Thomas Schatzl <[email protected]> wrote:
>> Roman Kennke has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix bit counts in GCForwarding
>
> src/hotspot/share/gc/shared/gcForwarding.hpp line 41:
>
>> 39: * bits (to indicate 'forwarded' state as usual).
>> 40: */
>> 41: class GCForwarding : public AllStatic {
>
> Since this class is only used for Full GCs, it may be useful to include that
> information, i.e. something like `FullGCForwarding` to avoid confusion why it
> is not used for other GCs too.
> (Unless this has been discussed and even rejected by me before).
I agree. In-fact, that has been my original name. It has been suggested that I
change it to SlidingForwarding when that was the approach that we were going to
take, but with the new implementation, FullGCForwarding makes most sense. I'll
change it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20677#discussion_r1751400378
