RFR: 8314949: linux PPC64 Big Endian: Implementation of Foreign Function & Memory API [v5]

[Jorn Vernee]

On Mon, 4 Sep 2023 14:56:18 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> I've found a way to solve the remaining FFI problem on linux PPC64 Big Endian. Large structs (>8 Bytes) which are passed in registers or on stack require shifting the Bytes in the last slot if the size is not a multiple of 8. This PR adds the required functionality to the Java code.
>> 
>> Please review and provide feedback. There may be better ways to implement it. I just found one which works and makes the tests pass:
>> 
>> Test summary
>> ==============================
>>    TEST                                              TOTAL  PASS  FAIL ERROR   
>>    jtreg:test/jdk/java/foreign                          88    88     0     0   
>> 
>> 
>> Note: This PR should be considered as preparation work for AIX which also uses ABIv1.
>
> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Separate type cast from Shift bindings.

src/java.base/share/classes/jdk/internal/foreign/abi/Binding.java line 425:

> 423:             if (type == int.class || isSubIntType(type)) {
> 424:                 bindings.add(Binding.cast(type, long.class));
> 425:                 type = long.class;

This line seems redundant now (`type` is not used below).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15417#discussion_r1315035355