Patchwork [ARM] Cleanup in arm_expand_epilogue

login
register
mail settings
Submitter Greta Yorsh
Date July 20, 2012, 6:33 p.m.
Message ID <000c01cd66a6$2ee2c430$8ca84c90$@Yorsh@arm.com>
Download mbox | patch
Permalink /patch/172324/
State New
Headers show

Comments

Greta Yorsh - July 20, 2012, 6:33 p.m.
The variable floats_from_frame in function arm_expand_epilogue became unused
after removal of FPA support. This patch cleans it up and simplifies the
initialization of num_regs variable.

Ok for trunk?

Thanks,
Greta

ChangeLog

gcc/

2012-07-20  Greta Yorsh  <Greta.Yorsh@arm.com>

        * config/arm/arm.c (arm_expand_epilogue): Remove unused variable
        floats_from_frame.

Patch

diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 627b436..659d6b3 100644
--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -23006,8 +23015,7 @@  arm_expand_epilogue (bool really_return)
   int num_regs = 0;
   int i;
   int amount;
-  int floats_from_frame = 0;
   arm_stack_offsets *offsets;
 
   func_type = arm_current_func_type ();
 
@@ -23033,18 +23042,7 @@  arm_expand_epilogue (bool really_return)
   /* Get frame offsets for ARM.  */
   offsets = arm_get_frame_offsets ();
   saved_regs_mask = offsets->saved_regs_mask;
-
-  /* Find offset of floating point register from frame pointer.
-     The initialization is done in this way to take care of frame pointer
-     and static-chain register, if stored.  */
-  floats_from_frame = offsets->saved_args - offsets->frame;
-  /* Compute how many registers saved and how far away the floats will be.  */
-  for (i = 0; i <= LAST_ARM_REGNUM; i++)
-    if (saved_regs_mask & (1 << i))
-      {
-        num_regs++;
-        floats_from_frame += 4;
-      }
+  num_regs = bit_count (saved_regs_mask);
 
   if (frame_pointer_needed)
     {