]> Zhao Yanbai Git Server - minix.git/commit
cdecl calling convention requires to push arguments on the stack in a
authorLorenzo Cavallaro <lorenzo@minix3.org>
Tue, 30 Mar 2010 09:36:46 +0000 (09:36 +0000)
committerLorenzo Cavallaro <lorenzo@minix3.org>
Tue, 30 Mar 2010 09:36:46 +0000 (09:36 +0000)
commit8dfc7699a6ffd4875ba4ead8215ed3a373b618f1
treebebc26858fcbd25c9424d33fa7b44c2cf73af01e
parent63e2d73d1b059e95c3b1a73194a2596787d3122f
cdecl calling convention requires to push arguments on the stack in a
reverse order to easily support variadic arguments. Thus, instead of
using the proper stdarg.h macros (that nowadays are
compiler-dependent), it may be tempting to directly take the address of
the last argument and considering it as the start of an array. This is
a shortcut that avoid looping to get all the arguments as the CPU
already pushed them on the stack before the call to the function.

Unfortunately, such an assumption is strictly compiler-dependent and
compilers are free to move the last argument on the stack, as a local
variable, and return the address of the location where the argument was
stored, if asked for. This will break things as the rest of the array's
argument are stored elsewhere (typically, a couple of words above the
location where the argument was stored).

This patch fixes the issue by allowing ACK to take the shortcut and
enabling gcc/llvm-gcc to follow the right way.
include/minix/compiler-ack.h
include/minix/compiler.h
include/stdarg.h
lib/libc/posix/_execl.c
lib/libc/posix/_execle.c
lib/libc/posix/_execlp.c
lib/libc/posix/_execve.c
lib/libc/posix/_execvp.c