[PATCH] D96001: [WebAssembly][lld] Precolor table numbers for MVP inputs

Andy Wingo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 01:51:42 PST 2021


wingo added a comment.

Looking over this changeset, I see that we need some tests.  But also there is a fair amount of code duplication for the "precoloring" logic.  I can make it be a part of TableSymbol::setTableNumber and InputTable::setTableNumber; it would seem that that much duplication is necessary.  But then the errors would probably be less helpful because there's less context at that point.  Thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96001



More information about the llvm-commits mailing list