[foreign] RFR 8222739: Port CallingSequenceTest to Windows

Henry Jen henry.jen at oracle.com
Sat Apr 27 01:53:23 UTC 2019


Looks reasonable to me.

Cheers,
Henry


> On Apr 23, 2019, at 4:10 AM, Jorn Vernee <jbvernee at xs4all.nl> wrote:
> 
> Hi,
> 
> I have ported the SysV CallingSequenceTest to Windows (and fixed a little bug in the process as well :) ).
> 
> In an effort to reduce code duplication in the tests, I have create a testing utility that automatically checks that the number of argument classes is correct. I've updated the existing SysV CallingSequenceTest to use this utility as well. You just have to specify the argument, and the expected bindings e.g. :
> 
>    new Verifier(new CallingSequenceBuilderImpl(null))
>            .arg(Layout.of("[u64u64u64u64]"),
>                    binding(STACK_ARGUMENT_SLOT, 0),
>                    binding(STACK_ARGUMENT_SLOT, 8),
>                    binding(STACK_ARGUMENT_SLOT, 16),
>                    binding(STACK_ARGUMENT_SLOT, 24))
>            .check(false);
> 
> Specifies 1 argument with the layout "[u64u64u64u64]", which is expected to have 4 bindings.
> 
> Please review the following.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222739
> Webrev: http://cr.openjdk.java.net/~jvernee/panama/webrevs/8222739/webrev.00/
> 
> Thanks,
> Jorn



More information about the panama-dev mailing list