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

Jungwoo Ha jwha at google.com
Fri May 1 13:51:04 UTC 2015


Thank you so much!

On Fri, May 1, 2015 at 6:45 AM, Jon Masamitsu <jon.masamitsu at oracle.com>
wrote:

>
>
> 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
>  Can you take a look?
> --Jungwoo
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20150501/e11a98c8/attachment.htm>


More information about the hotspot-gc-dev mailing list