FreeChainXenon/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

re PR target/79749 (Many sparc testcases FAIL at -O0 with -fomit-frame-pointer)

Browse source 
PR target/79749
	* config/sparc/sparc.c (sparc_frame_pointer_required): Add missing
	condition on optimize for the leaf function test.

From-SVN: r245791
This commit is contained in:
Eric Botcazou 2017-02-28 22:15:54 +00:00 • committed by Eric Botcazou
parent cbf6ad9d61
commit 7a1ae92374
4 changed files with 33 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
gcc/ChangeLog
View file

@ -1,3 +1,9 @@
2017-02-28 Eric Botcazou <ebotcazou@adacore.com>
PR target/79749
* config/sparc/sparc.c (sparc_frame_pointer_required): Add missing
condition on optimize for the leaf function test.
2017-02-28 Martin Liska <mliska@suse.cz>
PR lto/79625

5
gcc/config/sparc/sparc.c
View file

@ -12180,8 +12180,9 @@ sparc_frame_pointer_required (void)
if (TARGET_FLAT)
return false;
/* Otherwise, the frame pointer is required if the function isn't leaf. */
return !(crtl->is_leaf && only_leaf_regs_used ());
/* Otherwise, the frame pointer is required if the function isn't leaf, but
we cannot use sparc_leaf_function_p since it hasn't been computed yet. */
return !(optimize > 0 && crtl->is_leaf && only_leaf_regs_used ());
}
/* The way this is structured, we can't eliminate SFP in favor of SP

4
gcc/testsuite/ChangeLog
View file

@ -1,3 +1,7 @@
2017-02-28 Eric Botcazou <ebotcazou@adacore.com>
* gcc.target/sparc/20170228-1.c: New test.
2017-02-28 Uros Bizjak <ubizjak@gmail.com>
* gcc.target/i386/invsize-1.c: New test.

20
gcc/testsuite/gcc.target/sparc/20170228-1.c Normal file
View file

@ -0,0 +1,20 @@
/* PR target/79749 */
/* Reported by Rainer Orth <ro@gcc.gnu.org> */
/* { dg-do run } */
/* { dg-options "-fomit-frame-pointer" } */
extern void abort (void);
int foo (int x1, int x2, int x3, int x4, int x5, int x6, int x7)
{
return x7;
}
int main (void)
{
if (foo (100, 200, 300, 400, 500, 600, 700) != 700)
abort ();
return 0;
}