Skip to content

Commit e151e84

Browse files
committed
[MLIR] Apply clang-tidy fixes for bugprone-argument-comment in LinalgTransformOps.cpp (NFC)
1 parent 8009a5b commit e151e84

File tree

1 file changed

+25
-25
lines changed

1 file changed

+25
-25
lines changed

mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp

Lines changed: 25 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ void transform::BufferizeToAllocationOp::build(OpBuilder &b,
283283
return build(b, result,
284284
/*resultTypes=*/resultTypes,
285285
/*target=*/target,
286-
/*memorySpace=*/memorySpace);
286+
/*memory_space=*/memorySpace);
287287
}
288288

289289
void transform::BufferizeToAllocationOp::build(OpBuilder &b,
@@ -296,7 +296,7 @@ void transform::BufferizeToAllocationOp::build(OpBuilder &b,
296296
return build(b, result,
297297
/*resultTypes=*/resultTypes,
298298
/*target=*/target,
299-
/*memorySpace=*/b.getI64IntegerAttr(memorySpace));
299+
/*memory_space=*/b.getI64IntegerAttr(memorySpace));
300300
}
301301

302302
namespace {
@@ -1903,21 +1903,21 @@ void transform::PadOp::build(OpBuilder &b, OperationState &result, Value target,
19031903
StringRef copyBackOp,
19041904
bool usePrescribedTensorShapes) {
19051905
auto resultType = transform::AnyOpType::get(b.getContext());
1906-
return build(/*builder=*/b,
1906+
return build(/*odsBuilder=*/b,
19071907
/*result=*/result,
19081908
/*types=*/TypeRange{resultType, resultType},
19091909
/*target=*/target,
1910-
/*paddingValues=*/ArrayAttr(), // let inference handle this
1911-
/*paddingDimensions=*/b.getI64ArrayAttr(paddingDimensions),
1912-
/*padToMultipleOf=*/ValueRange{},
1910+
/*padding_values=*/ArrayAttr(), // let inference handle this
1911+
/*padding_dimensions=*/b.getI64ArrayAttr(paddingDimensions),
1912+
/*pad_to_multiple_of=*/ValueRange{},
19131913
/*padToMultipleOf=*/
19141914
(padToMultipleOf.empty()
19151915
? DenseI64ArrayAttr()
19161916
: b.getDenseI64ArrayAttr(padToMultipleOf)),
1917-
/*nofoldFlags=*/b.getI64ArrayAttr(nofoldFlags),
1918-
/*transposePaddings=*/b.getArrayAttr(transposePaddings),
1919-
/*copyBackOp=*/b.getStringAttr(copyBackOp),
1920-
/*usePrescribedTensorShapes=*/
1917+
/*nofold_flags=*/b.getI64ArrayAttr(nofoldFlags),
1918+
/*transpose_paddings=*/b.getArrayAttr(transposePaddings),
1919+
/*copy_back_op=*/b.getStringAttr(copyBackOp),
1920+
/*use_prescribed_tensor_shapes=*/
19211921
usePrescribedTensorShapes ? b.getUnitAttr() : nullptr);
19221922
}
19231923

@@ -1933,18 +1933,18 @@ void transform::PadOp::build(OpBuilder &b, OperationState &result, Value target,
19331933
SmallVector<Value> dynamicPadToMultipleOf;
19341934
dispatchIndexOpFoldResults(mixedPadToMultipleOf, dynamicPadToMultipleOf,
19351935
staticPadToMultipleOf);
1936-
return build(/*builder=*/b,
1936+
return build(/*odsBuilder=*/b,
19371937
/*result=*/result,
19381938
/*types=*/TypeRange{resultType, resultType},
19391939
/*target=*/target,
1940-
/*paddingValues=*/ArrayAttr(), // let inference handle this
1941-
/*paddingDimensions=*/b.getI64ArrayAttr(paddingDimensions),
1942-
/*padToMultipleOf=*/dynamicPadToMultipleOf,
1940+
/*padding_values=*/ArrayAttr(), // let inference handle this
1941+
/*padding_dimensions=*/b.getI64ArrayAttr(paddingDimensions),
1942+
/*pad_to_multiple_of=*/dynamicPadToMultipleOf,
19431943
/*padToMultipleOf=*/staticPadToMultipleOf,
1944-
/*nofoldFlags=*/b.getI64ArrayAttr(nofoldFlags),
1945-
/*transposePaddings=*/b.getArrayAttr(transposePaddings),
1946-
/*copyBackOp=*/copyBackOp,
1947-
/*usePrescribedTensorShapes=*/usePrescribedTensorShapes);
1944+
/*nofold_flags=*/b.getI64ArrayAttr(nofoldFlags),
1945+
/*transpose_paddings=*/b.getArrayAttr(transposePaddings),
1946+
/*copy_back_op=*/copyBackOp,
1947+
/*use_prescribed_tensor_shapes=*/usePrescribedTensorShapes);
19481948
}
19491949

19501950
void PadOp::getEffects(
@@ -2163,16 +2163,16 @@ void transform::PadTilingInterfaceOp::build(OpBuilder &b,
21632163
ArrayRef<int64_t> paddingSizes,
21642164
bool padToMultipleOf) {
21652165
auto resultType = transform::AnyOpType::get(b.getContext());
2166-
return build(/*builder=*/b,
2166+
return build(/*odsBuilder=*/b,
21672167
/*result=*/result,
21682168
/*types=*/TypeRange{resultType, resultType},
21692169
/*target=*/target,
2170-
/*paddingValues=*/ArrayAttr(), // let inference handle this
2171-
/*paddingSizes=*/ValueRange{},
2170+
/*padding_values=*/ArrayAttr(), // let inference handle this
2171+
/*padding_sizes=*/ValueRange{},
21722172
/*paddingSizes=*/
21732173
(paddingSizes.empty() ? DenseI64ArrayAttr()
21742174
: b.getDenseI64ArrayAttr(paddingSizes)),
2175-
/*padToMultipleOf=*/
2175+
/*pad_to_multiple_of=*/
21762176
padToMultipleOf ? b.getUnitAttr() : nullptr);
21772177
}
21782178

@@ -2184,12 +2184,12 @@ void transform::PadTilingInterfaceOp::build(
21842184
SmallVector<Value> dynamicPaddingSizes;
21852185
dispatchIndexOpFoldResults(mixedPaddingSizes, dynamicPaddingSizes,
21862186
staticPaddingSizes);
2187-
return build(/*builder=*/b,
2187+
return build(/*odsBuilder=*/b,
21882188
/*result=*/result,
21892189
/*types=*/TypeRange{resultType, resultType},
21902190
/*target=*/target,
2191-
/*paddingValues=*/ArrayAttr(), // let inference handle this
2192-
/*paddingSizes=*/dynamicPaddingSizes,
2191+
/*padding_values=*/ArrayAttr(), // let inference handle this
2192+
/*padding_sizes=*/dynamicPaddingSizes,
21932193
/*paddingSizes=*/staticPaddingSizes,
21942194
/*usePrescribedTensorShapes=*/padToMultipleOf);
21952195
}

0 commit comments

Comments
 (0)