RFR: 8292446: Make TableRateStatistics optional in CHT [v4]

Johan Sjölén duke at openjdk.org
Fri Aug 19 10:50:38 UTC 2022


On Thu, 18 Aug 2022 19:16:18 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Johan Sjölén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Declare functions inline
>
> src/hotspot/share/utilities/concurrentHashTable.hpp line 394:
> 
>> 392:   static const size_t DEFAULT_START_SIZE_LOG2 = 13;
>> 393:   static const size_t DEFAULT_GROW_HINT = 4; // Chain length
>> 394:   static const bool DEFAULT_ENABLE_STATISTICS = true;
> 
> Ok, one last thing. I wonder if we should default to false for enabling table rate statistics?  Only the StringTable and SymbolTable need it.

Fixed.

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

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


More information about the hotspot-dev mailing list