[Mlir-commits] [mlir] e965d06 - Pass options by const ref in TestLinalgCodegenStrategy (NFC)

[Mehdi Amini]

Author: Mehdi Amini
Date: 2022-01-17T23:16:47Z
New Revision: e965d068e08be3caddf44f8440dd16f29b4fc15b

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

LOG: Pass options by const ref in TestLinalgCodegenStrategy (NFC)

These aren't small object, fix Coverity report.

Added: 
    

Modified: 
    mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp b/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp
index 0e18725b5eb27..d56337130c073 100644
--- a/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp
+++ b/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp
@@ -55,9 +55,9 @@ struct TestLinalgCodegenStrategy
 
   void runOnFunction() override;
 
-  void runStrategy(LinalgTilingAndFusionOptions tilingAndFusionOptions,
-                   LinalgTilingOptions tilingOptions,
-                   LinalgTilingOptions registerTilingOptions,
+  void runStrategy(const LinalgTilingAndFusionOptions &tilingAndFusionOptions,
+                   const LinalgTilingOptions &tilingOptions,
+                   const LinalgTilingOptions &registerTilingOptions,
                    LinalgPaddingOptions paddingOptions,
                    vector::VectorContractLowering vectorContractLowering,
                    vector::VectorTransferSplit vectorTransferSplit);
@@ -168,9 +168,9 @@ struct TestLinalgCodegenStrategy
 };
 
 void TestLinalgCodegenStrategy::runStrategy(
-    LinalgTilingAndFusionOptions tilingAndFusionOptions,
-    LinalgTilingOptions tilingOptions,
-    LinalgTilingOptions registerTilingOptions,
+    const LinalgTilingAndFusionOptions &tilingAndFusionOptions,
+    const LinalgTilingOptions &tilingOptions,
+    const LinalgTilingOptions &registerTilingOptions,
     LinalgPaddingOptions paddingOptions,
     vector::VectorContractLowering vectorContractLowering,
     vector::VectorTransferSplit vectorTransferSplit) {