prep_cif.c (initialize_aggregate): Include tail padding in structure size.

	* src/prep_cif.c (initialize_aggregate): Include tail padding in
	structure size.
	* src/powerpc/linux64_closure.S (ffi_closure_LINUX64): Correct
	placement of float result.
	* testsuite/libffi.special/unwindtest.cc (closure_test_fn1): Correct
	cast of "resp" for big-endian 64 bit machines.

From-SVN: r71323

diff --git a/libffi/ChangeLog b/libffi/ChangeLog
index 2efe64b2129b75748f7554abc4ec6297f7ccbd16..15f9c6d54170e296512cf97cfab244a527eeac74 100644 (file)
--- a/libffi/ChangeLog
+++ b/libffi/ChangeLog
@@ -1,3 +1,12 @@
+2003-09-12  Alan Modra  <amodra@bigpond.net.au>
+
+       * src/prep_cif.c (initialize_aggregate): Include tail padding in
+       structure size.
+       * src/powerpc/linux64_closure.S (ffi_closure_LINUX64): Correct
+       placement of float result.
+       * testsuite/libffi.special/unwindtest.cc (closure_test_fn1): Correct
+       cast of "resp" for big-endian 64 bit machines.
+
 2003-09-11  Alan Modra  <amodra@bigpond.net.au>
 
        * src/types.c (double, longdouble): Merge identical SH and ARM

diff --git a/libffi/src/powerpc/linux64_closure.S b/libffi/src/powerpc/linux64_closure.S
index 6cad16ad77df5114dc8181a12d817ad8ef356048..2d44d67c20faa5ad0685a18309c87d7074e7647c 100644 (file)
--- a/libffi/src/powerpc/linux64_closure.S
+++ b/libffi/src/powerpc/linux64_closure.S
@@ -97,7 +97,7 @@ ffi_closure_LINUX64:
        addi %r1, %r1, 224
        blr
 # case FFI_TYPE_FLOAT
-       lfs %f1, 112+4(%r1)
+       lfs %f1, 112+0(%r1)
        mtlr %r0
        addi %r1, %r1, 224
        blr

diff --git a/libffi/src/prep_cif.c b/libffi/src/prep_cif.c
index d3c89c365e95ed15bcf8a9a752932ffeddacfe59..d60d98ee9f5cd5bbde8bbf4ada06dd86b9f408de 100644 (file)
--- a/libffi/src/prep_cif.c
+++ b/libffi/src/prep_cif.c
@@ -64,6 +64,15 @@ static ffi_status initialize_aggregate(/*@out@*/ ffi_type *arg)
       ptr++;
     }
 
+  /* Structure size includes tail padding.  This is important for
+     structures that fit in one register on ABIs like the PowerPC64
+     Linux ABI that right justify small structs in a register.
+     It's also needed for nested structure layout, for example
+     struct A { long a; char b; }; struct B { struct A x; char y; };
+     should find y at an offset of 2*sizeof(long) and result in a
+     total size of 3*sizeof(long).  */
+  arg->size = ALIGN (arg->size, arg->alignment);
+
   if (arg->size == 0)
     return FFI_BAD_TYPEDEF;
   else

diff --git a/libffi/testsuite/libffi.special/unwindtest.cc b/libffi/testsuite/libffi.special/unwindtest.cc
index 97b1536b2149fc04ef7764f0aba75c6dc0cbfaea..b6be367857a478283d4487af058c64183be63f5b 100644 (file)
--- a/libffi/testsuite/libffi.special/unwindtest.cc
+++ b/libffi/testsuite/libffi.special/unwindtest.cc
@@ -37,9 +37,9 @@ void closure_test_fn1(ffi_cif* cif,void* resp,void** args,
           (int)(*(int *)args[10]), (int)(*(float *)args[11]),
           (int)*(int *)args[12], (int)(*(int *)args[13]),
           (int)(*(int *)args[14]), *(int *)args[15],
-          (int)(long)userdata, *(int*)resp);
+          (int)(long)userdata, (int)*(ffi_arg*)resp);
     
-    throw *(int*)resp;
+    throw (int)*(ffi_arg*)resp;
 }
 
 typedef int (*closure_test_type1)(float, float, float, float, signed short,