RFR: 8264732: Clean up LinkResolver::vtable_index_of_interface_method() [v2]
    Vladimir Ivanov 
    vlivanov at openjdk.java.net
       
    Wed Apr  7 21:47:59 UTC 2021
    
    
  
On Wed, 7 Apr 2021 05:32:46 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Vladimir Ivanov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert jvmciCompilerToVM.cpp change.
>
> Seems quite reasonable - though unsure of the motivation.
> 
> Do we need a NoSafepointVerifier in LinkResolver::vtable_index_of_interface_method to ensure the raw Method* remains valid?
> 
> Thanks,
> David
Thinking more about it, `methodHandle` does look more natural than a raw `Method*` on `LinkResolver`.
What do you think about leaving `LinkResolver` as it is now and move `vtable_index_of_interface_method` implementation to `InstanceKlass` instead?
-------------
PR: https://git.openjdk.java.net/jdk/pull/3346
    
    
More information about the hotspot-runtime-dev
mailing list