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

Jorn Vernee jvernee at openjdk.java.net
Tue May 17 10:22:57 UTC 2022


On Mon, 16 May 2022 12:58:51 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use unaligned layout constants when filling in reconstituted structs (was accidentally dropped change)
>
> src/java.base/share/classes/jdk/internal/foreign/abi/CallingSequence.java line 109:
> 
>> 107:      * @return the caller method type.
>> 108:      */
>> 109:     public MethodType callerMethodType() {
> 
> Would it make sense to either rename these, or to point to the fact that these are equivalent to Linker::downcallType/upcallType ?

I don't think renaming them to down/upcallType makes sense, since both down/up calls use both the caller and callee method type. There can also be things that make these not equivalent to downcallType/upcallType, such as the SysV need to pass the number of floating point arguments for variadic calls as well: https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/jdk/internal/foreign/abi/x64/sysv/CallArranger.java#L109-L111

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

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


More information about the core-libs-dev mailing list