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

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Apr 20 00:34:36 UTC 2018


Thank you, Calvin

I assume this is trivial change and one review is enough.

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


More information about the hotspot-runtime-dev mailing list