[MLIR] Apply clang-tidy fixes for performance-unnecessary-value-param in ShardingInterface.cpp (NFC)

joker-eph · joker-eph · commit 47ddd941a9de · 2025-08-30T12:09:19.000-07:00
diff --git a/mlir/lib/Dialect/Shard/Interfaces/ShardingInterface.cpp b/mlir/lib/Dialect/Shard/Interfaces/ShardingInterface.cpp
@@ -513,8 +513,9 @@ LogicalResult shard::detail::defaultAddShardingAnnotations(
 }
 
 #ifndef NDEBUG
-static bool isValueCompatibleWithFullReplicationSharding(Value value,
-                                                         Sharding sharding) {
+static bool
+isValueCompatibleWithFullReplicationSharding(Value value,
+                                             const Sharding &sharding) {
   if (isa<RankedTensorType>(value.getType())) {
     return isFullReplication(sharding);
   }

Original file line number	Diff line number	Diff line change
`@@ -513,8 +513,9 @@ LogicalResult shard::detail::defaultAddShardingAnnotations(`
`513`	`513`	`}`
`514`	`514`
`515`	`515`	`#ifndef NDEBUG`
`516`		`-static bool isValueCompatibleWithFullReplicationSharding(Value value,`
`517`		`- Sharding sharding) {`
	`516`	`+static bool`
	`517`	`+isValueCompatibleWithFullReplicationSharding(Value value,`
	`518`	`+ const Sharding &sharding) {`
`518`	`519`	`if (isa<RankedTensorType>(value.getType())) {`
`519`	`520`	`return isFullReplication(sharding);`
`520`	`521`	`}`