On Mon, 28 Nov 2022 01:12:18 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> test of tier1-5 passed. > > src/hotspot/share/prims/jvmtiTagMap.cpp line 254: > >> 252: if (obj_tag != current_tag ) { >> 253: hashmap->remove(o); >> 254: hashmap->add(o, obj_tag); > > This change is not atomic - is that a problem? The concurrency aspects of > using this map are not clear. add() is supposed to add a new object with tag. There is an assert() in its body that checks it. By removing the assert, add() can be used for updating as well. ------------- PR: https://git.openjdk.org/jdk/pull/11288