RFR: 8307190: Refactor ref_at methods in Constant Pool [v2]
Matias Saavedra Silva
matsaave at openjdk.org
Wed May 17 17:40:02 UTC 2023
On Thu, 11 May 2023 21:52:20 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/HotSpotConstantPool.java line 438:
>>
>>> 436: *
>>> 437: * @param which constant pool index or constant pool cache index
>>> 438: * @param opcode bytecode
>>
>> Is this a param? You should remove the jvmci changes because they're not needed for this change.
>
> Or should the comment say that 'which' is the constant pool index only in this case?
The JVMCI should still be updated to match the internal functionality.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13872#discussion_r1196854307
More information about the hotspot-dev
mailing list