[11] RFR(XS) 8178825: unused variable threadObj in jvmci_counters_include

[Calvin Cheung]

On 4/19/18, 5:34 PM, Vladimir Kozlov wrote:
> Thank you, Calvin
>
> I assume this is trivial change and one review is enough.
Yes, I think it is a trivial change.

thanks,
Calvin
>
> Vladimir
>
> On 4/19/18 5:07 PM, Calvin Cheung wrote:
>> Looks good.
>>
>> thanks,
>> Calvin
>>
>> On 4/19/18, 4:08 PM, Vladimir Kozlov wrote:
>>> I found this old RFE (1 year old) which has simple suggested fix we 
>>> forgot to push.
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8178825
>>>
>>> diff -r e242740a92b8 src/hotspot/share/runtime/thread.cpp
>>> --- a/src/hotspot/share/runtime/thread.cpp
>>> +++ b/src/hotspot/share/runtime/thread.cpp
>>> @@ -1495,7 +1495,6 @@
>>>  jlong* JavaThread::_jvmci_old_thread_counters;
>>>
>>>  bool jvmci_counters_include(JavaThread* thread) {
>>> -  oop threadObj = thread->threadObj();
>>>    return !JVMCICountersExcludeCompiler || 
>>> !thread->is_Compiler_thread();
>>>  }
>>>
>>> Thanks,
>>> Vladimir