[PATCH] D55613: [WebAssembly] Add "needed" list to dylink section

[Phabricator via Phabricator via llvm-commits]

This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rL348990: [WebAssembly] Add "needed" list to dylink section (authored by sbc, committed by ).

Repository:
  rL LLVM

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

https://reviews.llvm.org/D55613

Files:
  lld/trunk/wasm/Writer.cpp


Index: lld/trunk/wasm/Writer.cpp
===================================================================
--- lld/trunk/wasm/Writer.cpp
+++ lld/trunk/wasm/Writer.cpp
@@ -480,6 +480,7 @@
   writeUleb128(OS, int(log2(MemAlign)), "MemAlign");
   writeUleb128(OS, IndirectFunctions.size(), "TableSize");
   writeUleb128(OS, 0, "TableAlign");
+  writeUleb128(OS, 0, "Needed");  // TODO: Support "needed" shared libraries
 }
 
 // Create the custom "linking" section containing linker metadata.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55613.177962.patch
Type: text/x-patch
Size: 484 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181212/65ff11c0/attachment.bin>