[foreign] RFR: 8223808: initial port for AArch64
Nick Gasson
nick.gasson at arm.com
Wed May 29 07:00:30 UTC 2019
Hi Maurizio,
On 28/05/2019 19:25, Maurizio Cimadamore wrote:
> Actually, a cleaner way to get there is:
>
> public static LayoutType<Short> SHORT = pick(LittleEndian.SysVABI.SHORT,
> LittleEndian.WinABI.SHORT, LittleEndian.AArch64.SHORT);
>
> and then define the logic inside pick() to select the right value
> depending on platform/os.
>
Yes this is much better. I've done this in webrev.3 here:
http://cr.openjdk.java.net/~ngasson/foreign/8223808/webrev.3/
I added NativeTypes.AArch64ABI which differs from SysVABI in that long
double is 128 bits and char is unsigned. I wonder if we should add a
"SCHAR" signed char type to NativeTypes? As currently we have CHAR and
UCHAR which are both unsigned on AArch64.
Also fixed the x86 regression found by Jorn and re-enabled the
UnalignedStructTest test.
Thanks,
Nick
More information about the panama-dev
mailing list