just move without deprecation

Author: rengolin

mlir/include/mlir/Dialect/Linalg/Passes.td

@@ -86,12 +86,6 @@ def LinalgSpecializeGenericOpsPass : Pass<"linalg-specialize-generic-ops">,
   let dependentDialects = ["linalg::LinalgDialect"];
 }
 
-def LinalgNamedOpConversionPass: Pass<"linalg-named-op-conversion">,
-                                 Deprecated<"Use 'simplify-depthwise-conv' instead."> {
-  let summary = "Convert from one named linalg op to another.";
-  let dependentDialects = ["linalg::LinalgDialect", "tensor::TensorDialect"];
-}
-
 // ------------------ End of "form" conversions
 
 def SimplifyDepthwiseConvPass: Pass<"simplify-depthwise-conv"> {

mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h

@@ -1966,10 +1966,6 @@ void populateFuseTensorPadWithProducerLinalgOpPatterns(
 /// Patterns to simplify depthwise convolutions.
 void populateSimplifyDepthwiseConvPatterns(RewritePatternSet &patterns);
 
-/// Patterns to convert from one named op to another. So far only used on
-/// depthwise convolutions, so deprecated. Use the pattern avove.
-void populateLinalgNamedOpConversionPatterns(RewritePatternSet &patterns);
-
 /// Patterns to fold unit-extent dimensions in operands/results of linalg ops on
 /// tensors via reassociative reshape ops.
 void populateFoldUnitExtentDimsPatterns(RewritePatternSet &patterns,

mlir/lib/Dialect/Linalg/Transforms/SimplifyDepthwiseConv.cpp

@@ -21,7 +21,6 @@
 #include "llvm/ADT/TypeSwitch.h"
 
 namespace mlir {
-#define GEN_PASS_DEF_LINALGNAMEDOPCONVERSIONPASS
 #define GEN_PASS_DEF_SIMPLIFYDEPTHWISECONVPASS
 #include "mlir/Dialect/Linalg/Passes.h.inc"
 } // namespace mlir
@@ -157,33 +156,10 @@ struct SimplifyDepthwiseConvPass
       return signalPassFailure();
   }
 };
-
-// Deprecated, use the one above
-struct LinalgNamedOpConversionPass
-    : public impl::LinalgNamedOpConversionPassBase<
-          LinalgNamedOpConversionPass> {
-  using impl::LinalgNamedOpConversionPassBase<
-      LinalgNamedOpConversionPass>::LinalgNamedOpConversionPassBase;
-
-  void runOnOperation() override {
-    Operation *op = getOperation();
-    RewritePatternSet patterns(op->getContext());
-    populateLinalgNamedOpConversionPatterns(patterns);
-    if (failed(applyPatternsGreedily(op, std::move(patterns))))
-      return signalPassFailure();
-  }
-};
 } // namespace
 
 void mlir::linalg::populateSimplifyDepthwiseConvPatterns(
     RewritePatternSet &patterns) {
   patterns.add<SimplifyDepthwiseConvOp, SimplifyDepthwiseConvQOp>(
       patterns.getContext());
 }
-
-// Deprecated, use the one above
-void mlir::linalg::populateLinalgNamedOpConversionPatterns(
-    RewritePatternSet &patterns) {
-  patterns.add<SimplifyDepthwiseConvOp, SimplifyDepthwiseConvQOp>(
-      patterns.getContext());
-}

mlir/test/Dialect/Linalg/namedop_conversion.mlir renamed to mlir/test/Dialect/Linalg/simplify-depthwise-conv.mlir

@@ -1,4 +1,4 @@
-// RUN: mlir-opt %s -linalg-named-op-conversion -split-input-file | FileCheck %s
+// RUN: mlir-opt %s --simplify-depthwise-conv -split-input-file | FileCheck %s
 
 // CHECK-LABEL: @depthwise_conv
 func.func @depthwise_conv(%arg0: tensor<?x?x?x?xf32>, %arg1: tensor<?x?x?x1xf32>, %arg2: tensor<?x?x?x?x1xf32>) -> tensor<?x?x?x?x1xf32> {