RFR: 8311630: [s390] Implementation of Foreign Function & Memory API (Preview) [v2]

sid8606 duke at openjdk.org
Sat Jul 8 10:59:58 UTC 2023


On Fri, 7 Jul 2023 10:33:10 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> sid8606 has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address Amit's review comments
>
> test/jdk/java/foreign/TestIllegalLink.java line 57:
> 
>> 55: 
>> 56:     private static final boolean IS_SYSV = CABI.current() == CABI.SYS_V;
>> 57:     private static final boolean byteorder = ByteOrder.nativeOrder() == ByteOrder.LITTLE_ENDIAN;
> 
> Please rename this field to something more descriptive, e.g. `IS_LE` (and capitalize).

Thanks, Made the changes.

> test/jdk/java/foreign/TestLayouts.java line 46:
> 
>> 44: public class TestLayouts {
>> 45: 
>> 46:     boolean byteorder = ByteOrder.nativeOrder() == ByteOrder.LITTLE_ENDIAN;
> 
> Same here

Fixed

> test/jdk/java/foreign/callarranger/platform/PlatformLayouts.java line 312:
> 
>> 310:      * This class defines layout constants modelling standard primitive types supported by the S390 ABI.
>> 311:      */
>> 312:     public static final class S390 {
> 
> These are only needed if you plan on adding a CallArranger test as well.

Removed For now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14801#discussion_r1257226257
PR Review Comment: https://git.openjdk.org/jdk/pull/14801#discussion_r1257226355
PR Review Comment: https://git.openjdk.org/jdk/pull/14801#discussion_r1257226562


More information about the core-libs-dev mailing list