[PATCH, GAS/ARM, ping] Fix expansion of ldr pseudo instruction
Thomas Preudhomme
thomas.preudhomme@foss.arm.com
Tue Jun 20 09:03:00 GMT 2017
Ping Tristan?
Best regards,
Thomas
On 14/06/17 10:25, Thomas Preudhomme wrote:
>
>
> On 14/06/17 10:11, Ramana Radhakrishnan wrote:
>> On Wed, Jun 14, 2017 at 9:49 AM, Thomas Preudhomme
>> <thomas.preudhomme@foss.arm.com> wrote:
>>> Patch applies cleanly on binutils-2_28-branch and shows no testsuite
>>> regression. Is this ok to commit to binutils 2.28?
>>>
>>
>> can you mark this as fixing PR21590 in your changelog for 2.28 ?
>
> Sure, please ignore the ChangeLog in the patch, this is coming from the
> cherry-pick. The proposed ChangeLog would be:
>
> 2017-06-14 Thomas Preud'homme <thomas.preudhomme@arm.com>
>
> Backport from mainline
> 2017-04-24 Thomas Preud'homme <thomas.preudhomme@arm.com>
>
> PR 21590
> * config/tc-arm.c (move_or_literal_pool): Remove code generating MOVS.
> Forbid MOV.W and MOVW if destination is SP or PC.
> * testsuite/gas/arm/thumb2_ldr_immediate_highregs_armv6t2.s: Explain
> expectation of LDR not generating a MOVS for low registers and small
> constants. Add tests of MOVW generation.
> * testsuite/gas/arm/thumb2_ldr_immediate_highregs_armv6t2.d: Update
> expected disassembly.
>
>>
>> However you need an ACK from Tristan about 2.28.
>
> Yes indeed, my mistake.
>
>>
>> Also could you please note the sha1 which fixed this on trunk on the
>> bz for posterity ?
>
> Done.
>
> Best regards,
>
> Thomas
>
>>
>> regards
>> Ramana
>>
>>
>>> Best regards,
>>>
>>> Thomas
>>>
>>>
>>> On 24/04/17 14:07, Nick Clifton wrote:
>>>>
>>>> Hi Thomas,
>>>>
>>>>> 2017-04-20 Thomas Preud'homme <thomas.preudhomme@arm.com>
>>>>>
>>>>> * config/tc-arm.c (move_or_literal_pool): Remove code generating
>>>>> MOVS.
>>>>> Forbid MOV.W and MOVW if destination is SP or PC.
>>>>> * testsuite/gas/arm/thumb2_ldr_immediate_highregs_armv6t2.s: Explain
>>>>> expectation of LDR not generating a MOVS for low registers and small
>>>>> constants. Add tests of MOVW generation.
>>>>> * testsuite/gas/arm/thumb2_ldr_immediate_highregs_armv6t2.d: Update
>>>>> expected disassembly.
>>>>
>>>>
>>>> Approved - please apply.
>>>>
>>>> Cheers
>>>> Nick
>>>>
>>>
More information about the Binutils
mailing list