RFR (S): 8231117: Remove G1EvacuationRootClosures::raw_strong_oops()

Thomas Schatzl thomas.schatzl at oracle.com
Wed Sep 18 17:15:39 UTC 2019


Hi Kim,

On 18.09.19 18:46, Kim Barrett wrote:
>> On Sep 18, 2019, at 11:02 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>>
>> Hi all,
>>
>>   can I have reviews for this small cleanup that removes the use of G1EvacuationRootClosures::raw_strong_oops() as its implementation always corresponds to a call to G1EvacuationRootClosures::strong_oops()?
>>
>> Also the comment referenced some non-existent flushing, which even more indicates that this is only some dead code.
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8231117
>> Webrev:
>> http://cr.openjdk.java.net/~tschatzl/8231117/webrev
>> Testing:
>> hs-tier1-5
>>
>> Thanks,
>>   Thomas
> 
> Looks good.
> 

   thanks for your review.

thomas



More information about the hotspot-gc-dev mailing list