RFR: 8329150: Remove CDS support for LatestMethodCache [v6]
Ioi Lam
iklam at openjdk.org
Fri Mar 29 16:14:03 UTC 2024
> The CDS support for LatestMethodCache has no meaningful performance benefit. I am removing it for simplcity.
>
> I also cleaned up the code a bit
> - No need to put LatestMethodCache in universe.hpp.
> - No need to use TRAPS in the init function
> - No need to malloc the LatestMethodCache objects
Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
- Merge branch 'master' into 8329150-remove-cds-support-for-LatestMethodCache
- more clean up
- aligned and simplified comments
- Moved methods relocated to LatestMethodCache close to each other
- Move definitions of _finalizer_register_cache close to the declaration of LatestMethodCache
- @coleenp comments - no need to expose LatestMethodCache in universe.hpp
- 8329150: Remove CDS support for LatestMethodCache
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18516/files
- new: https://git.openjdk.org/jdk/pull/18516/files/5eb86612..57bacfe3
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=18516&range=05
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=18516&range=04-05
Stats: 2824 lines in 129 files changed: 1394 ins; 876 del; 554 mod
Patch: https://git.openjdk.org/jdk/pull/18516.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18516/head:pull/18516
PR: https://git.openjdk.org/jdk/pull/18516
More information about the hotspot-runtime-dev
mailing list