[PATCH] D53172: [WebAssembly][NFC] Unify ARGUMENT classes

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 15:48:46 PDT 2018


aheejin added a comment.

This makes typenames in the arguments lowercase, which is inconsistent with other existing non-SIMD instructions, but currently other SIMD instructions are anyway, so I guess it would be ok. On a different note, do we need `ARGUMENT` instruction for every variation of v128 vector? If there's only one V128 type register in the backend, wouldn't a single `ARGUMENT_v128` suffice?


Repository:
  rL LLVM

https://reviews.llvm.org/D53172





More information about the llvm-commits mailing list