RFR: 8267189: Remove duplicated unregistered classes from dynamic archive [v2]

Ioi Lam iklam at openjdk.java.net
Thu Jun 17 20:55:55 UTC 2021


On Thu, 17 Jun 2021 18:25:43 GMT, Yumin Qi <minqi at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   comments from @calvinccheung and @yminqi
>
> src/hotspot/share/classfile/systemDictionaryShared.cpp line 1511:
> 
>> 1509: class UnregisteredClassesDuplicationChecker : StackObj {
>> 1510:   GrowableArray<InstanceKlass*> _list;
>> 1511:   Thread* _thread;
> 
> seems _thread only used at line 1542, could remove the field and use Thread::current() in the site.

_thread is used in the loop that iterates over all the classes in the _dumptime_table, so I wanted to avoid calling Thread::current() for each iteration.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4515


More information about the hotspot-runtime-dev mailing list