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

[Peter Smith via Phabricator via llvm-commits]

peter.smith updated this revision to Diff 244024.
peter.smith added a comment.

Updated to split the logic for the warning from the output. This permits some simplification. Updated tests.

While I'm here it is worth mentioning that I'll soon be no longer using peter.smith at linaro.org. My assignment from Arm to Linaro has ended so I'm back with peter.smith at arm.com (user psmith). This unfortunately means I won't have a lot of, if any, work time to spend on LLD so I'll most likely be slower to respond. I don't intend to disappear, so by all means ping if I'm late on a review.


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

https://reviews.llvm.org/D73542

Files:
  lld/ELF/Arch/ARM.cpp
  lld/test/ELF/Inputs/arm-thumb-abs.s
  lld/test/ELF/arm-thumb-interwork-abs.s
  lld/test/ELF/arm-thumb-interwork-notfunc.s
  lld/test/ELF/arm-thumb-undefined-weak.s
  lld/test/ELF/arm-undefined-weak.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73542.244024.patch
Type: text/x-patch
Size: 15935 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200211/cb811677/attachment.bin>