RFR (M): 8215297: Remove ParallelTaskTerminator

Thomas Schatzl thomas.schatzl at oracle.com
Thu Jan 30 16:40:01 UTC 2020


Hi,

On 30.01.20 16:47, Stefan Johansson wrote:
> Hi Thomas,
> 
> On 2020-01-30 11:45, Thomas Schatzl wrote:
>> Hi Zhengyu,
>>
>> On 29.01.20 14:45, Zhengyu Gu wrote:
>>> Hi Thomas,
>>>
>>> Shared changes look good to me.
>>
>>    thanks for your review; I touched up the files a little in
>>
>> http://cr.openjdk.java.net/~tschatzl/8215297/webrev.0_to_1 (diff)
>> http://cr.openjdk.java.net/~tschatzl/8215297/webrev.1 (full)
> Seems to be some webrev-hickup with webrev.1, but I guess nothing 
> changed in g1ConcurrentMark.cpp since first webrev so looked there instead.
> 
> Looks good, especially after the follow-up patches.

Yes, I'm having issues with webrev lately. I managed to fix the webrev.

Thanks for your review,
   Thomas


More information about the hotspot-gc-dev mailing list