[PATCH] D139764: [COFF] Respect weak externals for mangled symbol searching
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 11 00:59:07 PST 2022
mstorsjo added inline comments.
================
Comment at: lld/COFF/SymbolTable.cpp:817
+ // the weakAlias member variable. This matches link.exe's behavior.
+ if (Symbol *weakAlias = u->getWeakAlias())
+ return weakAlias;
----------------
What this symbol is non-null but is `Undefined` (with itself possibly might be a weak symbol)? Do we need to make this a loop which inspects that aspect? Or is it not relevant for the usecases of `findMangle` where it is enough to see that we apparently did hit a real symbol and don’t need to check for prefixes?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139764/new/
https://reviews.llvm.org/D139764
More information about the llvm-commits
mailing list