RFR: 8309808: BytecodeTracer prints wrong BSM for invokedynamic [v2]

Ioi Lam iklam at openjdk.org
Wed Jun 14 21:17:15 UTC 2023


On Wed, 14 Jun 2023 18:10:08 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   code clean up
>
> src/hotspot/share/interpreter/bytecodeTracer.cpp line 310:
> 
>> 308:   } else {
>> 309:     assert(ConstantPool::is_invokedynamic_index(i), "not secondary index?");
>> 310:     i = ConstantPool::decode_invokedynamic_index(i);
> 
> This could be changed to int indy_index = ... instead of reassigning i

Thanks. I've added an `indy_index` variable.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14429#discussion_r1230175766


More information about the hotspot-dev mailing list