[PATCH] D51659: [WebAssembly] v8x16.shuffle

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyInstrSIMD.td:227
+           i32imm_op:$m8, i32imm_op:$m9, i32imm_op:$mA, i32imm_op:$mB,
+           i32imm_op:$mC, i32imm_op:$mD, i32imm_op:$mE, i32imm_op:$mF),
+         (outs),
----------------
Is the size of these values i32? Isn't it `LaneIdx32`, which is a byte with values in the range 0–31? The same question for other places (in some of previous patches too)


Repository:
  rL LLVM

https://reviews.llvm.org/D51659