[Mlir-commits] [mlir] [MLIR][Linalg] Introduce broadcast/transpose semantic to 'linalg.batc… (PR #122275)

Md Asghar Ahmad Shahid llvmlistbot at llvm.org
Fri Jan 10 02:29:42 PST 2025


================
@@ -3450,6 +3467,46 @@ static LogicalResult verifyExtendedMatmulSemantic(MatmulOp matmulOp,
   return success();
 }
 
+/// Checks if the given AffineMap represents a valid batch dimension.
+/// It checks if the first result dimension is a function of the first
+/// dimension.
+static bool isValidBatchDim(AffineMap bcastMap) {
+  assert(bcastMap.getNumResults() == 3 && "Expected three result dim expr.");
----------------
shahidact wrote:

Yes, it will. By definition of batch_matmul, user should not pass >3 result dim and hence thought of using assert. I got your point that you want this to be explicitly reported, will update.

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


More information about the Mlir-commits mailing list