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

Saint Wesonga duke at openjdk.org
Tue Jan 10 17:42:16 UTC 2023


> There are 2 primary differences between the Windows ARM64 ABI and the macOS/Linux ARM64 ABI: variadic floating point arguments are passed in general purpose registers on Windows (instead of the vector registers). In addition to this, up to 64 bytes of a struct being passed to a variadic function can be placed in general purpose registers. This happens regardless of the type of struct (HFA or other generic struct). This means that a struct can be split across registers and the stack when invoking a variadic function.
> 
> This change introduces tests that compute the sum of the fields of structs containing 1-4 ints, floats, and doubles to verify that each field is correctly assigned a register or stack location when invoking a variadic function (both when the struct can be passed entirely in registers as well as when the struct spills onto the stack).
> 
> For details about the Foreign Function & Memory API, see JEP 434 defined at https://openjdk.org/jeps/434
> 
> The Windows ARM64 ABI conventions are documented at https://learn.microsoft.com/en-us/cpp/build/arm64-windows-abi-conventions
> 
> This work builds on @lewurm / Bernhard's branch at https://github.com/lewurm/openjdk/commits/foreign-windows-aarch64

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

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

Changes: https://git.openjdk.org/panama-foreign/pull/754/files
 Webrev: https://webrevs.openjdk.org/?repo=panama-foreign&pr=754&range=02
  Stats: 737 lines in 15 files changed: 710 ins; 1 del; 26 mod
  Patch: https://git.openjdk.org/panama-foreign/pull/754.diff
  Fetch: git fetch https://git.openjdk.org/panama-foreign pull/754/head:pull/754

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


More information about the panama-dev mailing list