[Mlir-commits] [mlir] [MLIR][SCF] Add an API to fuse consumer to a producer within scf loop (PR #88712)

[llvmlistbot at llvm.org]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff c006b9088fa564fe343550a8dd5f9b0ad0c15b75 99723e4f0bbea5d84b38012ea39295d6c2a5f97b -- mlir/include/mlir/Dialect/SCF/Transforms/TileUsingInterface.h mlir/lib/Dialect/Linalg/Transforms/TilingInterfaceImpl.cpp mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp mlir/test/lib/Interfaces/TilingInterface/TestTilingInterfaceTransformOps.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp b/mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp
index 45c8f8362a..a94e6fc663 100644
--- a/mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp
+++ b/mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp
@@ -942,8 +942,7 @@ tileAndFuseConsumerOfSliceSCFForall(
   SmallVector<Value> dpsInits = llvm::to_vector<4>(
       llvm::map_range(dstOp.getDpsInits(), [](Value v) { return v; }));
   if (llvm::is_contained(dpsInits, bridge)) {
-    llvm::outs() << "consumer op take result of scf.forall as init"
-                 << "\n";
+    llvm::outs() << "consumer op take result of scf.forall as init" << "\n";
     return failure();
   }
 
@@ -959,16 +958,14 @@ tileAndFuseConsumerOfSliceSCFForall(
       if (!targetInsertOp) {
         targetInsertOp = parallelInsertSliceOp;
       } else {
-        llvm::outs() << "containingOp's result inserted multi time"
-                     << "\n";
+        llvm::outs() << "containingOp's result inserted multi time" << "\n";
         return failure();
       }
     }
   }
 
   if (!targetInsertOp) {
-    llvm::outs() << "containingOp's result was not inserted"
-                 << "\n";
+    llvm::outs() << "containingOp's result was not inserted" << "\n";
     return failure();
   }
 
@@ -983,8 +980,7 @@ tileAndFuseConsumerOfSliceSCFForall(
         }
         return true;
       })) {
-    llvm::outs() << "containingOp's result yield with stride"
-                 << "\n";
+    llvm::outs() << "containingOp's result yield with stride" << "\n";
     return failure();
   }
 
@@ -1023,8 +1019,7 @@ tileAndFuseConsumerOfSliceSCFForall(
       tileableConsumer.getTiledImplementationFromOperandPosition(
           rewriter, operandNums.front(), offsets, sizes);
   if (failed(tileAndFuseResult)) {
-    llvm::outs() << "get tiled implementation failed"
-                 << "\n";
+    llvm::outs() << "get tiled implementation failed" << "\n";
     return failure();
   }
 
@@ -1181,8 +1176,7 @@ tileAndFuseConsumerOfSliceSCFFor(RewriterBase &rewriter,
   SmallVector<Value> dpsInits = llvm::to_vector<4>(
       llvm::map_range(dstOp.getDpsInits(), [](Value v) { return v; }));
   if (llvm::is_contained(dpsInits, bridge)) {
-    llvm::outs() << "consumer op take result of scf.for as init"
-                 << "\n";
+    llvm::outs() << "consumer op take result of scf.for as init" << "\n";
     return failure();
   }
 
@@ -1200,8 +1194,7 @@ tileAndFuseConsumerOfSliceSCFFor(RewriterBase &rewriter,
         }
         return true;
       })) {
-    llvm::outs() << "containingOp's result yield with stride"
-                 << "\n";
+    llvm::outs() << "containingOp's result yield with stride" << "\n";
     return failure();
   }
 
@@ -1237,8 +1230,7 @@ tileAndFuseConsumerOfSliceSCFFor(RewriterBase &rewriter,
       tileableConsumer.getTiledImplementationFromOperandPosition(
           rewriter, operandNums.front(), offsets, sizes);
   if (failed(tileAndFuseResult)) {
-    llvm::outs() << "get tiled implementation failed"
-                 << "\n";
+    llvm::outs() << "get tiled implementation failed" << "\n";
     return failure();
   }
 

``````````

</details>


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