Apply clang-tidy fixes for performance-for-range-copy in VectorOps.cpp (NFC)

joker-eph · joker-eph · commit f7cd3fc35ef8 · 2022-10-13T21:49:19.000Z
diff --git a/mlir/lib/Dialect/Vector/IR/VectorOps.cpp b/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
@@ -328,7 +328,7 @@ struct ElideUnitDimsInMultiDimReduction
   LogicalResult matchAndRewrite(MultiDimReductionOp reductionOp,
                                 PatternRewriter &rewriter) const override {
     ArrayRef<int64_t> shape = reductionOp.getSourceVectorType().getShape();
-    for (auto dim : enumerate(shape)) {
+    for (const auto &dim : enumerate(shape)) {
       if (reductionOp.isReducedDim(dim.index()) && dim.value() != 1)
         return failure();
     }

Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,7 @@ struct ElideUnitDimsInMultiDimReduction`
`328`	`328`	`LogicalResult matchAndRewrite(MultiDimReductionOp reductionOp,`
`329`	`329`	`PatternRewriter &rewriter) const override {`
`330`	`330`	`ArrayRef<int64_t> shape = reductionOp.getSourceVectorType().getShape();`
`331`		`- for (auto dim : enumerate(shape)) {`
	`331`	`+ for (const auto &dim : enumerate(shape)) {`
`332`	`332`	`if (reductionOp.isReducedDim(dim.index()) && dim.value() != 1)`
`333`	`333`	`return failure();`
`334`	`334`	`}`