On Tue, 10 Jan 2023 16:56:31 GMT, Afshin Zafari <d...@openjdk.org> wrote:

>> test of tier1-5 passed.
>
> Afshin Zafari has updated the pull request incrementally with three 
> additional commits since the last revision:
> 
>  - 8292741: Convert JvmtiTagMapTable to ResourceHashtable
>  - 8292741: Convert JvmtiTagMapTable to ResourceHashtable
>  - 8292741: Convert JvmtiTagMapTable to ResourceHashtable

src/hotspot/share/prims/jvmtiTagMap.cpp line 442:

> 440:     delete _fields->at(i);
> 441:   }
> 442:   delete _fields;

The changes here to ClassFieldMap are not part of the change, can you revert 
them (also leave mtInternal even though that's wrong).

src/hotspot/share/prims/jvmtiTagMap.hpp line 37:

> 35: class JvmtiTagMapKeyClosure;
> 36: 
> 37: class JvmtiTagMap :  public CHeapObj<mtInternal> {

Change to mtServiceability

src/hotspot/share/prims/jvmtiTagMapTable.hpp line 76:

> 74:                               JvmtiTagMapKey::equals> ResizableResourceHT;
> 75: 
> 76: class JvmtiTagMapTable : public CHeapObj<mtInternal> {

Change this to mtServiceability

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

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

Reply via email to