RFR: 8328364: Remove redundant fields in 'BOTConstants'

Thomas Schatzl tschatzl at openjdk.org
Tue Mar 19 10:25:20 UTC 2024


On Tue, 19 Mar 2024 05:51:51 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> The class `BOTConstants` has several redundant fields. This patch removes the redundant fields and adds a new field `CardTable::_card_shift_in_words` like `BOTConstants::_log_card_size_in_words`.
> 
> Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxiong

Fwiw, the BOT does not need to align to cards, but there is not a lot of use for having different granularity....

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

Marked as reviewed by tschatzl (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18371#pullrequestreview-1945759911


More information about the hotspot-gc-dev mailing list