[PATCH] D54690: [WebAssembly] Rename function types to signatures

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 25 00:37:01 PST 2018


aheejin abandoned this revision.
aheejin added a comment.

In this comment <https://github.com/WebAssembly/tool-conventions/pull/79#issuecomment-440574050>, Rossberg said

> Drive-by comment: the Wasm spec intentionally calls this type index, not signature index. The reason being that we expect additional forms of type definitions to appear in the type section in the future, see e.g. the GC proposal.

So I guess for the moment it's better to drop this CL to be cautious.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D54690





More information about the llvm-commits mailing list