[Mlir-commits] [mlir] 430b47a - [MLIR] Remove unused arg from affine tiling validity check

Uday Bondhugula llvmlistbot at llvm.org
Sat Sep 5 05:38:41 PDT 2020


Author: Uday Bondhugula
Date: 2020-09-05T18:04:20+05:30
New Revision: 430b47a17d2281bd566fc1aac19de80b99e6f0c6

URL: https://github.com/llvm/llvm-project/commit/430b47a17d2281bd566fc1aac19de80b99e6f0c6
DIFF: https://github.com/llvm/llvm-project/commit/430b47a17d2281bd566fc1aac19de80b99e6f0c6.diff

LOG: [MLIR] Remove unused arg from affine tiling validity check

Drop unused function arg from affine loop tiling validity check.

Added: 
    

Modified: 
    mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp b/mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp
index 1889711cbf7a..5bded917978a 100644
--- a/mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp
+++ b/mlir/lib/Dialect/Affine/Transforms/LoopTiling.cpp
@@ -167,8 +167,7 @@ constructTiledIndexSetHyperRect(MutableArrayRef<AffineForOp> origLoops,
 /// function will return failure when any dependence component is negative along
 /// any of `origLoops`.
 static LogicalResult
-checkTilingLegality(MutableArrayRef<mlir::AffineForOp> origLoops,
-                    ArrayRef<unsigned> tileSizes) {
+checkTilingLegality(MutableArrayRef<mlir::AffineForOp> origLoops) {
   assert(!origLoops.empty() && "no original loops provided");
 
   // We first find out all dependences we intend to check.
@@ -242,7 +241,7 @@ mlir::tilePerfectlyNested(MutableArrayRef<AffineForOp> input,
   auto origLoops = input;
 
   // Perform tiling legality test.
-  if (failed(checkTilingLegality(origLoops, tileSizes)))
+  if (failed(checkTilingLegality(origLoops)))
     origLoops[0].emitRemark("tiled code is illegal due to dependences");
 
   AffineForOp rootAffineForOp = origLoops[0];


        


More information about the Mlir-commits mailing list