RFR: 8329150: Remove CDS support for LatestMethodCache [v2]

Ioi Lam iklam at openjdk.org
Wed Mar 27 18:06:39 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 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/18516/files
  - new: https://git.openjdk.org/jdk/pull/18516/files/1db28dc6..6c04bb24

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18516&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18516&range=00-01

  Stats: 18 lines in 2 files changed: 6 ins; 12 del; 0 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