[PATCH] D42076: [WebAssembly] Symbol changes #1, LLD

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 10:54:09 PST 2018


sbc100 added a comment.

Landed two smaller parts as: https://reviews.llvm.org/rLLD322726 and https://reviews.llvm.org/rLLD322727



================
Comment at: wasm/Writer.cpp:472
 void Writer::createNameSection() {
-  // Create an array of all function sorted by function index space
-  std::vector<const Symbol *> Names;
+  unsigned Names = ImportedFunctions.size();
+  for (const InputFunction *F : DefinedFunctions)
----------------
Maybe `NumNames`?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42076





More information about the llvm-commits mailing list