From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21241 invoked by alias); 16 Nov 2011 08:24:04 -0000 Received: (qmail 21226 invoked by uid 22791); 16 Nov 2011 08:24:02 -0000 X-SWARE-Spam-Status: No, hits=-0.4 required=5.0 tests=AWL,BAYES_00,RDNS_DYNAMIC,T_FILL_THIS_FORM_SHORT X-Spam-Check-By: sourceware.org Received: from host-82-135-62-35.customer.m-online.net (HELO mail.embedded-brains.de) (82.135.62.35) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 16 Nov 2011 08:23:41 +0000 Received: by mail.embedded-brains.de (Postfix, from userid 65534) id D897F65261D; Wed, 16 Nov 2011 09:23:38 +0100 (CET) Received: from [192.168.96.31] (eb0011.eb.z [192.168.96.31]) by mail.embedded-brains.de (Postfix) with ESMTP id B445C65261B for ; Wed, 16 Nov 2011 09:23:35 +0100 (CET) Message-ID: <4EC37307.6030206@embedded-brains.de> Date: Wed, 16 Nov 2011 09:59:00 -0000 From: Sebastian Huber User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.23) Gecko/20110920 SUSE/3.1.15 Lightning/1.0b2 Thunderbird/3.1.15 MIME-Version: 1.0 To: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, ARM] Fix stack red zone bug (PR38644) for GCC 4.6 References: <000e01cc9907$01ce9b80$056bd280$@liu@arm.com> In-Reply-To: <000e01cc9907$01ce9b80$056bd280$@liu@arm.com> Content-Type: multipart/mixed; boundary="------------050609090301080803020200" X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-11/txt/msg01763.txt.bz2 This is a multi-part message in MIME format. --------------050609090301080803020200 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Content-length: 549 Hi, this is Jiangning Liu's patch to fix PR38644 in ARM back-end http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38644. intended for the GCC 4.6 branch. Test results: http://gcc.gnu.org/ml/gcc-testresults/2011-11/msg01619.html -- Sebastian Huber, embedded brains GmbH Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany Phone : +49 89 18 90 80 79-6 Fax : +49 89 18 90 80 79-9 E-Mail : sebastian.huber@embedded-brains.de PGP : Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG. --------------050609090301080803020200 Content-Type: text/x-patch; name="pr38644-4.6.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="pr38644-4.6.patch" Content-length: 1342 ChangeLog: 2011-11-04 Jiangning Liu PR rtl-optimization/38644 * config/arm/arm.c (thumb1_expand_epilogue): Add memory barrier for epilogue having stack adjustment. ChangeLog of testsuite: 2011-11-04 Jiangning Liu PR rtl-optimization/38644 * gcc.target/arm/stack-red-zone.c: New. diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index 75174a3..acdfbd5 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -20864,6 +20864,8 @@ thumb1_expand_epilogue (void) gcc_assert (amount >= 0); if (amount) { + emit_insn (gen_blockage ()); + if (amount < 512) emit_insn (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx, GEN_INT (amount))); diff --git a/gcc/testsuite/gcc.target/arm/stack-red-zone.c b/gcc/testsuite/gcc.target/arm/stack-red-zone.c new file mode 100644 index 0000000..b9f0f99 --- /dev/null +++ b/gcc/testsuite/gcc.target/arm/stack-red-zone.c @@ -0,0 +1,12 @@ +/* No stack red zone. PR38644. */ +/* { dg-options "-mthumb -O2" } */ +/* { dg-final { scan-assembler "ldrb\[^\n\]*\\n\[\t \]*add\[\t \]*sp" } } */ + +extern int doStreamReadBlock (int *, char *, int size, int); + +char readStream (int *s) +{ + char c = 0; + doStreamReadBlock (s, &c, 1, *s); + return c; +} --------------050609090301080803020200--