[PATCH] D42117: [WebAssembly] Symbol changes #6: syminfo index, LLVM

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 10:43:42 PST 2018


ncw created this revision.
ncw added a reviewer: sbc100.
Herald added subscribers: llvm-commits, sunfish, aheejin, jgravelle-google, dschuff, jfb.

Sixth chunk split out of https://reviews.llvm.org/D41954.

Must be committed simultaneously with LLD change, DXXX.

Changes:
--------

- Change the WASM_SYMBOL_INFO linking metadata to refer to symbols by index rather than by string name
- This should reduce the size of the Wasm object files. Hopefully uncontroversial?


Repository:
  rL LLVM

https://reviews.llvm.org/D42117

Files:
  include/llvm/Object/Wasm.h
  include/llvm/ObjectYAML/WasmYAML.h
  lib/MC/WasmObjectWriter.cpp
  lib/Object/WasmObjectFile.cpp
  lib/ObjectYAML/WasmYAML.cpp
  test/MC/WebAssembly/array-fill.ll
  test/MC/WebAssembly/comdat.ll
  test/MC/WebAssembly/global-ctor-dtor.ll
  test/MC/WebAssembly/unnamed-data.ll
  test/MC/WebAssembly/visibility.ll
  test/MC/WebAssembly/weak-alias.ll
  test/MC/WebAssembly/weak.ll
  test/ObjectYAML/wasm/linking_section.yaml
  test/ObjectYAML/wasm/weak_symbols.yaml
  test/tools/llvm-nm/wasm/weak-symbols.yaml
  tools/obj2yaml/wasm2yaml.cpp
  tools/yaml2obj/yaml2wasm.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42117.129991.patch
Type: text/x-patch
Size: 20541 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180116/d37f515a/attachment.bin>


More information about the llvm-commits mailing list