[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
Fri Feb 7 16:58:58 PST 2020


MaskRay added a comment.

The test change did not need to be deleted by c29003813ab9bd6ea7b6de40ea8f1fe21979f13f <https://reviews.llvm.org/rGc29003813ab9bd6ea7b6de40ea8f1fe21979f13f>. I added it back in f08099b9c26be463aca096f73e9863921dddf792 <https://reviews.llvm.org/rGf08099b9c26be463aca096f73e9863921dddf792>.

Do you mean that lld's behavior is correct and the patch should reland, i.e. revert c29003813ab9bd6ea7b6de40ea8f1fe21979f13f <https://reviews.llvm.org/rGc29003813ab9bd6ea7b6de40ea8f1fe21979f13f> ?


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