[Mlir-commits] [mlir] QuantileType relax quantileType conditions and inheritance issue (PR #204793)

[llvmlistbot at llvm.org]

================
@@ -41,7 +41,8 @@ unsigned QuantizedType::getFlags() const {
 }
 
 bool QuantizedType::classof(Type type) {
-  return llvm::isa<QuantDialect>(type.getDialect());
+  return llvm::isa<QuantDialect>(type.getDialect()) &&
----------------
vsimion26 wrote:

hmm I think we do. My understanding is that from this check, everything that is registered under QuantDialect can be treated as a QuantizedType. For our usecase this is not necessarily correct, we want QuantileType to be treated as a separate type, yet live under the same QuantDialect as QuantizedType. 

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