The following code, compiled for arm-none-elf at -O2: ~~~~~~~~~~ #define noinline __attribute__((noinline)) typedef struct { int data[4]; } arr16_t; int result = 0; void noinline func2 (int i, int j, arr16_t arr) { result = (arr.data[0] != 1 || arr.data[1] != 2 || arr.data[2] != 3 || arr.data[3] != 4); } void func1 (int i, int j, int k, int l, int m, int n, arr16_t a) { func2(i, j, a); } int main(int argc, const char *argv[]) { arr16_t arr = {{1, 2, 3, 4}}; func1(0, 0, 0, 0, 0, 0, arr); return result; } ~~~~~~~~~~ The call to func2 should be optimized into a sibcall, however store_one_arg() is preventing it because it incorrectly identifies the arr16_t coming into func1 as overlapping on the stack where it needs to put the outgoing arr16_t for the call to func2. In fact, the incoming arg only uses [SP,SP+7] (the other two words are passed in registers) and the outgoing arg uses [SP+8,SP+23].
I have a patch which addresses this (and pr32602) -- I will submit it shortly.
Subject: Bug 32603 Author: jconner Date: Fri Jul 6 16:57:19 2007 New Revision: 126422 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=126422 Log: 2007-07-06 Josh Conner <jconner@apple.com> PR middle-end/32602 PR middle-end/32603 * calls.c (store_one_arg): Handle arguments which are partially on the stack when detecting argument overlap. 2007-07-06 Josh Conner <jconner@apple.com> PR middle-end/32602 * gcc.dg/sibcall-8.c: New test. 2007-07-06 Josh Conner <jconner@apple.com> PR middle-end/32603 * gcc.target/arm/sibcall-1.c: New test. Added: trunk/gcc/testsuite/gcc.dg/sibcall-8.c trunk/gcc/testsuite/gcc.target/arm/sibcall-1.c Modified: trunk/gcc/ChangeLog trunk/gcc/calls.c trunk/gcc/testsuite/ChangeLog
Fixed.