RFR: 8261998: Remove unused shared entry support from utilities/hashtable

Coleen Phillimore coleenp at openjdk.java.net
Fri Feb 19 02:58:42 UTC 2021


On Fri, 19 Feb 2021 02:39:20 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this small cleanup in the utilities/hashtable facility.  The
> support for "shared" entries is no longer needed or used, so is being deleted.
> 
> Testing:
> mach5 tier1-4 (some CDS tests are in tier4)

We might want to share other hashtables like this, like the loader constraint table, but I don't think this will be needed.

src/hotspot/share/prims/jvmtiTagMapTable.cpp line 255:

> 253:       }
> 254:       // get next entry
> 255:       entry = (JvmtiTagMapEntry*)HashtableEntry<WeakHandle, mtServiceability>::make_ptr(*p);

Nice to get rid of this.  It would be nice if the hashtables didn't need to declare bucket() and bucket_addr() with all these casts but some template nonsense makes it not compile.

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

Marked as reviewed by coleenp (Reviewer).

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


More information about the serviceability-dev mailing list