RFR: 8267089: Use typedef KVHashtable for ID2KlassTable

Yi Yang yyang at openjdk.java.net
Mon May 17 02:24:49 UTC 2021


On Thu, 13 May 2021 07:08:11 GMT, Yi Yang <yyang at openjdk.org> wrote:

> While backporting some stuff to jdk11, I noticed main functionalities of ID2KlassTable have been removed afterJDK-8213587, it seems that we can use a simple typedef KVHashtable instead.

> This is nice! It's not "seems" , we can use typedef.
> 
> Another nit you can do along with this PR. move * to its class.
> 
> ```c++
>   ID2KlassTable* table() {
>     return &_id2klass_table;
>   }
> ```
Hi Xin, do you mean `typedef KVHashtable<int, InstanceKlass*, mtInternal>* ID2KlassTable`? This is not feasible since _id2klass_table is not a type of pointer.

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

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


More information about the hotspot-runtime-dev mailing list