RFR: 8209396: Make PtrQueueSets not statically allocated
Kim Barrett
kim.barrett at oracle.com
Mon Aug 13 19:55:17 UTC 2018
> On Aug 13, 2018, at 4:19 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
> Hi again,
>
> On Mon, 2018-08-13 at 10:00 +0200, Thomas Schatzl wrote:
>> Hi Kim,
>>
>> On Sun, 2018-08-12 at 18:07 -0400, Kim Barrett wrote:
>>> Please review this change to G1BarrierSet to make its two
>>> PtrQueueSet
>>> members (_satb_mark_queue_set and _dirty_card_queue_set) be
>>> ordinary
>>> data members rather than static data members.
>>> [...]
>>>
>>> CR:
>>> https://bugs.openjdk.java.net/browse/JDK-8209396
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~kbarrett/8209396/open.00/
>>>
>>
>> looks good.
>>
>> Please consider moving the return statement at g1BarrierSet.cpp:66 to
>> a new line and add braces.
OK.
>
> This also means that I do not need a re-review either way.
>
> Thanks,
> Thomsa
Thanks.
More information about the hotspot-gc-dev
mailing list