RFR: 8252888: Simplify G1MMUTracker class hierarchy [v2]

Albert Mingkun Yang ayang at openjdk.java.net
Tue Sep 22 12:31:54 UTC 2020


On Tue, 22 Sep 2020 07:59:49 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> John Tortugo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Rename MMUTrackerQueue[Element] to MMUTracker
>
> Thank you for the update. If you feel like it, `static consexpr uint QueueLength = 64;` reads better than `enum
> PrivateConstants { ... }`, IMO. Not sure why an `enum` was used in the first place. The rest look fine to me.

> https://bugs.openjdk.java.net/browse/JDK-8153116

Kim, thank you for the info.

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

PR: https://git.openjdk.java.net/jdk/pull/214



More information about the hotspot-gc-dev mailing list