RFR: 8286669: Replace MethodHandle specialization with ASM in mainline [v3]

Jorn Vernee jvernee at openjdk.java.net
Wed May 18 11:27:31 UTC 2022


On Tue, 17 May 2022 11:18:20 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/foreign/abi/BindingSpecializer.java line 943:
>> 
>>> 941:         Z, B, S, C, I, J, F, D, L;
>>> 942: 
>>> 943:         static BasicType of(Class<?> cls) {
>> 
>> This seems a duplication of ASM Type class for no good reason, Type.getOpcode(ILOAD) or Type.getOpcode(IRETURN) gives you the proper variant opcode
>
> Didn't know about that. Neat!

Removed the `BasicType` enum, switched to using `Type` and `Type.getSort()`.

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

PR: https://git.openjdk.java.net/jdk/pull/8685


More information about the core-libs-dev mailing list