[Mlir-commits] [mlir] [MLIR][Linalg] Add aggregate ops decomposition pass and softmax decom… (PR #97582)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Mon Jul 8 16:47:34 PDT 2024


================
@@ -30,6 +30,16 @@ class IteratorTypeAttr;
 class LinalgOp;
 class GenericOp;
 
+/// Container for result values of decomposition.
+/// - `decomposedOps` contains operations created by the decomposition that are
+/// returned to the caller for further transformations.
+/// - `decomposedValues` contains the values corresponding to the result of the
+/// aggregate operation.
+struct DecompositionResult {
+  SmallVector<Operation *> decomposedOps;
----------------
MaheshRavishankar wrote:

I prefer it to be `Operation *`. Thats the more general form, and changing ABI causes breakages. There is no reason for this to be Linalg ops always.

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


More information about the Mlir-commits mailing list