diff options
author | bde <bde@FreeBSD.org> | 2002-09-29 13:42:27 +0000 |
---|---|---|
committer | bde <bde@FreeBSD.org> | 2002-09-29 13:42:27 +0000 |
commit | 61d4be4f4d0a6b2f0bbec889950c1a80991938ce (patch) | |
tree | 703f7918e90169ebbb2d144a3a1b5157b2db85d1 /lib | |
parent | 7ce1df07a88c69597f49afb85fe366f4800efdf8 (diff) | |
download | FreeBSD-src-61d4be4f4d0a6b2f0bbec889950c1a80991938ce.zip FreeBSD-src-61d4be4f4d0a6b2f0bbec889950c1a80991938ce.tar.gz |
Align the stack suitably for the version of gcc in FreeBSD-4 (provided
-fomit-frame-pointer is not used). This is mostly moot for -current
because gcc-3 does the alignment (slightly incorrectly) in main().
This patch is intended for easy MFC'ing and should be backed out in
-current soon since it causes compiler warnings and better fixes are
possible in -current. The best fix is to do nothing here and wait for
gcc to do stack alignment right. gcc-3 aligns the stack in main(), but
does it too late for main()'s local variables and too late for anything
called before main(). A misaligned stack is now more than an efficiency
problem, since some SSE instructions in some or all (hardware)
implementations trap on misaligned operands even if alignment checking
is not enabled.
PR: 41528:
Submitted by: NIIMI Satoshi <sa2c@sa2c.net> (original version)
MFC after: 3 days
Diffstat (limited to 'lib')
-rw-r--r-- | lib/csu/amd64/crt1.c | 33 | ||||
-rw-r--r-- | lib/csu/i386-elf/crt1.c | 33 |
2 files changed, 66 insertions, 0 deletions
diff --git a/lib/csu/amd64/crt1.c b/lib/csu/amd64/crt1.c index a9caf9e..43adc10 100644 --- a/lib/csu/amd64/crt1.c +++ b/lib/csu/amd64/crt1.c @@ -100,7 +100,40 @@ _start(char *ap, ...) monstartup(&eprol, &etext); #endif _init(); +#ifndef __GNUC__ exit( main(argc, argv, env) ); +#else + /* + * Some versions of gcc-2 expect the stack frame to be aligned as + * follows after it is set up in main(): + * + * +--------------+ <--- aligned by PREFERRED_STACK_BOUNDARY + * +%ebp (if any) + + * +--------------+ + * |return address| + * +--------------+ + * | arguments | + * | : | + * | : | + * +--------------+ + * + * We implement the above to fix just the usual case in FreeBSD-4. + * Alignment for main() is too compiler-dependent to handle correctly + * in all cases here (or in the kernel). E.g., a different alignment + * is required for at least gcc-2.95.4 even for the small variation + * of compiling main() with -fomit-frame-pointer. + */ + __asm__(" + andl $~0xf, %%esp # align stack to 16-byte boundary + subl $12+12, %%esp # space for args and padding + movl %0, 0(%%esp) + movl %1, 4(%%esp) + movl %2, 8(%%esp) + call main + movl %%eax, 0(%%esp) + call exit + " : : "r" (argc), "r" (argv), "r" (env) : "ax", "cx", "dx", "memory"); +#endif } #ifdef GCRT diff --git a/lib/csu/i386-elf/crt1.c b/lib/csu/i386-elf/crt1.c index a9caf9e..43adc10 100644 --- a/lib/csu/i386-elf/crt1.c +++ b/lib/csu/i386-elf/crt1.c @@ -100,7 +100,40 @@ _start(char *ap, ...) monstartup(&eprol, &etext); #endif _init(); +#ifndef __GNUC__ exit( main(argc, argv, env) ); +#else + /* + * Some versions of gcc-2 expect the stack frame to be aligned as + * follows after it is set up in main(): + * + * +--------------+ <--- aligned by PREFERRED_STACK_BOUNDARY + * +%ebp (if any) + + * +--------------+ + * |return address| + * +--------------+ + * | arguments | + * | : | + * | : | + * +--------------+ + * + * We implement the above to fix just the usual case in FreeBSD-4. + * Alignment for main() is too compiler-dependent to handle correctly + * in all cases here (or in the kernel). E.g., a different alignment + * is required for at least gcc-2.95.4 even for the small variation + * of compiling main() with -fomit-frame-pointer. + */ + __asm__(" + andl $~0xf, %%esp # align stack to 16-byte boundary + subl $12+12, %%esp # space for args and padding + movl %0, 0(%%esp) + movl %1, 4(%%esp) + movl %2, 8(%%esp) + call main + movl %%eax, 0(%%esp) + call exit + " : : "r" (argc), "r" (argv), "r" (env) : "ax", "cx", "dx", "memory"); +#endif } #ifdef GCRT |