Skip to content

Commit f2c6334

Browse files
NatashaKnkNatashaKnk
committed
Fix capitaliztation
1 parent e431cc4 commit f2c6334

File tree

4 files changed

+32
-32
lines changed

4 files changed

+32
-32
lines changed

‎mlir/include/mlir/Dialect/Linalg/IR/LinalgInterfaces.td

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,32 +92,32 @@ def LinalgContractionOpInterface : OpInterface<"ContractionOpInterface"> {
9292
vector-matrix multiplication.
9393
}],
9494
/*retTy=*/"bool",
95-
/*methodName=*/"isVecMat",
95+
/*methodName=*/"isVecmat",
9696
/*args=*/(ins),
9797
/*methodBody=*/[{
98-
return mlir::isVecMat($_op.getIndexingMaps());
98+
return mlir::isVecmat($_op.getIndexingMaps());
9999
}]>,
100100
InterfaceMethod<
101101
/*desc=*/[{
102102
Returns whether the given op has indexing maps that correspond to a
103103
matrix-vector multiplication.
104104
}],
105105
/*retTy=*/"bool",
106-
/*methodName=*/"isMatVec",
106+
/*methodName=*/"isMatvec",
107107
/*args=*/(ins),
108108
/*methodBody=*/[{
109-
return mlir::isMatVec($_op.getIndexingMaps());
109+
return mlir::isMatvec($_op.getIndexingMaps());
110110
}]>,
111111
InterfaceMethod<
112112
/*desc=*/[{
113113
Returns whether the given op has indexing maps that correspond to a
114114
batched matrix-vector multiplication.
115115
}],
116116
/*retTy=*/"bool",
117-
/*methodName=*/"isBatchMatVec",
117+
/*methodName=*/"isBatchMatvec",
118118
/*args=*/(ins),
119119
/*methodBody=*/[{
120-
return mlir::isBatchMatVec($_op.getIndexingMaps());
120+
return mlir::isBatchMatvec($_op.getIndexingMaps());
121121
}]>,
122122
];
123123
}

‎mlir/include/mlir/Dialect/Utils/StructuredOpsUtils.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,19 +53,19 @@ bool isRowMajorBatchMatmul(ArrayAttr indexingMaps);
5353
/// test is permutation-invariant. Note that this only checks the affine maps
5454
/// from an operation, so does not perform any checks on the math being
5555
/// performed within the reduction.
56-
bool isVecMat(ArrayAttr indexingMaps);
56+
bool isVecmat(ArrayAttr indexingMaps);
5757

5858
/// Tests whether the given maps describe a matrix vector multiplication. The
5959
/// test is permutation-invariant. Note that this only checks the affine maps
6060
/// from an operation, so does not perform any checks on the math being
6161
/// performed within the reduction.
62-
bool isMatVec(ArrayAttr indexingMaps);
62+
bool isMatvec(ArrayAttr indexingMaps);
6363

6464
/// Tests whether the given maps describe a batch matrix vector multiplication.
6565
/// The test is permutation-invariant. Note that this only checks the affine
6666
/// maps from an operation, so does not perform any checks on the math being
6767
/// performed within the reduction.
68-
bool isBatchMatVec(ArrayAttr indexingMaps);
68+
bool isBatchMatvec(ArrayAttr indexingMaps);
6969

7070
/// Return positions in `iteratorTypes` that match `iteratorTypeName`.
7171
inline void findPositionsOfType(ArrayRef<utils::IteratorType> iteratorTypes,

‎mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ bool mlir::isRowMajorBatchMatmul(ArrayAttr indexingMaps) {
9696
return indexingMaps == maps;
9797
}
9898

99-
bool mlir::isVecMat(ArrayAttr indexingMaps) {
99+
bool mlir::isVecmat(ArrayAttr indexingMaps) {
100100
if (indexingMaps.size() != 3)
101101
return false;
102102
auto map0 = cast<AffineMapAttr>(indexingMaps[0]).getValue();
@@ -120,7 +120,7 @@ bool mlir::isVecMat(ArrayAttr indexingMaps) {
120120
return indexingMaps == maps;
121121
}
122122

123-
bool mlir::isMatVec(ArrayAttr indexingMaps) {
123+
bool mlir::isMatvec(ArrayAttr indexingMaps) {
124124
if (indexingMaps.size() != 3)
125125
return false;
126126
auto map0 = cast<AffineMapAttr>(indexingMaps[0]).getValue();
@@ -144,7 +144,7 @@ bool mlir::isMatVec(ArrayAttr indexingMaps) {
144144
return indexingMaps == maps;
145145
}
146146

147-
bool mlir::isBatchMatVec(ArrayAttr indexingMaps) {
147+
bool mlir::isBatchMatvec(ArrayAttr indexingMaps) {
148148
if (indexingMaps.size() != 3)
149149
return false;
150150
auto map0 = cast<AffineMapAttr>(indexingMaps[0]).getValue();

‎mlir/unittests/Dialect/Utils/StructuredOpsUtilsTest.cpp

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ TEST(isRowMajorBatchMatmul, FirstInputSwapped) {
240240
EXPECT_THAT(maps, Not(Truly(isRowMajorBatchMatmul)));
241241
}
242242

243-
TEST(isVecMat, Simple) {
243+
TEST(isVecmat, Simple) {
244244
MLIRContext context;
245245

246246
AffineExpr k, n;
@@ -250,10 +250,10 @@ TEST(isVecMat, Simple) {
250250
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
251251
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
252252

253-
EXPECT_THAT(maps, Truly(isVecMat));
253+
EXPECT_THAT(maps, Truly(isVecmat));
254254
}
255255

256-
TEST(isVecMat, BindingSwapped) {
256+
TEST(isVecmat, BindingSwapped) {
257257
MLIRContext context;
258258

259259
AffineExpr k, n;
@@ -263,10 +263,10 @@ TEST(isVecMat, BindingSwapped) {
263263
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
264264
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
265265

266-
EXPECT_THAT(maps, Truly(isVecMat));
266+
EXPECT_THAT(maps, Truly(isVecmat));
267267
}
268268

269-
TEST(isVecMat, WrongDimOrderMatrix) {
269+
TEST(isVecmat, WrongDimOrderMatrix) {
270270
MLIRContext context;
271271

272272
AffineExpr k, n;
@@ -276,10 +276,10 @@ TEST(isVecMat, WrongDimOrderMatrix) {
276276
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
277277
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
278278

279-
EXPECT_THAT(maps, Not(Truly(isVecMat)));
279+
EXPECT_THAT(maps, Not(Truly(isVecmat)));
280280
}
281281

282-
TEST(isMatVec, Simple) {
282+
TEST(isMatvec, Simple) {
283283
MLIRContext context;
284284

285285
AffineExpr k, n;
@@ -289,10 +289,10 @@ TEST(isMatVec, Simple) {
289289
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
290290
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
291291

292-
EXPECT_THAT(maps, Truly(isMatVec));
292+
EXPECT_THAT(maps, Truly(isMatvec));
293293
}
294294

295-
TEST(isMatVec, BindingSwapped) {
295+
TEST(isMatvec, BindingSwapped) {
296296
MLIRContext context;
297297

298298
AffineExpr k, n;
@@ -302,10 +302,10 @@ TEST(isMatVec, BindingSwapped) {
302302
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
303303
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
304304

305-
EXPECT_THAT(maps, Truly(isMatVec));
305+
EXPECT_THAT(maps, Truly(isMatvec));
306306
}
307307

308-
TEST(isMatVec, WrongDimOrderMatrix) {
308+
TEST(isMatvec, WrongDimOrderMatrix) {
309309
MLIRContext context;
310310

311311
AffineExpr k, n;
@@ -315,10 +315,10 @@ TEST(isMatVec, WrongDimOrderMatrix) {
315315
auto mapC = AffineMapAttr::get(AffineMap::get(2, 0, {n}, &context));
316316
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
317317

318-
EXPECT_THAT(maps, Not(Truly(isMatVec)));
318+
EXPECT_THAT(maps, Not(Truly(isMatvec)));
319319
}
320320

321-
TEST(isBatchMatVec, Simple) {
321+
TEST(isBatchMatvec, Simple) {
322322
MLIRContext context;
323323

324324
AffineExpr batch, k, n;
@@ -328,10 +328,10 @@ TEST(isBatchMatVec, Simple) {
328328
auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {batch, n}, &context));
329329
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
330330

331-
EXPECT_THAT(maps, Truly(isBatchMatVec));
331+
EXPECT_THAT(maps, Truly(isBatchMatvec));
332332
}
333333

334-
TEST(isBatchMatVec, BindingSwapped) {
334+
TEST(isBatchMatvec, BindingSwapped) {
335335
MLIRContext context;
336336

337337
AffineExpr batch, k, n;
@@ -341,10 +341,10 @@ TEST(isBatchMatVec, BindingSwapped) {
341341
auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {batch, n}, &context));
342342
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
343343

344-
EXPECT_THAT(maps, Truly(isBatchMatVec));
344+
EXPECT_THAT(maps, Truly(isBatchMatvec));
345345
}
346346

347-
TEST(isBatchMatVec, Matmul) {
347+
TEST(isBatchMatvec, Matmul) {
348348
MLIRContext context;
349349

350350
AffineExpr m, n, k;
@@ -354,10 +354,10 @@ TEST(isBatchMatVec, Matmul) {
354354
auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {m, n}, &context));
355355
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
356356

357-
EXPECT_THAT(maps, Not(Truly(isBatchMatVec)));
357+
EXPECT_THAT(maps, Not(Truly(isBatchMatvec)));
358358
}
359359

360-
TEST(isBatchMatVec, WrongDimOrderMatrix) {
360+
TEST(isBatchMatvec, WrongDimOrderMatrix) {
361361
MLIRContext context;
362362

363363
AffineExpr batch, k, n;
@@ -367,7 +367,7 @@ TEST(isBatchMatVec, WrongDimOrderMatrix) {
367367
auto mapC = AffineMapAttr::get(AffineMap::get(3, 0, {batch, n}, &context));
368368
auto maps = ArrayAttr::get(&context, {mapA, mapB, mapC});
369369

370-
EXPECT_THAT(maps, Not(Truly(isBatchMatVec)));
370+
EXPECT_THAT(maps, Not(Truly(isBatchMatvec)));
371371
}
372372

373373
} // namespace

0 commit comments

Comments
 (0)