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