On Tue, 7 Mar 2023 19:25:58 GMT, Matias Saavedra Silva <matsa...@openjdk.org> 
wrote:

>> src/hotspot/share/oops/cpCache.cpp line 727:
>> 
>>> 725:   set_reference_map(nullptr);
>>> 726: #if INCLUDE_CDS
>>> 727:   if (_initial_entries != nullptr) {
>> 
>> @iklam with moving invokedynamic entries out, do you still need to save 
>> initialized entries ?  Does invokehandle need this? (Should have separate 
>> RFE if more cleanup is possible)
>
> This along with the previous comment about `_invokedynamic_references_map` 
> would probably be better suited for their own RFE. I think the scope of this 
> PR should be limited to the indy structure and its implementation, so any 
> changes related to invokehandle can be traced more easily.

ok, that's fine.

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

PR: https://git.openjdk.org/jdk/pull/12778

Reply via email to