[foreign] RFR: 8223808: initial port for AArch64

Nick Gasson nick.gasson at arm.com
Tue May 28 10:36:42 UTC 2019


Hi Maurizio,

On 28/05/2019 16:48, Maurizio Cimadamore wrote:
> 
> * don't you need some extra set of constants under NativeTypes (for 
> aarch platform?)

Yes, we ought to. I think I reused the LittleEndian.SysVABI ones 
initially as they almost exactly match except for long double.

For definitions like this:

     public static LayoutType<Short> SHORT = isWindows ?
             LittleEndian.WinABI.SHORT : LittleEndian.SysVABI.SHORT;

Do you want me to change it into something like:

    public static LayoutType<Short> SHORT = isX86 ?
             (isWindows ?
              LittleEndian.WinABI.SHORT : LittleEndian.SysVABI.SHORT)
             : LittleEndian.AArch64.SHORT;

I can see this getting quite messy as we add more platform combinations.

> 
> * UnalignedStructTest - why doesn't it work? There's an f128 there but 
> we're not actually reading/setting it - it's mostly there to force 
> alignment a certain way
> 

I think it disabled it at one point because it was failing and saw it 
used long double so just assumed that was the cause. But I just went 
back and tested it and it passes so must be have been something else I 
fixed later. I'll re-enable it.

Thanks,
Nick


More information about the panama-dev mailing list