Apply clang-tidy fixes for performance-unnecessary-value-param in hlo_legalize_to_memref.cc (NFC)
PiperOrigin-RevId: 420343523
Change-Id: Ib7a13e222d45038943339b014569105539a46791
diff --git a/tensorflow/compiler/mlir/hlo/include/mlir-hlo/Dialect/mhlo/transforms/rewriters.h b/tensorflow/compiler/mlir/hlo/include/mlir-hlo/Dialect/mhlo/transforms/rewriters.h
index 766d9d2..3264244 100644
--- a/tensorflow/compiler/mlir/hlo/include/mlir-hlo/Dialect/mhlo/transforms/rewriters.h
+++ b/tensorflow/compiler/mlir/hlo/include/mlir-hlo/Dialect/mhlo/transforms/rewriters.h
@@ -73,9 +73,8 @@
void populateHLOToMemrefConversionPattern(
bufferization::BufferizeTypeConverter *converter,
RemoveSignTypeConverter *sign_converter, OwningRewritePatternList *patterns,
- std::function<bool(Operation *)> enforce_identity_map = [](Operation *) {
- return true;
- });
+ const std::function<bool(Operation *)> &enforce_identity_map =
+ [](Operation *) { return true; });
// Collection of rewrite patterns for lowering of shape operations from the HLO
// dialect to the standard dialect.
diff --git a/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/transforms/hlo_legalize_to_memref.cc b/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/transforms/hlo_legalize_to_memref.cc
index aabd5f2..b2adf89 100644
--- a/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/transforms/hlo_legalize_to_memref.cc
+++ b/tensorflow/compiler/mlir/hlo/lib/Dialect/mhlo/transforms/hlo_legalize_to_memref.cc
@@ -340,7 +340,7 @@
void populateHLOToMemrefConversionPattern(
bufferization::BufferizeTypeConverter* converter,
RemoveSignTypeConverter* sign_converter, OwningRewritePatternList* patterns,
- std::function<bool(Operation*)> enforce_identity_maps) {
+ const std::function<bool(Operation*)>& enforce_identity_maps) {
MLIRContext* context = patterns->getContext();
patterns->insert<HloToMemrefDynamicBroadcastInDimOpConverter>(
*converter, sign_converter, context, std::move(enforce_identity_maps));
