[Mlir-commits] [mlir] [mlir][spirv] Make `CooperativeMatrixType` a `ShapedType` (PR #142784)

[Jakub Kuderski]

================
@@ -195,7 +195,7 @@ std::optional<int64_t> CompositeType::getSizeInBytes() {
 
 struct spirv::detail::CooperativeMatrixTypeStorage final : TypeStorage {
   using KeyTy =
-      std::tuple<Type, uint32_t, uint32_t, Scope, CooperativeMatrixUseKHR>;
+      std::tuple<Type, int64_t, int64_t, Scope, CooperativeMatrixUseKHR>;
----------------
kuhar wrote:

IIRC, the original motivation for uint32 was to match the type definition from the spirv spec and make (de)serialization easy. Would it be worth adding a comment that we diverge from the spec here and explain the motivation?

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