[PATCH] D44313: [WebAssembly] Implement GC for imports

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

Similarity to or disparity from ELF isn't that important, but perhaps yes, though I'm not 100% sure if it's the best way. "Used" is probably a bit ambiguous, so I'd rename it "Referenced" or something.

Also, I think that the current way of representing an imported symbol doesn't hit the sweet spot. IIUC, undefined symbols are used to represent imported symbols, but I believe we should have a dedicated symbol type for imported symbols. After resolving and before writing, we can read all .import files, find all symbols referenced by the files, and replace them with some kind of imported symbol type, so that we can handle undefined symbols as really undefined.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44313