[Mlir-commits] [mlir] [MLIR][WasmSSA] Instruction parser refactoring of WasmSSA importer (PR #195500)

[Luc Forget]

================
@@ -237,6 +237,15 @@ class ValueStack {
 
 using local_val_t = TypedValue<wasmssa::LocalRefType>;
 
+template <typename TargetType, size_t... IS>
+constexpr std::integer_sequence<TargetType, TargetType{IS}...>
+    castIndexSequence(std::index_sequence<IS...>) {
+  return {};
+}
+
+constexpr auto all8bitsBytes =
+    castIndexSequence<std::byte>(std::make_index_sequence<256>());
----------------
lforg37 wrote:

I have a fix using the `ByteSequence` type which is already defined in that module. I'll push later today, as I can't test it at the moment. I think it is actually conform to the specs to reject `std::byte` in `std::integer_sequence` as it is not an integral type.  

https://github.com/llvm/llvm-project/pull/195500