RFR: Use CardTable::card_size_in_words rather than hard coded constant [v3]
William Kemper
wkemper at openjdk.org
Mon Dec 19 21:13:18 UTC 2022
On Mon, 19 Dec 2022 20:47:00 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:
>> William Kemper has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>>
>> Do not assume cards will have 64 words
>
> src/hotspot/share/gc/shenandoah/shenandoahScanRemembered.hpp line 1033:
>
>> 1031: // ShenandoahCardCluster<ShenandoahDirectCardMarkRememberedSet>::CardsPerCluster;
>> 1032: // We can't perform this computation here, because of encapsulation and initialization constraints. We paste
>> 1033: // the magic number here, and assert that this number matches the intended computation in constructor.
>
> Is the portion of the comment beginning "We paste the maginc number ... etc." not obsolete now, and should there be deleted?
Yes - I'll clean that up.
-------------
PR: https://git.openjdk.org/shenandoah/pull/186
More information about the shenandoah-dev
mailing list