[Mlir-commits] [mlir] f62cb13 - [mlir][linalg][transform] Rename ApplyPatternsOp.{region => patterns}.

Ingo Müller llvmlistbot at llvm.org
Thu Jul 20 07:20:56 PDT 2023


Author: Ingo Müller
Date: 2023-07-20T14:20:45Z
New Revision: f62cb13fb285bfce1135be73afb9d2f1c731012b

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

LOG: [mlir][linalg][transform] Rename ApplyPatternsOp.{region => patterns}.

This gives the region a more meaningful name. The topic came up in a
discussion on https://reviews.llvm.org/D155435, where the name `region`
would have led to a situation where a convenience accessor called
`region` (after the ODS name) would have returned a Block.

Reviewed By: ftynse

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

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/Transform/IR/TransformOps.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/Transform/IR/TransformOps.td b/mlir/include/mlir/Dialect/Transform/IR/TransformOps.td
index 6820f3c7412a27..37312e8f7420ab 100644
--- a/mlir/include/mlir/Dialect/Transform/IR/TransformOps.td
+++ b/mlir/include/mlir/Dialect/Transform/IR/TransformOps.td
@@ -190,9 +190,9 @@ def ApplyPatternsOp : TransformDialectOp<"apply_patterns",
   let arguments = (ins
     TransformHandleTypeInterface:$target);
   let results = (outs);
-  let regions = (region MaxSizedRegion<1>:$region);
+  let regions = (region MaxSizedRegion<1>:$patterns);
 
-  let assemblyFormat = "`to` $target $region attr-dict `:` type($target)";
+  let assemblyFormat = "`to` $target $patterns attr-dict `:` type($target)";
   let hasVerifier = 1;
 
   let skipDefaultBuilders = 1;


        


More information about the Mlir-commits mailing list