RFR: 8315559: Delay TempSymbol cleanup to avoid symbol table churn [v7]

Aleksey Shipilev shade at openjdk.org
Fri Nov 10 11:46:06 UTC 2023


On Thu, 9 Nov 2023 18:12:55 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Oli Gillespie has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Set queue size to power of 2, use constant in test
>
> src/hotspot/share/oops/symbolHandle.hpp line 47:
> 
>> 45: template <bool TEMP>
>> 46: class SymbolHandleBase : public StackObj {
>> 47:   static Symbol* volatile _cleanup_delay_queue[];
> 
> This looks off. Should be `static Symbol** volatile _cleanup_delay_queue`, maybe?

Nevermind, that's not doable here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16398#discussion_r1389292875


More information about the hotspot-dev mailing list