[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