[GAS][ARM]Positively emit symbols for alignment. --> [PATCH][LD][ARM] Fix two test case failures.
Richard Earnshaw
Richard.Earnshaw@foss.arm.com
Wed Apr 29 09:23:00 GMT 2015
On 29/04/15 10:21, Renlin Li wrote:
> Hi Hans-Peter,
>
> Thank you for your report! I have attached the patch to fix the test cases.
>
> Previously, the padding bytes are treated as instruction, because data
> mapping symbols are missing. With the new patch, data mapping symbols
> are emitted, the bytes thereafter are correctly interpreted as data.
>
> Okay to commit?
>
OK.
R.
> Regards,
> Renlin Li
>
> ld/testsuite/ChangeLog:
>
> 2015-04-29 Renlin Li <renlin.li@arm.com>
>
> * ld-arm/ifunc-10.dd: Adjust expected output.
> * ld-arm/ifunc-2.dd: Likewise.
>
> On 28/04/15 22:07, Hans-Peter Nilsson wrote:
>> Maybe you refer to some other target (which one?) than
>> arm-unknown-eabi, because for arm-unknown-eabi, this patch
>> caused failures in the ld testsuite:
>>
>> Running /tmp/hpautotest-binutils/bsrc/src/ld/testsuite/ld-arm/arm-elf.exp ...
>> FAIL: IFUNC test 2
>> FAIL: IFUNC test 10
>>
>> I'm guessing you just need to adjust the
>> expected-result-templates for the "mapping symbols".
>>
>> brgds, H-P
>>
>>
>> ld.diff
>>
>>
>> diff --git a/ld/testsuite/ld-arm/ifunc-10.dd b/ld/testsuite/ld-arm/ifunc-10.dd
>> index 88bae50..d96c086 100644
>> --- a/ld/testsuite/ld-arm/ifunc-10.dd
>> +++ b/ld/testsuite/ld-arm/ifunc-10.dd
>> @@ -267,7 +267,7 @@ Disassembly of section \.text:
>>
>> 0000a010 <tbf1>:
>> a010: 46f7 mov pc, lr
>> - a012: 0000 movs r0, r0
>> + a012: 0000 .short 0x0000
>> a014: e1a0f00e mov pc, lr
>> a018: e1a0f00e mov pc, lr
>> a01c: e1a0f00e mov pc, lr
>> diff --git a/ld/testsuite/ld-arm/ifunc-2.dd b/ld/testsuite/ld-arm/ifunc-2.dd
>> index f6d57fa..91eab54 100644
>> --- a/ld/testsuite/ld-arm/ifunc-2.dd
>> +++ b/ld/testsuite/ld-arm/ifunc-2.dd
>> @@ -84,10 +84,10 @@ Disassembly of section \.text:
>>
>> 0000a00c <f4>:
>> a00c: 46f7 mov pc, lr
>> - a00e: 0000 movs r0, r0
>> + a00e: 0000 .short 0x0000
>> a010: e1a0f00e mov pc, lr
>> a014: 46f7 mov pc, lr
>> - a016: 0000 movs r0, r0
>> + a016: 0000 .short 0x0000
>> a018: e1a0f00e mov pc, lr
>> a01c: 46f7 mov pc, lr
>> \.\.\.
More information about the Binutils
mailing list