[Mlir-commits] [mlir] Sub-channel quantized type implementation (PR #120172)

[Kevin Gleason]

================
@@ -81,20 +82,31 @@ def UniformQuantizedPerAxisType: DialectType<(type
   }];
 }
 
+def UniformQuantizedSubChannelType
+    : DialectType<(type VarInt:$flags, Type:$storageType, Type:$expressedType,
----------------
GleasonK wrote:

Do we need `$flags`? Still unclear on the benefit of having this in other quantized types.

Maybe cc @stellaraccident has context here? I'm curious if this is a relic of a time before unsigned types, or if there's some nuance to why this is needed to specify signedness when storage type can do the same?

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