[PATCH] D111086: [WebAssembly] Remove WasmTagType

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 5 17:10:28 PDT 2021


aheejin added a comment.

In D111086#3044244 <https://reviews.llvm.org/D111086#3044244>, @sbc100 wrote:

> Not sure if the description needs updating after the simplification?

The original comment didn't say anything about how we handled `SigIndex`, so the description is still all valid. I added one sentence:

> This instead moves `SigIndex` to `struct WasmTag` itself, as we did for `struct WasmFunction` in D111104 <https://reviews.llvm.org/D111104>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111086



More information about the llvm-commits mailing list