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