[PATCH] D59074: [WebAssembly] Fix build after rL355577

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 20:18:01 PST 2019


sbc100 created this revision.
Herald added subscribers: llvm-commits, sunfish, aheejin, jgravelle-google, dschuff.
Herald added a project: LLVM.

Turns own that IsUsedInRegularObject is set for lazy (archive) symbols.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D59074

Files:
  lld/wasm/Writer.cpp


Index: lld/wasm/Writer.cpp
===================================================================
--- lld/wasm/Writer.cpp
+++ lld/wasm/Writer.cpp
@@ -950,7 +950,7 @@
   };
 
   for (Symbol *Sym : Symtab->getSymbols())
-    if (Sym->IsUsedInRegularObj)
+    if (!Sym->isLazy() && Sym->IsUsedInRegularObj)
       AddSymbol(Sym);
 
   for (ObjFile *File : Symtab->ObjectFiles) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59074.189645.patch
Type: text/x-patch
Size: 373 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190307/86b0a77f/attachment.bin>


More information about the llvm-commits mailing list