diff mbox

[4/4] aarch64: Don't duplicate calls_alloca check

Message ID 1408745131-24297-5-git-send-email-rth@redhat.com
State New
Headers show

Commit Message

Richard Henderson Aug. 22, 2014, 10:05 p.m. UTC 
Generic code already handles calls_alloca for determining
the need for a frame pointer.

	* config/aarch64/aarch64.c (aarch64_frame_pointer_required): Don't
	check calls_alloca.
---
 gcc/config/aarch64/aarch64.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Marcus Shawcroft Sept. 3, 2014, 11:06 a.m. UTC | #1 
On 22 August 2014 23:05, Richard Henderson <rth@redhat.com> wrote:
> Generic code already handles calls_alloca for determining
> the need for a frame pointer.
>
>         * config/aarch64/aarch64.c (aarch64_frame_pointer_required): Don't
>         check calls_alloca.

Ok Thanks /Marcus
Richard Henderson Sept. 3, 2014, 5:02 p.m. UTC | #2 
On 09/03/2014 04:06 AM, Marcus Shawcroft wrote:
> On 22 August 2014 23:05, Richard Henderson <rth@redhat.com> wrote:
>> Generic code already handles calls_alloca for determining
>> the need for a frame pointer.
>>
>>         * config/aarch64/aarch64.c (aarch64_frame_pointer_required): Don't
>>         check calls_alloca.
> 
> Ok Thanks /Marcus
> 

Thanks.

I fixed up the conflicts with the rtx_insn patch set and committed
all four patches squashed together.


r~
diff mbox

Patch

diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index c890773..b80f283 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -1805,11 +1805,6 @@  aarch64_libgcc_cmp_return_mode (void)
 static bool
 aarch64_frame_pointer_required (void)
 {
-  /* If the function contains dynamic stack allocations, we need to
-     use the frame pointer to access the static parts of the frame.  */
-  if (cfun->calls_alloca)
-    return true;
-
   /* In aarch64_override_options_after_change
      flag_omit_leaf_frame_pointer turns off the frame pointer by
      default.  Turn it back on now if we've not got a leaf