RFR: Cleanups, TODOs, asserts (part 2) [v2]

Aleksey Shipilev shade at openjdk.org
Thu Apr 13 08:29:23 UTC 2023


On Wed, 12 Apr 2023 17:57:22 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - More review comments
>>  - Move and rename remset verifications
>>  - Review Ramki
>
> src/hotspot/share/gc/shenandoah/shenandoahMark.inline.hpp line 291:
> 
>> 289:       mark_ref(q, mark_context, weak, obj);
>> 290:       shenandoah_assert_marked(p, obj);
>> 291:       // TODO: This is v-call on very hot path, can we sense the same from GENERATION?
> 
> Maybe if there was a ShenandoahGenerationType that distinguished single-gen from the generational modes..  Agree we would like to template-resolve this path.

Yes, let me look at that separately.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/252#discussion_r1165183449


More information about the shenandoah-dev mailing list