On Mon, 12 Jun 2023 22:24:26 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> resolvedIndyEntry.hpp was added in 
>> [JDK-8301995](https://bugs.openjdk.org/browse/JDK-8301995) and is included 
>> in the popular cpCache.hpp. As a result, resolvedIndyEntry.hpp is included 
>> in 807 out of about 1160 hotspot .o files.
>> 
>> The contents of resolvedIndyEntry.hpp is infrequently used. Its inclusion 
>> should be moved from cpCache.hpp to cpCache.inline.hpp. This improves 
>> hotspot build time.
>> 
>> After this PR, resolvedIndyEntry.hpp is included in only 30 hotspot .o files.
>
> Looks good.
> Thanks,
> Serguei

Thanks @sspitsyn @coleenp @matias9927 for the review.

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

PR Comment: https://git.openjdk.org/jdk/pull/14427#issuecomment-1590238854

Reply via email to