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

David Holmes david.holmes at oracle.com
Tue Mar 14 00:44:33 UTC 2017

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!

David

More information about the hotspot-dev mailing list