[Mlir-commits] [mlir] 940e290 - Apply clang-tidy fixes for performance-unnecessary-value-param in OneShotModuleBufferize.cpp (NFC)

[Mehdi Amini]

Author: Mehdi Amini
Date: 2022-05-30T18:44:28Z
New Revision: 940e290860894d274c986c88bea2dcc17f1e93c3

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

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

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/Bufferization/Transforms/OneShotModuleBufferize.h
    mlir/lib/Dialect/Bufferization/Transforms/OneShotModuleBufferize.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/Bufferization/Transforms/OneShotModuleBufferize.h b/mlir/include/mlir/Dialect/Bufferization/Transforms/OneShotModuleBufferize.h
index 367eddeeb45c7..63a1e07d16a22 100644
--- a/mlir/include/mlir/Dialect/Bufferization/Transforms/OneShotModuleBufferize.h
+++ b/mlir/include/mlir/Dialect/Bufferization/Transforms/OneShotModuleBufferize.h
@@ -32,9 +32,9 @@ LogicalResult bufferizeModuleOp(ModuleOp moduleOp,
 /// function call analysis to determine which function arguments are
 /// inplaceable. Then analyzes and bufferizes FuncOps one-by-one with One-Shot
 /// Bufferize.
-LogicalResult
-runOneShotModuleBufferize(ModuleOp moduleOp,
-                          bufferization::OneShotBufferizationOptions options);
+LogicalResult runOneShotModuleBufferize(
+    ModuleOp moduleOp,
+    const bufferization::OneShotBufferizationOptions &options);
 
 } // namespace bufferization
 } // namespace mlir

diff  --git a/mlir/lib/Dialect/Bufferization/Transforms/OneShotModuleBufferize.cpp b/mlir/lib/Dialect/Bufferization/Transforms/OneShotModuleBufferize.cpp
index e358979868403..f00097064cc02 100644
--- a/mlir/lib/Dialect/Bufferization/Transforms/OneShotModuleBufferize.cpp
+++ b/mlir/lib/Dialect/Bufferization/Transforms/OneShotModuleBufferize.cpp
@@ -486,7 +486,7 @@ LogicalResult mlir::bufferization::bufferizeModuleOp(
 }
 
 LogicalResult mlir::bufferization::runOneShotModuleBufferize(
-    ModuleOp moduleOp, OneShotBufferizationOptions options) {
+    ModuleOp moduleOp, const OneShotBufferizationOptions &options) {
   assert(options.bufferizeFunctionBoundaries &&
          "expected that function boundary bufferization is activated");
   OneShotAnalysisState analysisState(moduleOp, options);