[mlir][linalg][transform] Rename ApplyPatternsOp.{region => patterns}.
authorIngo Müller <ingomueller@google.com>
Thu, 20 Jul 2023 10:13:55 +0000 (10:13 +0000)
committerIngo Müller <ingomueller@google.com>
Thu, 20 Jul 2023 14:20:45 +0000 (14:20 +0000)
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

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

index 6820f3c..37312e8 100644 (file)
@@ -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;