Re: [PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors

2015-04-23 Thread Kyrill Tkachov
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

Re: [PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors

2015-04-23 Thread Bin.Cheng
On Thu, Apr 23, 2015 at 4:19 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: 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

Re: [PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors

2015-04-23 Thread Kyrill Tkachov
On 23/04/15 09:53, Bin.Cheng wrote: On Thu, Apr 23, 2015 at 4:19 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: 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

[PATCH ARM]Fix pr42172-1.c failure on pre armv7 processors

2015-04-22 Thread Bin Cheng
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