RFR: 8307190: Refactor ref_at methods in Constant Pool [v4]

Coleen Phillimore coleenp at openjdk.org
Fri May 19 22:21:53 UTC 2023


On Fri, 19 May 2023 20:12:41 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Removed unused imports
>
> test/hotspot/jtreg/compiler/jvmci/compilerToVM/ConstantPoolTestsHelper.java line 113:
> 
>> 111:           case CONSTANT_METHODREF:
>> 112:           case CONSTANT_INTERFACEMETHODREF:
>> 113:               return Bytecodes.INVOKEVIRTUAL;
> 
> It's better to return INVOKEINTERFACE for CONSTANT_METHODREF.

Why?  It seems like it should be

`CONSTANT_METHODREF: should return Bytecodes.INVOKEVIRTUAL;
 CONSTANT_INTERFACEMETHODREF: should return Bytecodes.INVOKEVIRTUAL`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13872#discussion_r1199470585


More information about the hotspot-dev mailing list