This is the mail archive of the
libffi-discuss@sourceware.org
mailing list for the libffi project.
Re: libffi rebased
- From: Dan Witte <dwitte at mozilla dot com>
- To: Anthony Green <green at moxielogic dot com>
- Cc: libffi-discuss at sourceware dot org
- Date: Mon, 22 Nov 2010 13:22:31 -0800 (PST)
- Subject: Re: libffi rebased
Also, the ARM build doesn't. Patch attached fixes it for me.
Cheers,
Dan
----- Original Message -----
> ----- Original Message -----
> > Dan - it would be great if you could test out the git tree to make
> > sure
> > I didn't botch anything. I'll push the remaining patches into GCC
> > once
> > you give the all clear.
>
> Hmm. It looks like one of my csets was partially reverted -- just in
> msvcc.sh (!):
>
> git diff 84e8de6e9fc19388f6f1102c013b7d0d52940ecc
> 2db72615b50eb5c0f29725c02c740a2f0d7fc7d9 msvcc.sh
>
> shouldn't be there. It also looks like the 'if MOXIE' bits here were
> reverted:
>
> git diff 84e8de6e9fc19388f6f1102c013b7d0d52940ecc
> 2db72615b50eb5c0f29725c02c740a2f0d7fc7d9 Makefile.am
>
> (which would evidently require an autoreconf.) I can't speak to the
> src/arm/ changes, but those are the two I noticed.
>
> Does that sound right?
>
> Cheers,
> Dan.
diff --git a/src/arm/ffitarget.h b/src/arm/ffitarget.h
--- a/src/arm/ffitarget.h
+++ b/src/arm/ffitarget.h
@@ -34,19 +34,19 @@ typedef unsigned long ffi_arg;
typedef signed long ffi_sarg;
typedef enum ffi_abi {
FFI_FIRST_ABI = 0,
FFI_SYSV,
FFI_VFP,
FFI_LAST_ABI,
#ifdef __ARM_PCS_VFP
- FFI_DEFAULT_ABI = FFI_VFP,
+ FFI_DEFAULT_ABI = FFI_VFP
#else
- FFI_DEFAULT_ABI = FFI_SYSV,
+ FFI_DEFAULT_ABI = FFI_SYSV
#endif
} ffi_abi;
#endif
#define FFI_EXTRA_CIF_FIELDS \
int vfp_used; \
short vfp_reg_free, vfp_nargs; \
signed char vfp_args[16] \