[Mlir-commits] [mlir] 4bd6679 - Apply clang-tidy fixes for performance-unnecessary-value-param in TestTilingInterface.cpp (NFC)

Mehdi Amini llvmlistbot at llvm.org
Mon Sep 12 15:58:35 PDT 2022


Author: Mehdi Amini
Date: 2022-09-12T22:58:11Z
New Revision: 4bd66792bb0773fb1e8f5b5edb827ada47c08101

URL: https://github.com/llvm/llvm-project/commit/4bd66792bb0773fb1e8f5b5edb827ada47c08101
DIFF: https://github.com/llvm/llvm-project/commit/4bd66792bb0773fb1e8f5b5edb827ada47c08101.diff

LOG: Apply clang-tidy fixes for performance-unnecessary-value-param in TestTilingInterface.cpp (NFC)

Added: 
    

Modified: 
    mlir/test/lib/Interfaces/TilingInterface/TestTilingInterface.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/test/lib/Interfaces/TilingInterface/TestTilingInterface.cpp b/mlir/test/lib/Interfaces/TilingInterface/TestTilingInterface.cpp
index 2ba93caf02ba6..b20cd640c3881 100644
--- a/mlir/test/lib/Interfaces/TilingInterface/TestTilingInterface.cpp
+++ b/mlir/test/lib/Interfaces/TilingInterface/TestTilingInterface.cpp
@@ -11,6 +11,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+#include <utility>
+
 #include "mlir/Dialect/Affine/IR/AffineOps.h"
 #include "mlir/Dialect/Func/IR/FuncOps.h"
 #include "mlir/Dialect/Linalg/IR/Linalg.h"
@@ -40,7 +42,8 @@ struct TestTileUsingSCFForOpWithFilter : public scf::TileUsingSCFForOp {
                                   linalg::LinalgTransformationFilter filter =
                                       linalg::LinalgTransformationFilter(),
                                   PatternBenefit benefit = 1)
-      : scf::TileUsingSCFForOp(context, options, benefit), filter(filter) {}
+      : scf::TileUsingSCFForOp(context, std::move(options), benefit),
+        filter(std::move(filter)) {}
 
   /// Construct a generic pattern applied to `opName`.
   TestTileUsingSCFForOpWithFilter(StringRef opName, MLIRContext *context,
@@ -48,7 +51,8 @@ struct TestTileUsingSCFForOpWithFilter : public scf::TileUsingSCFForOp {
                                   linalg::LinalgTransformationFilter filter =
                                       linalg::LinalgTransformationFilter(),
                                   PatternBenefit benefit = 1)
-      : scf::TileUsingSCFForOp(context, options, benefit), filter(filter) {}
+      : scf::TileUsingSCFForOp(context, std::move(options), benefit),
+        filter(std::move(filter)) {}
 
   LogicalResult matchAndRewrite(TilingInterface op,
                                 PatternRewriter &rewriter) const override {
@@ -78,9 +82,9 @@ struct TestTileConsumerAndFuseProducersUsingSCFForOpWithFilter
       linalg::LinalgTransformationFilter filter =
           linalg::LinalgTransformationFilter(),
       PatternBenefit benefit = 1)
-      : scf::TileConsumerAndFuseProducersUsingSCFForOp(context, options,
-                                                       benefit),
-        filter(filter) {}
+      : scf::TileConsumerAndFuseProducersUsingSCFForOp(
+            context, std::move(options), benefit),
+        filter(std::move(filter)) {}
 
   /// Construct a generic pattern applied to `opName`.
   TestTileConsumerAndFuseProducersUsingSCFForOpWithFilter(
@@ -88,9 +92,9 @@ struct TestTileConsumerAndFuseProducersUsingSCFForOpWithFilter
       linalg::LinalgTransformationFilter filter =
           linalg::LinalgTransformationFilter(),
       PatternBenefit benefit = 1)
-      : scf::TileConsumerAndFuseProducersUsingSCFForOp(context, options,
-                                                       benefit),
-        filter(filter) {}
+      : scf::TileConsumerAndFuseProducersUsingSCFForOp(
+            context, std::move(options), benefit),
+        filter(std::move(filter)) {}
 
   LogicalResult matchAndRewrite(TilingInterface op,
                                 PatternRewriter &rewriter) const override {


        


More information about the Mlir-commits mailing list