[PATCH] D144260: [mlir][linalg][TransformOps] Connect hoistRedundantVectorTransfers
Nicolas Vasilache via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 17 05:05:43 PST 2023
nicolasvasilache accepted this revision.
nicolasvasilache added inline comments.
This revision is now accepted and ready to land.
================
Comment at: mlir/include/mlir/Dialect/Linalg/TransformOps/LinalgTransformOps.td:1727
+
+ The operation always returns the handle to the transformed function op.
+ }];
----------------
`always succeeds and returns a handle to` ?
================
Comment at: mlir/test/Dialect/Linalg/transform-op-hoisting.mlir:1
+// RUN: mlir-opt -test-transform-dialect-interpreter --split-input-file --allow-unregistered-dialect %s | FileCheck %s
+
----------------
This looks similar to what I remember adding in the past.
Can we take the opportunity to also delete the old test pass and just update the test command-line ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144260/new/
https://reviews.llvm.org/D144260
More information about the llvm-commits
mailing list