[Mlir-commits] [mlir] [mlir][nfc] Update tests for Contract -> Op transforms (PR #76054)

Benjamin Maxwell llvmlistbot at llvm.org
Thu Dec 21 02:47:34 PST 2023


================
@@ -1,20 +1,22 @@
 // RUN: mlir-opt %s --transform-interpreter --split-input-file | FileCheck %s
 
-// NOTE - tests in this file are duplicated so that there's a version for
-//    * _fixed width_ and for _scalable_ vectors.
-// In order for the "vector.contract -> vector.outerproduct" patterns to work,
-// only the non-reduction dimension can be scalable (*). For Matmul operations
-// that is set to be the N dimension (i.e. rows of the output matrix), which
-// matches how matrix multiplication are normally implemented for e.g. 
-// Arm SVE. However, making the M dimension scalable (i.e. columns of the
-// output matrix) should work as well.
-//
-// (*) The conversion tested in this file unrolls along the reduction
-// dimension, which is not supported for scalable vectors.
+/// Tests for `vector.contract` -> `vector.outerproduct` transformations for
+/// Matmul operations:
----------------
MacDue wrote:

```suggestion
/// matmul operations:
```

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


More information about the Mlir-commits mailing list