RFR: 8175318: Performance issue regarding local JNI references

[Daniel D. Daugherty]

On 7/20/17 5:54 PM, Kim Barrett wrote:
>> On Jul 19, 2017, at 11:34 AM, Volker Simonis <volker.simonis at gmail.com> wrote:
>>
>> Looks good!
>>
>> You could use:
>>
>> +        for ( current = current->_next; current != NULL; current =
>> current->_next) {
>>
>> instead of:
>>
>> +        for ( ; current != NULL; current = current->_next) {
>>
>> in the debug code as you already now that the first 'top' is NULL.
>> But that's nit picking. I'll leave it up to you if you want to do that.
>> There's also no need for a new webrev in case you do that change.
>>
>> Thank you and best regards,
> Thanks.
>
> I made this change, since I was already making the other change of eliminating
> the unused clear() function (mentioned in previous email).  I also deleted a bit of
> whitespace that made it harder to search for “_top = 0”.  I think those only show up
> in the patch view.
>
> new webrevs:
> full: http://cr.openjdk.java.net/~kbarrett/8175318/hotspot.01/
> incr: http://cr.openjdk.java.net/~kbarrett/8175318/hotspot.01.inc/

src/share/vm/runtime/jniHandles.cpp
     No comments.

src/share/vm/runtime/jniHandles.hpp
     No comments.

Thumbs up!

Dan