RFR (S): 8079091: Remove dictionary NULL check on common path of BlockFreeList methods

Jon Masamitsu jon.masamitsu at oracle.com
Fri May 1 13:45:02 UTC 2015



On 5/1/2015 6:38 AM, Jon Masamitsu wrote:
> Jungwoo,
>
> Looks good.
>
> Reviewed.
>
> I'll sponsor it.
>
> Stefan had some questions about the performance numbers but
> since it is a nice cleanup on it's own,  I'm going to go ahead and
> push it.

Having second thoughts on that.  Stefan did make a code change suggestion
so should get a chance to look at that, but I'll handle it from here on.

Jon

>
> Jon
>
>
> On 4/30/2015 6:04 PM, Jungwoo Ha wrote:
>>
>>     >
>>     > All,
>>     >
>>     > Let's go back to Jungwoo's 00 version.  It's a clean, limited
>>     change.
>>     >
>>     > Objections?
>>
>>     That’s mostly ok with me.
>>
>>     I think the constructor and destructor suggestions that Stefan
>>     and I made should be added.
>>
>>
>> Done.
>> http://cr.openjdk.java.net/~jwha/8079091/webrev.03 
>> <http://cr.openjdk.java.net/%7Ejwha/8079091/webrev.03>
>> Can you take a look?
>> --Jungwoo
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20150501/3fb26458/attachment.html>


More information about the hotspot-gc-dev mailing list