[Mlir-commits] [mlir] 95bdbb9 - [mlir][affine] Make loop tiling default options explicit

[llvmlistbot at llvm.org]

Author: lewuathe
Date: 2022-06-15T11:28:21+09:00
New Revision: 95bdbb97476ad9686d6c6b52211efeab41333647

URL: https://github.com/llvm/llvm-project/commit/95bdbb97476ad9686d6c6b52211efeab41333647
DIFF: https://github.com/llvm/llvm-project/commit/95bdbb97476ad9686d6c6b52211efeab41333647.diff

LOG: [mlir][affine] Make loop tiling default options explicit

Make default loop tiling options explicit from CLI options. We can also set default value for separate option which is declared implicitly.

Reviewed By: ayzhuang

Differential Revision: https://reviews.llvm.org/D127711

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/Affine/Passes.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/Affine/Passes.td b/mlir/include/mlir/Dialect/Affine/Passes.td
index 7f942405cd7e9..9187fa8574d90 100644
--- a/mlir/include/mlir/Dialect/Affine/Passes.td
+++ b/mlir/include/mlir/Dialect/Affine/Passes.td
@@ -186,9 +186,9 @@ def AffineLoopTiling : Pass<"affine-loop-tile", "func::FuncOp"> {
   let constructor = "mlir::createLoopTilingPass()";
   let options = [
     Option<"cacheSizeInKiB", "cache-size", "uint64_t", /*default=*/"512",
-           "Set size of cache to tile for in KiB">,
-    Option<"separate", "separate", "bool", /*default=*/"",
-           "Separate full and partial tiles">,
+           "Set size of cache to tile for in KiB (default: 512)">,
+    Option<"separate", "separate", "bool", /*default=*/"false",
+           "Separate full and partial tiles (default: false)">,
     Option<"tileSize", "tile-size", "unsigned", /*default=*/"",
            "Use this tile size for all loops">,
     ListOption<"tileSizes", "tile-sizes", "unsigned",