Apply clang-tidy fixes for performance-unnecessary-value-param in One…

…ShotModuleBufferize.cpp (NFC)
llvm · May 30, 2022 · 940e290 · 940e290
1 parent 118d9eb
commit 940e290
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git 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
@@ -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);