RFR: 8329150: Remove CDS support for LatestMethodCache [v2]
Ioi Lam
iklam at openjdk.org
Wed Mar 27 18:06:40 UTC 2024
On Wed, 27 Mar 2024 17:35:20 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Move definitions of _finalizer_register_cache close to the declaration of LatestMethodCache
>> - @coleenp comments - no need to expose LatestMethodCache in universe.hpp
>
> src/hotspot/share/memory/universe.cpp line 161:
>
>> 159: LatestMethodCache Universe::_throw_illegal_access_error_cache;
>> 160: LatestMethodCache Universe::_throw_no_such_method_error_cache;
>> 161: LatestMethodCache Universe::_do_stack_walk_cache;
>
> Just make these LatestMethodCache and not static data members in Universe, and move them up with the LatestMethodCache type declaration so there're all in one place.
Done. Thanks for the suggestion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18516#discussion_r1541609169
More information about the hotspot-runtime-dev
mailing list