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

Andrew Dinn adinn at redhat.com
Mon Apr 16 14:59:23 UTC 2018


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


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