On Mon, 8 May 2023 14:15:18 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
>> The ResourceHashtable conversion for JDK-8292741 didn't add the resizing >> code. The old hashtable code was tuned for resizing in anticipation of >> large hashtables for JVMTI tags. This patch ports over the old hashtable >> resizing code. It also adds a ResourceHashtable::put_fast() function that >> prepends to the bucket list, which is also reclaims the performance of the >> old hashtable for this test with 10M tags. The ResourceHashtable put >> function is really a put_if_absent. This can be cleaned up in a future >> change. Also, the remove function needed a lambda to destroy the >> WeakHandle, since resizing requires copying entries. >> >> Tested with JVMTI and JDI tests locally, and tier1-4 tests. > > Coleen Phillimore has updated the pull request incrementally with one > additional commit since the last revision: > > Rename and comment put_when_absent. Not sure, if `put_if_absent_fast` is worth to consider. :) ------------- PR Comment: https://git.openjdk.org/jdk/pull/13818#issuecomment-1539448542