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