RFR: 8301992: Embed SymbolTable CHT node [v4]

Coleen Phillimore coleenp at openjdk.org
Thu Feb 16 18:48:29 UTC 2023


On Thu, 16 Feb 2023 16:56:56 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> Please review this patch for embedding the Symbol inside a ConcurrentHashTable Node instead of having a pointer to a Symbol. This eliminates malloc/free for each Symbol.
>> 
>> This patch is co-authored by @robehn.
>> 
>> Passed tiers 1 - 4 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add Afree() for permanent symbols

This seems fine.  I haven't actually observed us getting to this code, except maybe once, but you might as well keep it.

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

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.org/jdk/pull/12562


More information about the hotspot-dev mailing list