RFR: 8301992: Embed SymbolTable CHT node [v3]
Coleen Phillimore
coleenp at openjdk.org
Wed Feb 15 22:56:15 UTC 2023
On Wed, 15 Feb 2023 20:30:30 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:
>
> remove unused allocate_symbol() declaration in symbolTable.hpp
This changes look great! thank you!
-------------
Marked as reviewed by coleenp (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12562
More information about the hotspot-dev
mailing list