RFR: 8339198: Remove tag field from AbstractPoolEntry
Adam Sotona
asotona at openjdk.org
Fri Sep 20 15:41:39 UTC 2024
On Thu, 19 Sep 2024 17:07:41 GMT, Chen Liang <liach at openjdk.org> wrote:
> The `tag` field in `AbstractPoolEntry` is redundant, as it's always of the same value for each subtype. This is now replaced with an override. This can reduce the footprint of the entries..
>
> The removal of `hash` was considered but withdrawn; in part because hash computations are heavy; in c2 things as simple as identity hash code are better stored in object fields for performance.
Looks good to me.
-------------
Marked as reviewed by asotona (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21093#pullrequestreview-2318634277
More information about the core-libs-dev
mailing list