[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 243462.
peter.smith added a comment.

I've updated the diff to add a warning about no interworking for the cases when the behaviour changes from lld 10.0. I've set the details in a large comment in front of warnOnStateMismatch.

I've added checks for the warning to the existing test, and added a new test that checks the case where there is a non STT_FUNC symbol with the bottom bit set. I expect that case to be much rarer.


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.243462.patch
Type: text/x-patch
Size: 16453 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200209/37545b8b/attachment-0001.bin>