Skip to content

Commit 5901316

Browse files
committed
Fixups
1 parent d875a1a commit 5901316

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -244,9 +244,8 @@ struct AssignTileIDsPattern
244244

245245
// Set all operations dependent on `tileOp` to use the same tile ID.
246246
// This is a naive tile allocation scheme, but works for common cases. For
247-
// example, as this only allocates tile IDs to existing ops, it can't
248-
// solve cases like this (%tileA and %tileB come from different root
249-
// operations):
247+
// example, as this only allocates tile IDs to existing ops, it can't solve
248+
// cases like this (%tileA and %tileB come from different root operations):
250249
//
251250
// %tile = scf.if %some_cond -> vector<[4]x[4]xi32> {
252251
// scf.yield %tileA {tile_id = 0} : vector<[4]x[4]xi32>
@@ -255,9 +254,9 @@ struct AssignTileIDsPattern
255254
// }
256255
//
257256
// This case would require allocating a new tile for the result of the
258-
// scf.if, and moving the contents of %tileA or %tileB to result tile
259-
// (based on the %some_cond). Find all the ops that (transitively) depend
260-
// on this tile.
257+
// scf.if, and moving the contents of %tileA or %tileB to result tile (based
258+
// on the %some_cond).
259+
// Find all the ops that (transitively) depend on this tile.
261260
SetVector<Operation *> dependantOps;
262261
findDependantOps(tileOp->getResult(0), dependantOps);
263262
auto tileIDAttr = rewriter.getI32IntegerAttr(*tileId);

mlir/lib/Dialect/ArmSME/Transforms/VectorLegalization.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -351,10 +351,10 @@ struct VectorLegalizationPass
351351
SmallVectorImpl<Type> &types) -> std::optional<LogicalResult> {
352352
if (!isMultipleOfSMETileVectorType(vectorType))
353353
return std::nullopt;
354-
auto smeTileTileCount = getNumberOfSMETilesForVectorType(vectorType);
354+
auto smeTileCount = getNumberOfSMETilesForVectorType(vectorType);
355355
auto smeTileType =
356356
getSMETileTypeForElement(vectorType.getElementType());
357-
types = SmallVector<Type>(smeTileTileCount, smeTileType);
357+
types = SmallVector<Type>(smeTileCount, smeTileType);
358358
return success();
359359
});
360360

0 commit comments

Comments
 (0)