[PATCH] D79114: [lld-macho] Dylib symbols should always replace undefined symbols

[Jez Ng via Phabricator via llvm-commits]

int3 marked an inline comment as done.
int3 added a comment.

> where can we find authoritative symbol resolution rules?

I guess the authoritative source is whatever ld64 is doing. I took a brief look at their SymbolTable.cpp but it seems rather complex, handling many kinds of symbols. The test behavior in this diff does emulate the observed behavior of ld64 though.

Thanks for the pointer to the ELF resolution rule design, will keep that in mind as we expand support for symbol kinds.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79114