RFR: 8270333: -XX:+VerifyStringTableAtExit should not do linear search [v3]

David Holmes dholmes at openjdk.java.net
Wed Jul 14 06:36:11 UTC 2021


On Wed, 14 Jul 2021 06:02:35 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> The implementation of `-XX:+VerifyStringTableAtExit` used a linear search to check for duplicated strings in the interned string table. This would be excessively slow if there were more than a few thousand strings.
>> 
>> This PR uses a hashtable so we can avoid doing a linear search. The table is sized such that each bucket has 8 items on average.
>> 
>> Since this code is executed only for verification, I added a new `java_lang_String::hash_code_noupdate(oop)` function, so we can avoid causing side effects.
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixed build and make sure table size is not zero

Okay ... I'm not sure why you want the code to be duplicated though?

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4772


More information about the hotspot-runtime-dev mailing list