RFR (M) 8201537: Remove is_alive closure from Klass::is_loader_alive()

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Mon Apr 16 15:20:21 UTC 2018



On 4/16/18 10:59 AM, Andrew Dinn wrote:
> Hi Coleen,
>
> On 16/04/18 15:00, coleen.phillimore at oracle.com wrote:
>> Summary: remove is_alive closure from callers of Klass::is_loader_alive
>> so that cleaning metadata doesn't require GC closure.
>>
>> This changes runtime, GC and compiler code.
>>
>> Tested with mach5 tier1-5 with no failures.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8201537.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8201537
> The changes look good to me.
>
> I think you missed copyright updates in
>
>    loaderConstraints.cpp
>    resolutionErrors.cpp
>    genMarkSweep.cpp

Thanks Andrew.

I forgot to mention that I update the copyrights in my commit script so 
that people don't have to see that change.
thanks!
Coleen
>
> regards,
>
>
> Andrew Dinn
> -----------
> Senior Principal Software Engineer
> Red Hat UK Ltd
> Registered in England and Wales under Company Registration No. 03798903
> Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander



More information about the hotspot-dev mailing list