OpenE2K/gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Simplify logic saving the int registers in the prologue.

Browse Source 
From-SVN: r162884
This commit is contained in:
Richard Henderson 2010-08-04 16:57:56 -07:00 committed by Richard Henderson
parent 9bf40564e1
commit cdcb5f261e
2 changed files with 20 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
gcc/ChangeLog
View File

@ -1,5 +1,8 @@
2010-08-04 Richard Henderson <rth@redhat.com> 2010-08-04 Richard Henderson <rth@redhat.com>
* config/i386/i386.c (ix86_expand_prologue): Simplify logic
saving the int registers.
* dwarf2out.c: Remove most of the DWARF2_DEBUGGING_INFO * dwarf2out.c: Remove most of the DWARF2_DEBUGGING_INFO
and DWARF2_UNWIND_INFO conditional compilation. and DWARF2_UNWIND_INFO conditional compilation.

16
gcc/config/i386/i386.c
View File

@ -9194,6 +9194,7 @@ ix86_expand_prologue (void)
struct ix86_frame frame; struct ix86_frame frame;
HOST_WIDE_INT allocate; HOST_WIDE_INT allocate;
int gen_frame_pointer = frame_pointer_needed; int gen_frame_pointer = frame_pointer_needed;
bool int_registers_saved = false;
ix86_finalize_stack_realign_flags (); ix86_finalize_stack_realign_flags ();
@ -9347,7 +9348,10 @@ ix86_expand_prologue (void)
allocate = frame.to_allocate + frame.nsseregs * 16 + frame.padding0; allocate = frame.to_allocate + frame.nsseregs * 16 + frame.padding0;
if (!frame.save_regs_using_mov) if (!frame.save_regs_using_mov)
{
ix86_emit_save_regs (); ix86_emit_save_regs ();
int_registers_saved = true;
}
else else
allocate += frame.nregs * UNITS_PER_WORD; allocate += frame.nregs * UNITS_PER_WORD;
@ -9356,7 +9360,7 @@ ix86_expand_prologue (void)
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK) if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK)
{ {
/* We expect the registers to be saved when probes are used. */ /* We expect the registers to be saved when probes are used. */
gcc_assert (!frame.save_regs_using_mov); gcc_assert (int_registers_saved);
if (STACK_CHECK_MOVING_SP) if (STACK_CHECK_MOVING_SP)
{ {
@ -9382,13 +9386,17 @@ ix86_expand_prologue (void)
avoid doing this if I am going to have to probe the stack since avoid doing this if I am going to have to probe the stack since
at least on x86_64 the stack probe can turn into a call that clobbers at least on x86_64 the stack probe can turn into a call that clobbers
a red zone location */ a red zone location */
if (!TARGET_64BIT_MS_ABI && TARGET_RED_ZONE && frame.save_regs_using_mov if (!int_registers_saved
&& (TARGET_RED_ZONE && !TARGET_64BIT_MS_ABI)
&& (! TARGET_STACK_PROBE || allocate < CHECK_STACK_LIMIT)) && (! TARGET_STACK_PROBE || allocate < CHECK_STACK_LIMIT))
{
ix86_emit_save_regs_using_mov ((frame_pointer_needed ix86_emit_save_regs_using_mov ((frame_pointer_needed
&& !crtl->stack_realign_needed) && !crtl->stack_realign_needed)
? hard_frame_pointer_rtx ? hard_frame_pointer_rtx
: stack_pointer_rtx, : stack_pointer_rtx,
-frame.nregs * UNITS_PER_WORD); -frame.nregs * UNITS_PER_WORD);
int_registers_saved = true;
}
if (allocate == 0) if (allocate == 0)
; ;
@ -9439,9 +9447,7 @@ ix86_expand_prologue (void)
} }
} }
if (frame.save_regs_using_mov if (!int_registers_saved)
&& !(!TARGET_64BIT_MS_ABI && TARGET_RED_ZONE
&& (! TARGET_STACK_PROBE || allocate < CHECK_STACK_LIMIT)))
{ {
if (!frame_pointer_needed if (!frame_pointer_needed
|| !(frame.to_allocate + frame.padding0) || !(frame.to_allocate + frame.padding0)