On 07/23/2014 05:33 PM, Marat Zakirov wrote:
> Hi all!
>
> This is a friendly reminder message.
>
> On 07/17/2014 03:22 PM, Marat Zakirov wrote:
>>
>> On 07/16/2014 01:32 PM, Kyrill Tkachov wrote:
>>>
>>> On 16/07/14 10:22, Marat Zakirov wrote:
>>>> Christophe,
>>>>
>>>> Please look at a new patch.  Draft tests are OK.
>>>> I'll ask your commit approval when full regression (ARM/thumb1/thumb2)
>>>> tests are done.
>>> Hi Marat,
>>>
>>> I was about to propose the thumb2.md hunk myself, but I'll defer to 
>>> the arm maintainers to comment on the other parts.
>>>
>>> Also, in the ChangeLog it is helpful to specify which patterns are 
>>> being affected, so in your case it would be something like:
>>>
>>>     * config/arm/thumb1.md (*thumb1_movhi_insn): Handle stack pointer.
>>>     (*thumb1_movqi_insn): Likewise.
>>>     * config/arm/thumb2.md (*thumb2_movhi_insn): Ditto.
>>>
>>>
>>> Kyrill
>>>
>>>
>> Christophe, Kirill,
>>
>> finally I've finished regression testing.
>> Please check if my patch is OK for trunk.
>>
>> The following configures were used:
>>
>> configure --build=x86_64-pc-linux-gnu --host=x86_64-pc-linux-gnu 
>> --target=arm-linux-gnueabi  --with-interwork --enable-long-long 
>> --enable-languages=c,c++,fortran --enable-shared --with-gnu-as 
>> --with-gnu-ld --with-arch=$ARCH --with-mode=$MODE
>>
>> Thumb-1
>>
>> $ARCH=armv4t
>> $MODE=thumb
>>
>> Thumb-2
>>
>> $ARCH=armv7
>> $MODE=thumb
>>
>> ARM
>>
>> $ARCH=armv7-a
>> $MODE=arm
>>
>> No regressions detected, test pr61561.c passed in all cases.
>>
>> Thank you all.
>> --Marat
>
>