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