Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 15f7c6e

Browse files
kazutakahiratakazutakahirata
authored
[mlir] Remove unused local variables (NFC) (#138481)
1 parent e1cff21 commit 15f7c6e

23 files changed

+1
-36
lines changed

‎mlir/lib/Dialect/AMDGPU/Transforms/TransferReadToLoad.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,6 @@ struct TransferReadLowering final : OpRewritePattern<vector::TransferReadOp> {
175175
// It computes the multiplied sizes of all dimensions instead of taking
176176
// the maximum of each dimension size * stride.
177177
SmallVector<AffineExpr> productExpressions;
178-
SmallVector<Value> productResults;
179178
unsigned sourceRank = cast<ShapedType>(src.getType()).getRank();
180179

181180
SmallVector<AffineExpr> symbols(2 * sourceRank);

‎mlir/lib/Dialect/Affine/Analysis/Utils.cpp

-6
Original file line numberDiff line numberDiff line change
@@ -1356,12 +1356,6 @@ std::optional<int64_t> MemRefRegion::getRegionSize() {
13561356
return false;
13571357
}
13581358

1359-
// Indices to use for the DmaStart op.
1360-
// Indices for the original memref being DMAed from/to.
1361-
SmallVector<Value, 4> memIndices;
1362-
// Indices for the faster buffer being DMAed into/from.
1363-
SmallVector<Value, 4> bufIndices;
1364-
13651359
// Compute the extents of the buffer.
13661360
std::optional<int64_t> numElements = getConstantBoundingSizeAndShape();
13671361
if (!numElements) {

‎mlir/lib/Dialect/Affine/TransformOps/AffineTransformOps.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,6 @@ SimplifyBoundedAffineOpsOp::apply(transform::TransformRewriter &rewriter,
118118
}
119119
targets.push_back(target);
120120
}
121-
SmallVector<Operation *> transformed;
122121
RewritePatternSet patterns(getContext());
123122
// Canonicalization patterns are needed so that affine.apply ops are composed
124123
// with the remaining affine.min/max ops.

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

-1
Original file line numberDiff line numberDiff line change
@@ -1808,7 +1808,6 @@ mlir::affine::normalizeMemRef(memref::ReinterpretCastOp reinterpretCastOp) {
18081808
ArrayRef<int64_t> oldShape = memrefType.getShape();
18091809
ValueRange oldSizes = reinterpretCastOp.getSizes();
18101810
unsigned idx = 0;
1811-
SmallVector<int64_t> newStaticSizes;
18121811
OpBuilder b(reinterpretCastOp);
18131812
// Collect the map operands which will be used to compute the new normalized
18141813
// memref shape.

‎mlir/lib/Dialect/Affine/Utils/ViewLikeInterfaceUtils.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ LogicalResult mlir::affine::mergeOffsetsSizesAndStrides(
4141
combinedStrides[i] = producerStrides[i];
4242
continue;
4343
}
44-
SmallVector<OpFoldResult> offsetSymbols, strideSymbols;
4544
// The combined offset is computed as
4645
// producer_offset + consumer_offset * producer_strides.
4746
combinedOffsets[i] = makeComposedFoldedAffineApply(

‎mlir/lib/Dialect/EmitC/IR/EmitC.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -465,7 +465,6 @@ ParseResult ForOp::parse(OpAsmParser &parser, OperationState &result) {
465465

466466
// Parse the optional initial iteration arguments.
467467
SmallVector<OpAsmParser::Argument, 4> regionArgs;
468-
SmallVector<OpAsmParser::UnresolvedOperand, 4> operands;
469468
regionArgs.push_back(inductionVariable);
470469

471470
// Parse optional type, else assume Index.

‎mlir/lib/Dialect/GPU/IR/GPUDialect.cpp

-3
Original file line numberDiff line numberDiff line change
@@ -936,9 +936,6 @@ ParseResult LaunchOp::parse(OpAsmParser &parser, OperationState &result) {
936936
SmallVector<OpAsmParser::UnresolvedOperand, LaunchOp::kNumConfigOperands>
937937
sizes(LaunchOp::kNumConfigOperands);
938938

939-
// Actual (data) operands passed to the kernel.
940-
SmallVector<OpAsmParser::UnresolvedOperand, 4> dataOperands;
941-
942939
// Region arguments to be created.
943940
SmallVector<OpAsmParser::UnresolvedOperand, 16> regionArgs(
944941
LaunchOp::kNumConfigRegionAttributes);

‎mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -1416,7 +1416,6 @@ static void addBodyWithPayloadOp(OpAsmParser &parser, OperationState &result,
14161416
Region *body = result.addRegion();
14171417
Block &block = body->emplaceBlock();
14181418
b.setInsertionPointToStart(&block);
1419-
SmallVector<Value> bbArgs;
14201419
for (auto &operand : operands) {
14211420
block.addArgument(
14221421
llvm::cast<ShapedType>(operand.getType()).getElementType(),

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

-2
Original file line numberDiff line numberDiff line change
@@ -2288,7 +2288,6 @@ transform::ScalarizeOp::applyToOne(transform::TransformRewriter &rewriter,
22882288
}
22892289
return tileSizes;
22902290
});
2291-
SmallVector<int64_t> emptyTileSizes;
22922291
rewriter.setInsertionPoint(target);
22932292
FailureOr<scf::SCFTilingResult> maybeTilingResult = tileUsingSCF(
22942293
rewriter, cast<TilingInterface>(target.getOperation()), tilingOptions);
@@ -2347,7 +2346,6 @@ transform::RewriteInDestinationPassingStyleOp::applyToOne(
23472346
transform::TransformRewriter &rewriter, Operation *target,
23482347
transform::ApplyToEachResultList &results,
23492348
transform::TransformState &state) {
2350-
SmallVector<Operation *> res;
23512349
rewriter.setInsertionPoint(target);
23522350
FailureOr<Operation *> maybeResult =
23532351
TypeSwitch<Operation *, FailureOr<Operation *>>(target)

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -869,7 +869,7 @@ fuseWithReshapeByExpansion(LinalgOp linalgOp, Operation *reshapeOp,
869869
"preconditions for fuse operation failed");
870870

871871
Location loc = linalgOp.getLoc();
872-
SmallVector<OpFoldResult> expandedShape, collapsedShape;
872+
SmallVector<OpFoldResult> expandedShape;
873873
SmallVector<AffineMap, 4> reassociationIndices;
874874
Value src;
875875
if (auto expandingReshapeOp = dyn_cast<tensor::ExpandShapeOp>(reshapeOp)) {

‎mlir/lib/Dialect/Linalg/Transforms/Fusion.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ getShapeDefiningLoopRange(LinalgOp op, unsigned loopDepth,
8585
<< opOperand.getOperandNumber() << "\n");
8686
LLVM_DEBUG(llvm::dbgs()
8787
<< "getShapeDefiningLoopRange map: " << map << "\n");
88-
SmallVector<Value, 8> shapeRanges(map.getNumResults(), nullptr);
8988
for (const auto &en : llvm::enumerate(map.getResults())) {
9089
auto dimExpr = dyn_cast<AffineDimExpr>(en.value());
9190
if (!dimExpr)

‎mlir/lib/Dialect/Linalg/Transforms/Promotion.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,6 @@ FailureOr<PromotionInfo> mlir::linalg::promoteSubviewAsNewBuffer(
270270
partialSizes.push_back(
271271
b.createOrFold<memref::DimOp>(loc, subView, resultDimIdx++));
272272
}
273-
SmallVector<int64_t, 4> dynSizes(fullSizes.size(), ShapedType::kDynamic);
274273
// If a callback is not specified, then use the default implementation for
275274
// allocating the promoted buffer.
276275
std::optional<Value> fullLocalView =

‎mlir/lib/Dialect/Linalg/Transforms/TilingInterfaceImpl.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -1187,7 +1187,6 @@ struct UnPackOpTiling
11871187
loc, unPackOp.getDest(), outputOffsets, outputSizes, strides);
11881188
tiledOperands.push_back(extractDestSlice);
11891189

1190-
SmallVector<OpFoldResult> inputOffsets, inputSizes;
11911190
strides.append(unPackOp.getSourceRank() - outputRank, oneAttr);
11921191
// Create slice of the source operand.
11931192
auto extractSourceSlice = b.create<tensor::ExtractSliceOp>(

‎mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ Type mlir::memref::getTensorTypeFromMemRefType(Type type) {
6868
OpFoldResult memref::getMixedSize(OpBuilder &builder, Location loc, Value value,
6969
int64_t dim) {
7070
auto memrefType = llvm::cast<MemRefType>(value.getType());
71-
SmallVector<OpFoldResult> result;
7271
if (memrefType.isDynamicDim(dim))
7372
return builder.createOrFold<memref::DimOp>(loc, value, dim);
7473

‎mlir/lib/Dialect/MemRef/Transforms/ExpandStridedMetadata.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -505,7 +505,6 @@ getCollapsedStride(memref::CollapseShapeOp collapseShape, OpBuilder &builder,
505505

506506
auto [strides, offset] = sourceType.getStridesAndOffset();
507507

508-
SmallVector<OpFoldResult> groupStrides;
509508
ArrayRef<int64_t> srcShape = sourceType.getShape();
510509

511510
OpFoldResult lastValidStride = nullptr;

‎mlir/lib/Dialect/MemRef/Transforms/FoldMemRefAliasOps.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,6 @@ resolveSourceIndicesCollapseShape(Location loc, PatternRewriter &rewriter,
168168
ValueRange indices,
169169
SmallVectorImpl<Value> &sourceIndices) {
170170
int64_t cnt = 0;
171-
SmallVector<Value> tmp(indices.size());
172171
SmallVector<OpFoldResult> dynamicIndices;
173172
for (ArrayRef<int64_t> groups : collapseShapeOp.getReassociationIndices()) {
174173
assert(!groups.empty() && "association indices groups cannot be empty");

‎mlir/lib/Dialect/Mesh/Interfaces/ShardingInterface.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,6 @@ MeshSharding getSharding(OpResult result, const ShardingOption &shardingOption,
409409

410410
// process the split axes
411411
for (auto it : llvm::enumerate(map.getResults())) {
412-
SmallVector<MeshAxis> tmp_axes;
413412
AffineExpr expr = it.value();
414413
// `expr` must be an `AffineDimExpr` because `map` is verified by
415414
// isProjectedPermutation

‎mlir/lib/Dialect/SCF/Transforms/BufferizableOpInterfaceImpl.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,6 @@ struct IndexSwitchOpInterface
392392
int64_t resultNum = cast<OpResult>(value).getResultNumber();
393393

394394
// Helper function to get buffer type of a case.
395-
SmallVector<BaseMemRefType> yieldedTypes;
396395
auto getYieldedBufferType = [&](Block &b) -> FailureOr<BaseMemRefType> {
397396
auto yieldOp = cast<scf::YieldOp>(b.getTerminator());
398397
Value yieldedValue = yieldOp->getOperand(resultNum);

‎mlir/lib/Dialect/SCF/Transforms/TileUsingInterface.cpp

-4
Original file line numberDiff line numberDiff line change
@@ -483,8 +483,6 @@ static LogicalResult generateLoopNestUsingForallOp(
483483
assert(loopRanges.size() == tileSizes.size() &&
484484
"expected as many tile sizes as loop ranges");
485485
OpBuilder::InsertionGuard guard(rewriter);
486-
SmallVector<OpFoldResult> offsets(loopRanges.size()),
487-
sizes(loopRanges.size());
488486

489487
std::optional<ArrayAttr> mappingAttr;
490488
if (!mappingVector.empty())
@@ -865,7 +863,6 @@ FailureOr<LoopLikeOpInterface> yieldTiledValuesAndReplaceLoop(
865863
static LogicalResult addInitOperandsToLoopNest(
866864
RewriterBase &rewriter, MutableArrayRef<LoopLikeOpInterface> loops,
867865
ValueRange newInitValues, YieldTiledValuesFn getNewTiledYieldsFn) {
868-
SmallVector<scf::ForOp> newLoops;
869866
if (loops.empty())
870867
return success();
871868
OpBuilder::InsertionGuard g(rewriter);
@@ -1535,7 +1532,6 @@ mlir::scf::tileConsumerAndFuseProducersUsingSCF(
15351532

15361533
// 1. First tile the consumer.
15371534
SetVector<Operation *> fusedProducers, tiledAndFusedOps;
1538-
llvm::SmallDenseMap<Value, size_t> origProducerToLoopResultNum;
15391535

15401536
FailureOr<scf::SCFTilingResult> tilingResult =
15411537
tileUsingSCF(rewriter, consumer, options.tilingOptions);

‎mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp

-2
Original file line numberDiff line numberDiff line change
@@ -776,8 +776,6 @@ void spirv::EntryPointOp::build(OpBuilder &builder, OperationState &state,
776776
ParseResult spirv::EntryPointOp::parse(OpAsmParser &parser,
777777
OperationState &result) {
778778
spirv::ExecutionModel execModel;
779-
SmallVector<OpAsmParser::UnresolvedOperand, 0> identifiers;
780-
SmallVector<Type, 0> idTypes;
781779
SmallVector<Attribute, 4> interfaceVars;
782780

783781
FlatSymbolRefAttr fn;

‎mlir/lib/Dialect/SPIRV/IR/SPIRVParsingUtils.h

-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ ParseResult
4949
parseEnumKeywordAttr(EnumClass &value, ParserType &parser,
5050
StringRef attrName = spirv::attributeName<EnumClass>()) {
5151
StringRef keyword;
52-
SmallVector<NamedAttribute, 1> attr;
5352
auto loc = parser.getCurrentLocation();
5453
if (parser.parseKeyword(&keyword))
5554
return failure();

‎mlir/lib/Dialect/SparseTensor/Transforms/SparseGPUCodegen.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -1031,7 +1031,6 @@ static LogicalResult rewrite2To4SpMM(PatternRewriter &rewriter,
10311031
.getAsyncToken();
10321032
token = rewriter.create<gpu::DestroyDnTensorOp>(loc, tokenTp, token, dnC)
10331033
.getAsyncToken();
1034-
SmallVector<Value> newDynamicSizes;
10351034
token = genDeallocMemRef(rewriter, loc, buffer1, token);
10361035
token = genDeallocMemRef(rewriter, loc, buffer2, token);
10371036
token = genDeallocMemRef(rewriter, loc, buffer3, token);

‎mlir/lib/Dialect/Tensor/IR/TensorOps.cpp

-2
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ Operation *TensorDialect::materializeConstant(OpBuilder &builder,
6060
OpFoldResult tensor::getMixedSize(OpBuilder &builder, Location loc, Value value,
6161
int64_t dim) {
6262
auto tensorType = llvm::cast<RankedTensorType>(value.getType());
63-
SmallVector<OpFoldResult> result;
6463
if (tensorType.isDynamicDim(dim))
6564
return builder.createOrFold<tensor::DimOp>(loc, value, dim);
6665

@@ -1703,7 +1702,6 @@ OpFoldResult ReshapeOp::fold(FoldAdaptor adaptor) {
17031702
if (auto dimOp = element.getDefiningOp<tensor::DimOp>()) {
17041703
dynamicNoop &= dimOp.getSource() == source;
17051704

1706-
APSInt dim;
17071705
auto cst = getConstantIntValue(dimOp.getIndex());
17081706
dynamicNoop &=
17091707
cst.has_value() && cst.value() == static_cast<int64_t>(id);

0 commit comments

Comments
 (0)