[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