From patchwork Fri Aug 22 22:05:31 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Henderson X-Patchwork-Id: 382363 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 0E0E714007B for ; Sat, 23 Aug 2014 08:01:24 +1000 (EST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:cc:subject:date:message-id:in-reply-to:references; q=dns; s= default; b=OWFnyAnteQfnTkVK1Drimot/e3VKaIg/VnBCYAycQT8rMWcc/OmzQ M8UmXzRngagodbph1TZusprwBLlWJcfCe7t4+9yH/UjDmwYtWq3gbjKqW2v9RXnr 5eRhJrOWp/kV0ORlZzg+Hr05Z+gcouH+guSEis4ZEGezEPCb7S4Mes= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:cc:subject:date:message-id:in-reply-to:references; s= default; bh=klwwihDcZeV5QPcq4NZgNfow3Ek=; b=Gzx1fUw//jQwT+8oYkl3 o203DlBwtOVjTuHl8aTjVJWwgLLrUGLOj0QfPaWJrUcR5i2YN0l7J3YHxMfwIxQf r4FUZnPhLjTWdxz0FUPZlicxqmCIwJvbM0llYFCBdXl/BupSR3Ge4EoC/5HwQ3su qgta4m3zzSkU0SNqtRRVuEI= Received: (qmail 23242 invoked by alias); 22 Aug 2014 22:00:24 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 23115 invoked by uid 89); 22 Aug 2014 22:00:23 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=BAYES_00, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW, SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-qg0-f50.google.com Received: from mail-qg0-f50.google.com (HELO mail-qg0-f50.google.com) (209.85.192.50) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Fri, 22 Aug 2014 22:00:22 +0000 Received: by mail-qg0-f50.google.com with SMTP id z107so7436052qgd.37 for ; Fri, 22 Aug 2014 15:00:20 -0700 (PDT) X-Received: by 10.140.84.21 with SMTP id k21mr11506735qgd.70.1408744820405; Fri, 22 Aug 2014 15:00:20 -0700 (PDT) Received: from localhost.localdomain (50-194-63-110-static.hfc.comcastbusiness.net. [50.194.63.110]) by mx.google.com with ESMTPSA id q9sm57791221qat.32.2014.08.22.15.00.18 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 22 Aug 2014 15:00:19 -0700 (PDT) From: Richard Henderson To: gcc-patches@gcc.gnu.org Cc: marcus.shawcroft@arm.com, richard.earnshaw@arm.com Subject: [PATCH 4/4] aarch64: Don't duplicate calls_alloca check Date: Fri, 22 Aug 2014 15:05:31 -0700 Message-Id: <1408745131-24297-5-git-send-email-rth@redhat.com> In-Reply-To: <1408745131-24297-1-git-send-email-rth@redhat.com> References: <1408745131-24297-1-git-send-email-rth@redhat.com> X-IsSubscribed: yes 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(-) 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