RFR: 8335007: Inline OopMapCache table

Coleen Phillimore coleenp at openjdk.org
Wed Jun 26 13:02:10 UTC 2024


On Tue, 25 Jun 2024 13:26:33 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> OopMapCache uses a (closed hashing) table. The table is C-heap allocated. OopMapCache itself is also C-heap allocated. Since table size is fixed, we can just inline the table to OopMapCache and save one unnecessary pointer-hop on lookup.

This change looks fine.  If you change _size, you should do the same with _probe_depth.   Until recently, there's not a lot of maintenance work in this code, so if you leave it, I'm fine with that.

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19882#pullrequestreview-2141742552


More information about the hotspot-runtime-dev mailing list