[Mlir-commits] [mlir] 776ac21 - [mlir] minor documentation fix in GPUTransformOps.td (#121157)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Dec 26 11:18:38 PST 2024


Author: Oleksandr "Alex" Zinenko
Date: 2024-12-26T20:18:35+01:00
New Revision: 776ac21c7f95e092759ba39e5533aad90d63c86e

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

LOG: [mlir] minor documentation fix in GPUTransformOps.td (#121157)

- do not refer to handles as `PDLOperation`, this is an outdated and
incorrect vision of what they are based on the type used in the early
days;
 - use backticks around inline code.

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/GPU/TransformOps/GPUTransformOps.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/GPU/TransformOps/GPUTransformOps.td b/mlir/include/mlir/Dialect/GPU/TransformOps/GPUTransformOps.td
index 80b4547c32c101..61d4ccec5f0bd5 100644
--- a/mlir/include/mlir/Dialect/GPU/TransformOps/GPUTransformOps.td
+++ b/mlir/include/mlir/Dialect/GPU/TransformOps/GPUTransformOps.td
@@ -168,13 +168,13 @@ def MapNestedForallToThreads :
 
       #### Return modes:
 
-      This operation ignores non-gpu_launch ops and drops them in the return.
+      This operation ignores non-`gpu_launch` ops and drops them in the return.
 
       If any scf.forall with tensors is found, the transform definitely
       fails.
 
-      If all the scf.forall operations with gpu.thread mapping contained
-      within the LaunchOp referred to by the `target` PDLOperation lower to GPU
+      If all the `scf.forall` operations with gpu.thread mapping contained
+      within the `LaunchOp` referred to by the `target` handle lower to GPU
       properly, the transform succeeds. Otherwise the transform definitely
       fails.
 
@@ -277,8 +277,8 @@ def MapForallToBlocks :
     If any scf.forall with tensors is found, the transform definitely
     fails.
 
-    If all the scf.forall operations contained within the LaunchOp
-    referred to by the `target` PDLOperation lower to GPU properly, the
+    If all the `scf.forall` operations contained within the LaunchOp
+    referred to by the `target` handle lower to GPU properly, the
     transform succeeds. Otherwise the transform definitely fails.
 
     The returned handle points to the same LaunchOp operand, consuming it and


        


More information about the Mlir-commits mailing list