[PATCH] D154941: [mlir][ArmSME] Add custom get_tile_id and cast ops

[Frank (Fang) Gao via Phabricator via llvm-commits]

WanderAway added a comment.

LGTM % a few minor details. Great work!



================
Comment at: mlir/include/mlir/Dialect/ArmSME/IR/ArmSME.td:54
+def nxnxv2i64  : SMETileType<I64,  [2,  2 ], "vector<[2]x[2]xi64">;
+def nxnxv1i128 : SMETileType<I128, [1,  1 ], "vector<[1]x[1]xi128">;
+
----------------
What about `f32` and `f64`?


================
Comment at: mlir/include/mlir/Dialect/ArmSME/IR/ArmSME.td:188
+    ```
+  }];
+
----------------
It may be useful to also indicate what is the intended live range of the allocated tile?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154941