[PATCH] D80169: [lld-macho] Error on encountering undefined symbols

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 14:18:53 PDT 2020


smeenai accepted this revision.
smeenai added a comment.
This revision is now accepted and ready to land.

LGTM, thanks for fixing!

LLD ELF does this as part of its `scanRelocations` loop. That might be nicer cos we'd bail out with undefined symbol errors earlier. It doesn't matter a whole lot right now, but we could add a TODO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80169





More information about the llvm-commits mailing list