From e965d068e08be3caddf44f8440dd16f29b4fc15b Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Mon, 17 Jan 2022 23:16:18 +0000 Subject: [PATCH] Pass options by const ref in TestLinalgCodegenStrategy (NFC) These aren't small object, fix Coverity report. --- .../lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp b/mlir/test/lib/Dialect/Linalg/TestLinalgCodegenStrategy.cpp index 0e18725b5eb2..d56337130c07 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 ®isterTilingOptions, 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 ®isterTilingOptions, LinalgPaddingOptions paddingOptions, vector::VectorContractLowering vectorContractLowering, vector::VectorTransferSplit vectorTransferSplit) {