RFR: Add generations to freeset [v5]
Kelvin Nilsen
kdnilsen at openjdk.org
Tue Apr 18 21:34:49 UTC 2023
On Thu, 13 Apr 2023 11:21:09 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove debug instrumentation
>
> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 652:
>
>> 650: }
>> 651:
>> 652: bool ShenandoahFreeSet::expand_collector_bounds_maybe(size_t idx) {
>
> For this and the following method, the return value is not used, can be dropped.
done
> src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp line 1065:
>
>> 1063: buffer[idx] = 'C';
>> 1064: }
>> 1065: else if (r->is_humongous()) {
>
> Suggestion:
>
> } else if (r->is_humongous()) {
fixed.
> src/hotspot/share/gc/shenandoah/shenandoahHeap.hpp line 323:
>
>> 321:
>> 322: // The evacuation reserves for old-gen and young-gen are available
>> 323: VALID_EVACUATION_RESERVE_QUANTITIES_BITPOS = 6
>
> I think we only do these when we want to access the flags from generated code. If we only do the accesses from the GC code, this could be just a plan `bool` field.
Thanks for explanation. I'm changing to use a boolean.
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170588359
PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170589727
PR Review Comment: https://git.openjdk.org/shenandoah/pull/250#discussion_r1170590252
More information about the shenandoah-dev
mailing list