diff options
| author | Nathan Sidwell <nathan@codesourcery.com> | 2006-12-21 09:03:18 +0000 |
|---|---|---|
| committer | Nathan Sidwell <nathan@gcc.gnu.org> | 2006-12-21 09:03:18 +0000 |
| commit | 970af8677126773a729c5fde809ab307b6494082 (patch) | |
| tree | d16ac4c285b07d9272716d077f44f7d69f5968b5 /gcc/reload1.c | |
| parent | a957d77fc36e52b459a42043b8ea9cadab574277 (diff) | |
| download | gcc-970af8677126773a729c5fde809ab307b6494082.zip gcc-970af8677126773a729c5fde809ab307b6494082.tar.gz gcc-970af8677126773a729c5fde809ab307b6494082.tar.bz2 | |
re PR target/28966 (-maltivec -m32 causes the stack to be saved and restored even though there is no need for it)
gcc/
PR target/28966
PR target/29248
* reload1.c (reload): Realign stack after it changes size.
gcc/testsuite/
PR target/28966
PR target/29248
* gcc.dg/rs6000-leaf.c: New.
From-SVN: r120102
Diffstat (limited to 'gcc/reload1.c')
| -rw-r--r-- | gcc/reload1.c | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/gcc/reload1.c b/gcc/reload1.c index 4543835..efa1a39 100644 --- a/gcc/reload1.c +++ b/gcc/reload1.c @@ -888,16 +888,8 @@ reload (rtx first, int global) { int something_changed; int did_spill; - HOST_WIDE_INT starting_frame_size; - /* Round size of stack frame to stack_alignment_needed. This must be done - here because the stack size may be a part of the offset computation - for register elimination, and there might have been new stack slots - created in the last iteration of this loop. */ - if (cfun->stack_alignment_needed) - assign_stack_local (BLKmode, 0, cfun->stack_alignment_needed); - starting_frame_size = get_frame_size (); set_initial_elim_offsets (); @@ -964,6 +956,20 @@ reload (rtx first, int global) /* If we allocated another stack slot, redo elimination bookkeeping. */ if (starting_frame_size != get_frame_size ()) continue; + if (starting_frame_size && cfun->stack_alignment_needed) + { + /* If we have a stack frame, we must align it now. The + stack size may be a part of the offset computation for + register elimination. So if this changes the stack size, + then repeat the elimination bookkeeping. We don't + realign when there is no stack, as that will cause a + stack frame when none is needed should + STARTING_FRAME_OFFSET not be already aligned to + STACK_BOUNDARY. */ + assign_stack_local (BLKmode, 0, cfun->stack_alignment_needed); + if (starting_frame_size != get_frame_size ()) + continue; + } if (caller_save_needed) { |