[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