RFR (XL) 8155672: Remove instanceKlassHandles and KlassHandles (original) (raw)

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Tue Mar 14 00:50:45 UTC 2017


On 3/13/17 17:44, David Holmes wrote:

Something caught my eye ...

On 14/03/2017 10:23 AM, coleen.phillimore at oracle.com wrote: On 3/13/17 8:10 PM, serguei.spitsyn at oracle.com wrote:

http://cr.openjdk.java.net/~coleenp/8155672.03/webrev/src/share/vm/services/classLoadingService.hpp.frames.html

124 // currentthread is for creating a Klass* with a faster version constructor 125 static Thread* currentthread; Does the currentthread become redundant now? Yes, it does! Good find. I removed this and am rerunning monitoring tests. Current thread stored in a static! That was a bug!

Right. Good catch!

Thanks, Serguei

David -----



More information about the hotspot-dev mailing list