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