On Mon, 4 Sep 2023 12:22:00 GMT, Martin Doerr <mdo...@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:
> 
>   Split Shift into ShiftLeft and ShiftRight + minor improvements.

Hmm. Do you see a good place for such a comment? Maybe it would be better to 
use a different size for the last chunk. Maybe three or five Bytes. That's even 
less straight-forward.

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

PR Comment: https://git.openjdk.org/jdk/pull/15417#issuecomment-1705403519

Reply via email to