[foreign] RFR 8222739: Port CallingSequenceTest to Windows

Jorn Vernee jbvernee at xs4all.nl
Sat Apr 27 11:48:51 UTC 2019


Pushed. Thanks for the review!

Jorn

Henry Jen schreef op 2019-04-27 03:53:
> 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