diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0f3037d7209..9b7c4b24968 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,11 @@ +2009-03-19 Jakub Jelinek + + PR target/39496 + * config/i386/i386.c (ix86_function_regparm): Don't optimize local + functions using regparm calling conventions when not optimizing. + (ix86_function_sseregparm): Similarly for sseregparm calling + conventions. + 2009-03-19 Li Feng PR middle-end/39500 diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index bde1604c08c..18063e6b388 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -4309,7 +4309,9 @@ ix86_function_regparm (const_tree type, const_tree decl) return 2; /* Use register calling convention for local functions when possible. */ - if (decl && TREE_CODE (decl) == FUNCTION_DECL + if (decl + && TREE_CODE (decl) == FUNCTION_DECL + && optimize && !profile_flag) { /* FIXME: remove this CONST_CAST when cgraph.[ch] is constified. */ @@ -4394,7 +4396,7 @@ ix86_function_sseregparm (const_tree type, const_tree decl, bool warn) /* For local functions, pass up to SSE_REGPARM_MAX SFmode (and DFmode for SSE2) arguments in SSE registers. */ - if (decl && TARGET_SSE_MATH && !profile_flag) + if (decl && TARGET_SSE_MATH && optimize && !profile_flag) { /* FIXME: remove this CONST_CAST when cgraph.[ch] is constified. */ struct cgraph_local_info *i = cgraph_local_info (CONST_CAST_TREE(decl)); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 4065c86403f..541c73213f2 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2009-03-19 Jakub Jelinek + + PR target/39496 + * gcc.target/i386/pr39496.c: New test. + * g++.dg/other/pr39496.C: New test. + 2009-03-19 Li Feng PR middle-end/39500 diff --git a/gcc/testsuite/g++.dg/other/pr39496.C b/gcc/testsuite/g++.dg/other/pr39496.C new file mode 100644 index 00000000000..94e33abbe11 --- /dev/null +++ b/gcc/testsuite/g++.dg/other/pr39496.C @@ -0,0 +1,35 @@ +// PR target/39496 +// { dg-do compile { target { { i?86-*-linux* x86_64-*-linux* } && ilp32 } } } +// { dg-options "-O0 -fverbose-asm -fno-omit-frame-pointer -msse2 -mfpmath=sse" } +// Verify that {foo,bar}{,2}param are all passed on the stack, using +// normal calling conventions, when not optimizing. +// { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*fooparam," } } +// { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*barparam," } } +// { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*foo2param," } } +// { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*bar2param," } } + +static inline int foo (int fooparam) +{ + return fooparam; +} + +static int bar (int barparam) +{ + return foo (barparam); +} + +static inline double foo2 (double foo2param) +{ + return foo2param; +} + +static double bar2 (double bar2param) +{ + return foo2 (bar2param); +} + +int +main () +{ + return bar (0) + bar2 (0.0); +} diff --git a/gcc/testsuite/gcc.target/i386/pr39496.c b/gcc/testsuite/gcc.target/i386/pr39496.c new file mode 100644 index 00000000000..d90a2c7f3ea --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr39496.c @@ -0,0 +1,35 @@ +/* PR target/39496 */ +/* { dg-do compile { target { { i?86-*-linux* x86_64-*-linux* } && ilp32 } } } */ +/* { dg-options "-O0 -fverbose-asm -fno-omit-frame-pointer -msse2 -mfpmath=sse" } */ +/* Verify that {foo,bar}{,2}param are all passed on the stack, using + normal calling conventions, when not optimizing. */ +/* { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*fooparam," } } */ +/* { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*barparam," } } */ +/* { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*foo2param," } } */ +/* { dg-final { scan-assembler "\[^0-9-\]8\\(%ebp\\),\[^\n\]*bar2param," } } */ + +static inline int foo (int fooparam) +{ + return fooparam; +} + +static int bar (int barparam) +{ + return foo (barparam); +} + +static inline double foo2 (double foo2param) +{ + return foo2param; +} + +static double bar2 (double bar2param) +{ + return foo2 (bar2param); +} + +int +main () +{ + return bar (0) + bar2 (0.0); +}