On Wed, 2 Nov 2022 20:44:58 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
>> Use identity_hash for objects in the JVMTI TagMap table. If the object has >> no hashcode, it's not in the table. >> Tested with tier1-6. > > Coleen Phillimore has updated the pull request with a new target base due to > a merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains two additional > commits since the last revision: > > - Merge branch 'master' into jvmti > - 8256072: Eliminate JVMTI tagmap rehashing Looks good. Yay code deletion! I was particularly happy to see `set_needs_rehashing` removed from all the GCs. As a followup, I think `CollectedHeap::hash_code` is unused after this change. ------------- Marked as reviewed by kbarrett (Reviewer). PR: https://git.openjdk.org/jdk/pull/10938