RFR (XL) 8155672: Remove instanceKlassHandles and KlassHandles
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 ...
>
> <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!
Right.
Good catch!
Thanks,
Serguei
>
> David
> -----
More information about the hotspot-dev
mailing list