[foreign-memaccess+abi] RFR: 8295290: Add Windows ARM64 ABI support to the Foreign Function & Memory API [v3]

[Jorn Vernee]

On Tue, 10 Jan 2023 17:47:31 GMT, Saint Wesonga <duke at openjdk.org> wrote:

>> Saint Wesonga has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
>> 
>>  - Remove redundant check
>>  - Delete VaList
>>  - Merge branch 'foreign-memaccess+abi' into WindowsAArch64ABI
>>  - Merge branch 'foreign-memaccess+abi' into WindowsAArch64ABI
>>  - Update Windows FFM implementation to match latest Preview API
>>  - Merge branch 'foreign-memaccess+abi' into WindowsAArch64ABI
>>  - Simplify newly added tests in TestVarArgs
>>  - Move storage decisions into StorageCalculator
>>  - Remove toSessionImpl method
>>  - Remove unnecessary null check
>>  - ... and 2 more: https://git.openjdk.org/panama-foreign/compare/5427e47b...ccae2d80
>
>> Few more comments inline.
>> 
>> FWIW, I think CallArranger can be cleaned up further, and I've been playing with a patch [1](https://github.com/openjdk/panama-foreign/compare/pull/754/head...JornVernee:panama-foreign:Refector_CallArranger). But I'll file a followup PR for that.
> 
> @JornVernee just to clarify, do you expect more refactoring of the CallArranger in this PR or will that be subsumed by your patch?

@swesonga No, I think this PR is good. I'll file a followup PR myself for the other refactoring I did.

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

PR: https://git.openjdk.org/panama-foreign/pull/754