RE: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-20 Thread Bin Cheng
> -Original Message- > From: Ramana Radhakrishnan [mailto:ramana@googlemail.com] > Sent: Wednesday, November 21, 2012 6:02 AM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target > > On F

Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-20 Thread Ramana Radhakrishnan
On Fri, Nov 16, 2012 at 5:37 AM, Bin Cheng wrote: > Hi, > This patch defines LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target and prefers > short circuit for armv6-m and Thumb2+Os. > > I tested the patch on arm-none-eabi on armv6-m/Thumb2 for both Os/O2. The > patch introduces new fails on ARMv6-m: > >

RE: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-19 Thread Bin Cheng
> -Original Message- > From: Matthew Gretton-Dann [mailto:matthew.gretton-d...@linaro.org] > Sent: Monday, November 19, 2012 8:20 PM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target > > O

Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-19 Thread Matthew Gretton-Dann
On 16 November 2012 12:22, Bin Cheng wrote: > > >> -Original Message- >> From: Matthew Gretton-Dann [mailto:matthew.gretton-d...@linaro.org] >> Sent: Friday, November 16, 2012 6:30 PM >> To: Bin Cheng >> Cc: gcc-patches@gcc.gnu.o

RE: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-16 Thread Bin Cheng
> -Original Message- > From: Matthew Gretton-Dann [mailto:matthew.gretton-d...@linaro.org] > Sent: Friday, November 16, 2012 6:30 PM > To: Bin Cheng > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target > > O

Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target

2012-11-16 Thread Matthew Gretton-Dann
On 16 November 2012 05:37, Bin Cheng wrote: > Hi, > This patch defines LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target and prefers > short circuit for armv6-m and Thumb2+Os. > > === > --- gcc/config/arm/arm.h (revision 193494) > +++ gcc/