[PATCH] D73542: [LLD][ELF][ARM] Do not substitute BL/BLX for non STT_FUNC symbols.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 5 14:16:40 PST 2020


MaskRay added a comment.

In D73542#1860617 <https://reviews.llvm.org/D73542#1860617>, @psmith wrote:

> In D73542#1860602 <https://reviews.llvm.org/D73542#1860602>, @MaskRay wrote:
>
> > I am also investigating it, calls to the undefined weak getauxval need to use BLX. I'll try fixing it.
>
>
> Thanks. It should just be a matter of changing rel.type to rel.expr on line 457 and line 412.


Fixed by 5461fa2b1fcfcfcd8e28e3ac3383d2245d5d90bf <https://reviews.llvm.org/rG5461fa2b1fcfcfcd8e28e3ac3383d2245d5d90bf> . R_ARM_THM_CALL tests were added to `arm-thumb-interwork-shared.s`

There is no R_ARM_CALL test.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73542/new/

https://reviews.llvm.org/D73542





More information about the llvm-commits mailing list