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

Richard Earnshaw rearnsha@arm.com
Wed Aug 22 14:16:00 GMT 2012


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.guo@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.guo@arm.com>
> 
>         * gcc.target/arm/combine-movs.c: Check movs for ARM mode 
>         and lsrs for Thumb2 mode.
> 
> 

OK.

R.



More information about the Gcc-patches mailing list