RFR (XL) 8155672: Remove instanceKlassHandles and KlassHandles

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


Something caught my eye ...

<trimming>

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 // _current_thread is for creating a Klass* with a faster version
>> constructor
>>  125   static Thread*                     _current_thread;
>>
>> Does the _current_thread 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