On Tue, Feb 10, 2015 at 10:35 AM, Richard Henderson <r...@redhat.com> wrote: > On 02/09/2015 12:40 AM, Andrew Pinski wrote: >> #ifndef LIBFFI_ASM >> +#ifdef __ILP32__ >> +typedef unsigned long long ffi_arg; >> +typedef signed long long ffi_sarg; >> +#else > > You need to set FFI_SIZEOF_ARG too. > > Otherwise it looks ok.
Thanks. I am testing the patch with the following added under the ILP32 check: +#define FFI_SIZEOF_ARG 8 +#define FFI_SIZEOF_JAVA_RAW 4 Thanks, Andrew Pinski > > > r~