[PATCH] D105519: [WebAssembly] Deduplicate imports of the same module name, field name, and type

Fri Jul 16 10:44:15 PDT 2021

fitzgen added a comment.

In D105519#2883746 <https://reviews.llvm.org/D105519#2883746>, @sbc100 wrote:

> What do you think about landing https://reviews.llvm.org/D106026 first?  (would avoid the use of `template <typename UndefinedT>` in this patch).

I don't really mind either way.

Obviously it is one tiny rebase less work for me if this lands first, but that's not much work and if that patch is green and ready to land, then I say go for it.

By the way, I am not LLVM committer, so I'll need your help actually landing this patch when the time comes.

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

https://reviews.llvm.org/D105519