This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors
- From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- To: Bin Cheng <bin dot cheng at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 23 Apr 2015 09:19:04 +0100
- Subject: Re: [PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors
- Authentication-results: sourceware.org; auth=none
- References: <000801d07cd8$42809e90$c781dbb0$ at arm dot com>
On 22/04/15 09:42, Bin Cheng wrote:
Hi,
Case pr42172-1.c failed on pre-armv7 processors because GCC actually
generates better code without ldr instruction. This patch just refines test
case by checking str instead of ldr, makes sure the case passes on all arm
processors. In the end, we need to fix GCC combiner to generate optimal
code on armv7 processors too. PR42172 is kept open for that purpose.
This is obvious change, is it OK for branches too?
For the record, for -mcpu=arm7tdmi we now generate:
init_A:
mov r3, #8
strb r3, [r0]
bx
Ok for trunk. Is this test failing on the branches too?
Kyrill
gcc/testsuite/ChangeLog
2015-04-22 Bin Cheng <bin.cheng@arm.com>
* gcc.target/arm/pr42172-1.c: Check str instead of ldr.