[PATCH] D41975: [WebAssembly] Don't allow functions to be named more than once

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 12 06:51:14 PST 2018


ncw accepted this revision.
ncw added a comment.

Fine by me then!

Although the names section can name imports (oddly...) I don't think clang does this when it writes out object files; I don't see any reason for LLD to do that either. Imports are already named in a canonical way by the import name! I've preserved the behaviour just for the time being in my changes.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41975





More information about the llvm-commits mailing list