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 <terry....@arm.com> >>> >>> * gcc.target/arm/combine-movs.c: Check movs for ARM mode >>> and lsrs for other mode. >>> >> >> This can't be right. Thumb1 doesn't use unified syntax. >> >> R. >> > > oops. You are right. Sorry for making such obvious mistake. > Here is patch updated to distinguish ARM and Thumb2. > Tested for Thumb1, Thumb2 and ARM modes. No regression. > > Is it OK? > > BR, > Terry > > 2012-08-21 Terry Guo <terry....@arm.com> > > * gcc.target/arm/combine-movs.c: Check movs for ARM mode > and lsrs for Thumb2 mode. > >
OK. R.
