Skip to content

Commit 215666d

Browse files
committed
[mlir][Linalg] Apply ClangTidy fixes (NFC)
1 parent e95ca5b commit 215666d

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ static void generateFusedElementwiseOpRegion(
205205
mapper.map(bbArg, fusedBlock->addArgument(bbArg.getType(), bbArg.getLoc()));
206206

207207
// 6. All of the producer's output operands
208-
for (auto bbArg : llvm::enumerate(
208+
for (const auto &bbArg : llvm::enumerate(
209209
producerBlock.getArguments().take_back(producer.getNumDpsInits()))) {
210210
if (!preservedProducerResults.count(bbArg.index()))
211211
continue;
@@ -254,7 +254,8 @@ static void generateFusedElementwiseOpRegion(
254254
SmallVector<Value> fusedYieldValues;
255255
fusedYieldValues.reserve(producerYieldOp.getNumOperands() +
256256
consumerYieldOp.getNumOperands());
257-
for (auto producerYieldVal : llvm::enumerate(producerYieldOp.getOperands())) {
257+
for (const auto &producerYieldVal :
258+
llvm::enumerate(producerYieldOp.getOperands())) {
258259
if (preservedProducerResults.count(producerYieldVal.index()))
259260
fusedYieldValues.push_back(
260261
mapper.lookupOrDefault(producerYieldVal.value()));
@@ -281,7 +282,7 @@ mlir::linalg::fuseElementwiseOps(RewriterBase &rewriter,
281282
"expected producer of input operand");
282283
/// Find the results of the producer that have uses outside of the consumer.
283284
llvm::SmallDenseSet<int> preservedProducerResults;
284-
for (auto producerResult : llvm::enumerate(producer->getResults())) {
285+
for (const auto &producerResult : llvm::enumerate(producer->getResults())) {
285286
auto outputOperand = producer.getDpsInitOperand(producerResult.index());
286287
if (producer.payloadUsesValueFromOperand(outputOperand) ||
287288
!producer.canOpOperandsBeDropped(outputOperand) ||
@@ -335,7 +336,7 @@ mlir::linalg::fuseElementwiseOps(RewriterBase &rewriter,
335336
}
336337

337338
// 6. Collect all of the producer outputs.
338-
for (auto opOperand : llvm::enumerate(producer.getDpsInitOperands())) {
339+
for (const auto &opOperand : llvm::enumerate(producer.getDpsInitOperands())) {
339340
if (!preservedProducerResults.count(opOperand.index()))
340341
continue;
341342

0 commit comments

Comments
 (0)