[WebAssembly] Add comment as follow-up to rL362276. NFC.
Subscribers: dschuff, jgravelle-google, aheejin, sunfish, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D62824
git-svn-id: https://llvm.org/svn/llvm-project/lld/trunk@362522 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/wasm/SymbolTable.cpp b/wasm/SymbolTable.cpp
index d109527..c4a460f 100644
--- a/wasm/SymbolTable.cpp
+++ b/wasm/SymbolTable.cpp
@@ -199,10 +199,12 @@
Flags, nullptr, Function);
}
+// Adds an optional, linker generated, data symbols. The symbol will only be
+// added if there is an undefine reference to it, or if it is explictly exported
+// via the --export flag. Otherwise we don't add the symbol and return nullptr.
DefinedData *SymbolTable::addOptionalDataSymbol(StringRef Name, uint32_t Value,
uint32_t Flags) {
Symbol *S = find(Name);
- // Enable --export of optional symbols
if (!S && (Config->ExportAll || Config->ExportedSymbols.count(Name) != 0))
S = insertName(Name).first;
else if (!S || S->isDefined())