[Mlir-commits] [mlir] [mlir][linalg] Hoist masked vector transfer pairs (PR #221481)

[Andrzej Warzyński]

================
@@ -330,8 +370,10 @@ void mlir::linalg::hoistRedundantVectorTransfers(Operation *root,
       // Check 3.
       // TODO: may want to memoize this information for performance but it
       // likely gets invalidated often.
+      // Compare the hoisted units (the `vector.mask` ops when masked) since the
+      // inner transfers live in sibling mask regions and never dominate.
----------------
banach-space wrote:

This comment feels off. Basically, you want to compare the candidate **read** (or **masked read**) with the candidate **write** (or **masked write**), right?

https://github.com/llvm/llvm-project/pull/221481