ag/cpython-source-deps
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Clean up code to appease modern GCC compiler.

Browse Source
This commit is contained in:
Anthony Green
2013-11-18 06:52:29 -05:00
parent 16d56c51ad
commit 2f450822a8
3 changed files with 18 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
ChangeLog
View File

@@ -1,3 +1,15 @@
2013-11-18 Anthony Green <green@moxielogic.com>
* src/arm/ffi.c (ffi_call): Hoist declaration of temp to top of
function.
* src/arm/ffi.c (ffi_closure_inner): Moderize function declaration
to appease compiler.
Thanks for Gregory P. Smith <greg@krypto.org>.
2013-11-18 Anthony Green <green@moxielogic.com>
* README (tested): Mention PowerPC ELFv2.
2013-11-16 Alan Modra <amodra@gmail.com> 2013-11-16 Alan Modra <amodra@gmail.com>
* src/powerpc/ppc_closure.S: Move errant #endif to where it belongs. * src/powerpc/ppc_closure.S: Move errant #endif to where it belongs.

3
README
View File

@@ -80,7 +80,8 @@ tested:
| PowerPC | Mac OSX | GCC | | PowerPC | Mac OSX | GCC |
| PowerPC | FreeBSD | GCC | | PowerPC | FreeBSD | GCC |
| PowerPC 64-bit | FreeBSD | GCC | | PowerPC 64-bit | FreeBSD | GCC |
| PowerPC 64-bit | Linux | GCC | | PowerPC 64-bit | Linux ELFv1 | GCC |
| PowerPC 64-bit | Linux ELFv2 | GCC |
| S390 | Linux | GCC | | S390 | Linux | GCC |
| S390X | Linux | GCC | | S390X | Linux | GCC |
| SPARC | Linux | GCC | | SPARC | Linux | GCC |

11
src/arm/ffi.c
View File

@@ -307,11 +307,11 @@ void ffi_call(ffi_cif *cif, void (*fn)(void), void *rvalue, void **avalue)
int vfp_struct = (cif->flags == FFI_TYPE_STRUCT_VFP_FLOAT int vfp_struct = (cif->flags == FFI_TYPE_STRUCT_VFP_FLOAT
|| cif->flags == FFI_TYPE_STRUCT_VFP_DOUBLE); || cif->flags == FFI_TYPE_STRUCT_VFP_DOUBLE);
unsigned int temp;
ecif.cif = cif; ecif.cif = cif;
ecif.avalue = avalue; ecif.avalue = avalue;
unsigned int temp;
/* If the return value is a struct and we don't have a return */ /* If the return value is a struct and we don't have a return */
/* value address then we need to make one */ /* value address then we need to make one */
@@ -367,11 +367,8 @@ void ffi_closure_VFP (ffi_closure *);
/* This function is jumped to by the trampoline */ /* This function is jumped to by the trampoline */
unsigned int unsigned int
ffi_closure_inner (closure, respp, args, vfp_args) ffi_closure_inner (ffi_closure *closure,
ffi_closure *closure; void **respp, void *args, void *vfp_args)
void **respp;
void *args;
void *vfp_args;
{ {
// our various things... // our various things...
ffi_cif *cif; ffi_cif *cif;