Re: [Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-09-14 Thread Ulrich Weigand
Terry Guo wrote: > -/* { dg-final { scan-assembler "movs\tr\[0-9\]" } } */ > +/* { dg-final { scan-assembler "lsrs\tr\[0-9\]" { target arm_thumb2_ok } } > } */ > +/* { dg-final { scan-assembler "movs\tr\[0-9\]" { target { ! arm_thumb2_ok > } } } } */ This causes the arm.exp testcase to fail with

RE: [Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-09-03 Thread Terry Guo
> -Original Message- > From: Richard Earnshaw > Sent: Wednesday, August 22, 2012 10:00 PM > To: Terry Guo > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [Patch ARM] Update the test case to differ movs and lsrs > for ARM mode and non-ARM mode > > On 22/0

Re: [Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-08-22 Thread Richard Earnshaw
On 22/08/12 12:16, Terry Guo wrote: > >>> >>> Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS >>> instruction while the ARM mode uses MOVS instruction. So the >> following case >>> is updated accordingly. Is it OK to trunk? >>> >>> BR, >>> Terry >>> >>> 2012-08-21 Terry Guo >>

RE: [Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-08-22 Thread Terry Guo
> > > > Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS > > instruction while the ARM mode uses MOVS instruction. So the > following case > > is updated accordingly. Is it OK to trunk? > > > > BR, > > Terry > > > > 2012-08-21 Terry Guo > > > > * gcc.target/arm/combine-

Re: [Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-08-22 Thread Richard Earnshaw
On 22/08/12 02:20, Terry Guo wrote: > Hi, > > Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS > instruction while the ARM mode uses MOVS instruction. So the following case > is updated accordingly. Is it OK to trunk? > > BR, > Terry > > 2012-08-21 Terry Guo > > * gc

[Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

2012-08-21 Thread Terry Guo
Hi, Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS instruction while the ARM mode uses MOVS instruction. So the following case is updated accordingly. Is it OK to trunk? BR, Terry 2012-08-21 Terry Guo * gcc.target/arm/combine-movs.c: Check movs for ARM mode