On Tue, 17 May 2022 11:18:20 GMT, Jorn Vernee <jver...@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

Reply via email to