This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 0/8] [ARC] Several fixes for ARC linker


Pushed all of the approved patches.

Nick: Thanks for the review and comments.

Best regards,
Cupertino

On 05/23/2017 05:41 PM, Cupertino Miranda wrote:
> Hi,
>
> Please find attach a series of patches fixing several identified issues for
> ARC linker.
>
>
> [PATCH 1/9] [ARC] Avoid creating dynamic relocs when static linked is
> [PATCH 2/9] [ARC] Disable warning on absolute relocs when symbol is
> [PATCH 3/9] [ARC] Reassign the symbol got information to actual
>
>   FAIL -> UNSUPPORTED: Run pr18718 with PIE (1)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIE (2)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIE (3)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIE (4)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIC (1)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIC (2)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIC (3)
>   FAIL -> UNSUPPORTED: Run pr18718 with PIC (4)
>
> [PATCH 4/9] [ARC] Don't convert _DYNAMIC@ to _GLOBAL_OFFSET_TABLE_
> [PATCH 5/9] [ARC] ARC_PC32 dynamic reloc incorrectly generated.
> [PATCH 6/9] [ARC] Add RELOC_FOR_GLOBAL_SYMBOL in ARC target code.
>
>   PASS -> FAIL: plugin claimfile lost symbol
>   PASS -> FAIL: plugin claimfile replace symbol
>   PASS -> FAIL: plugin claimfile resolve symbol
>   PASS -> FAIL: plugin claimfile lost symbol with source
>   PASS -> FAIL: plugin claimfile replace symbol with source
>   PASS -> FAIL: plugin claimfile resolve symbol with source
>   PASS -> FAIL: plugin 2 with source lib
>   PASS -> FAIL: load plugin 2 with source
>   PASS -> FAIL: plugin 3 with source lib
>   PASS -> FAIL: load plugin 3 with source
>
>   This is the case because after adding RELOC_FOR_GLOBAL_SYMBOL, the linker
>   became more meticulous identifying undefined symbols.
>   After breafly checking the logs, I noticed that our target fails to pass those
>   tests as it aborts with a undefined symbol message. ARM for example reports
>   the same undefined symbol, however it does not abort the linker.
>
> [PATCH 7/9] [ARC] Fixed TLS for IE model.
> [PATCH 8/9] [ARC] Fixed condition to generate TEXTREL.
>
>   FAIL -> UNSUPPORTED: Run pr19579
>   FAIL -> UNSUPPORTED: Run pr19579 (-z now)
>
> [PATCH 9/9] [ARC] Corrected conditions for dynamic sections creation.
>
> Looking forward to your review.
>
> Cheers,
> Cupertino
>
>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]