RFR(S): 8240204: Optimize package handling for archived classes

[Claes Redestad]

Hi Yumin,

looks good to me!

Pre-existing, but in klassFactory.cpp path_index isn't used anywhere
else, so could fold it into the call:

   new_ik->set_classpath_index(ik->shared_classpath_index())

Thanks!

/Claes

On 2020-04-17 07:56, Yumin Qi wrote:
> Hi,
> 
>    Please review:
>    bug: 8240204: https://bugs.openjdk.java.net/browse/JDK-8240204
>    webrev: http://cr.openjdk.java.net/~minqi/8240204/webrev-01/
> 
>    Summary: Move ClassLoader::add_package to InstanceKlass since what it 
> does is calling pkg->set_classpath_index. Also a minor optimization for 
> InstanceKlass::set_package, for shared class, avoid call 
> check_prohibited_package since CDS does not archive prohibited classes.
> 
>    Tests: hs-tier1-4
>    Performance data: javac and zprint showed a little improvement.
> 
>    Thanks
>    Yumin