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