[Mlir-commits] [mlir] [mlir] Use *Set::insert_range (NFC) (PR #132326)
Kazu Hirata
llvmlistbot at llvm.org
Thu Mar 20 20:50:05 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/132326
DenseSet, SmallPtrSet, SmallSet, SetVector, and StringSet recently
gained C++23-style insert_range. This patch replaces:
Dest.insert(Src.begin(), Src.end());
with:
Dest.insert_range(Src);
This patch does not touch custom begin like succ_begin for now.
>From c57e3f1d80aac03ed67770c70225210cd3286e65 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Wed, 19 Mar 2025 15:36:53 -0700
Subject: [PATCH] [mlir] Use *Set::insert_range (NFC)
DenseSet, SmallPtrSet, SmallSet, SetVector, and StringSet recently
gained C++23-style insert_range. This patch replaces:
Dest.insert(Src.begin(), Src.end());
with:
Dest.insert_range(Src);
This patch does not touch custom begin like succ_begin for now.
---
mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h | 2 +-
mlir/lib/Analysis/SliceAnalysis.cpp | 4 ++--
mlir/lib/Analysis/TopologicalSortUtils.cpp | 4 ++--
mlir/lib/Conversion/VectorToGPU/VectorToGPU.cpp | 6 +++---
mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp | 3 +--
mlir/lib/Dialect/GPU/TransformOps/GPUTransformOps.cpp | 3 +--
mlir/lib/Dialect/GPU/Transforms/KernelOutlining.cpp | 2 +-
mlir/lib/Dialect/GPU/Utils/DistributionUtils.cpp | 2 +-
mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp | 7 +++----
.../lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp | 3 +--
mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp | 5 ++---
mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp | 2 +-
mlir/lib/Dialect/Vector/IR/VectorOps.cpp | 2 +-
mlir/lib/IR/Location.cpp | 3 +--
mlir/lib/Rewrite/FrozenRewritePatternSet.cpp | 6 ++----
mlir/lib/Target/LLVMIR/ModuleImport.cpp | 2 +-
mlir/lib/Transforms/Mem2Reg.cpp | 2 +-
mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp | 4 ++--
.../test/lib/Analysis/DataFlow/TestDenseDataFlowAnalysis.h | 2 +-
.../DataFlow/TestSparseBackwardDataFlowAnalysis.cpp | 2 +-
.../lib/Dialect/Linalg/TestLinalgElementwiseFusion.cpp | 2 +-
mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp | 2 +-
mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp | 3 +--
mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp | 3 +--
24 files changed, 33 insertions(+), 43 deletions(-)
diff --git a/mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h b/mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h
index c302f6d682d69..4f90fc8831bc6 100644
--- a/mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h
+++ b/mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h
@@ -356,7 +356,7 @@ struct LinalgPromotionOptions {
std::optional<DenseSet<unsigned>> operandsToPromote;
LinalgPromotionOptions &setOperandsToPromote(ArrayRef<int64_t> operands) {
operandsToPromote = DenseSet<unsigned>();
- operandsToPromote->insert(operands.begin(), operands.end());
+ operandsToPromote->insert_range(operands);
return *this;
}
/// If ith element of `useFullTiles` is true the full view should be used
diff --git a/mlir/lib/Analysis/SliceAnalysis.cpp b/mlir/lib/Analysis/SliceAnalysis.cpp
index e01cb3a080b5c..e5f4ab57a9d88 100644
--- a/mlir/lib/Analysis/SliceAnalysis.cpp
+++ b/mlir/lib/Analysis/SliceAnalysis.cpp
@@ -171,12 +171,12 @@ mlir::getSlice(Operation *op, const BackwardSliceOptions &backwardSliceOptions,
// Compute and insert the backwardSlice starting from currentOp.
backwardSlice.clear();
getBackwardSlice(currentOp, &backwardSlice, backwardSliceOptions);
- slice.insert(backwardSlice.begin(), backwardSlice.end());
+ slice.insert_range(backwardSlice);
// Compute and insert the forwardSlice starting from currentOp.
forwardSlice.clear();
getForwardSlice(currentOp, &forwardSlice, forwardSliceOptions);
- slice.insert(forwardSlice.begin(), forwardSlice.end());
+ slice.insert_range(forwardSlice);
++currentIndex;
}
return topologicalSort(slice);
diff --git a/mlir/lib/Analysis/TopologicalSortUtils.cpp b/mlir/lib/Analysis/TopologicalSortUtils.cpp
index c406960fdecc3..2c3413fa3a292 100644
--- a/mlir/lib/Analysis/TopologicalSortUtils.cpp
+++ b/mlir/lib/Analysis/TopologicalSortUtils.cpp
@@ -159,7 +159,7 @@ SetVector<Block *> mlir::getBlocksSortedByDominance(Region ®ion) {
for (Block &b : region) {
if (blocks.count(&b) == 0) {
llvm::ReversePostOrderTraversal<Block *> traversal(&b);
- blocks.insert(traversal.begin(), traversal.end());
+ blocks.insert_range(traversal);
}
}
assert(blocks.size() == region.getBlocks().size() &&
@@ -222,7 +222,7 @@ class TopoSortHelper {
}
}
auto firstRange = llvm::make_first_range(regionCounts);
- ancestorRegions.insert(firstRange.begin(), firstRange.end());
+ ancestorRegions.insert_range(firstRange);
return res;
}
diff --git a/mlir/lib/Conversion/VectorToGPU/VectorToGPU.cpp b/mlir/lib/Conversion/VectorToGPU/VectorToGPU.cpp
index eaefe9e385793..ba05a5a000cb9 100644
--- a/mlir/lib/Conversion/VectorToGPU/VectorToGPU.cpp
+++ b/mlir/lib/Conversion/VectorToGPU/VectorToGPU.cpp
@@ -318,7 +318,7 @@ getSliceContract(Operation *op,
// Compute and insert the backwardSlice starting from currentOp.
backwardSlice.clear();
getBackwardSlice(currentOp, &backwardSlice, backwardSliceOptions);
- slice.insert(backwardSlice.begin(), backwardSlice.end());
+ slice.insert_range(backwardSlice);
// Compute and insert the forwardSlice starting from currentOp.
forwardSlice.clear();
@@ -334,7 +334,7 @@ getSliceContract(Operation *op,
} else {
getForwardSlice(currentOp, &forwardSlice, forwardSliceOptions);
}
- slice.insert(forwardSlice.begin(), forwardSlice.end());
+ slice.insert_range(forwardSlice);
++currentIndex;
}
return slice;
@@ -374,7 +374,7 @@ static SetVector<Operation *> getOpToConvert(mlir::Operation *op,
}))
return;
- opToConvert.insert(dependentOps.begin(), dependentOps.end());
+ opToConvert.insert_range(dependentOps);
});
// Sort the operations so that we can convert them in topological order.
return topologicalSort(opToConvert);
diff --git a/mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp b/mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp
index 31e81107f655c..6f93099f5f947 100644
--- a/mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp
+++ b/mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp
@@ -306,8 +306,7 @@ outlineExecuteOp(SymbolTable &symbolTable, ExecuteOp execute) {
// Collect all outlined function inputs.
SetVector<mlir::Value> functionInputs(execute.getDependencies().begin(),
execute.getDependencies().end());
- functionInputs.insert(execute.getBodyOperands().begin(),
- execute.getBodyOperands().end());
+ functionInputs.insert_range(execute.getBodyOperands());
getUsedValuesDefinedAbove(execute.getBodyRegion(), functionInputs);
// Collect types for the outlined function inputs and outputs.
diff --git a/mlir/lib/Dialect/GPU/TransformOps/GPUTransformOps.cpp b/mlir/lib/Dialect/GPU/TransformOps/GPUTransformOps.cpp
index 0737b2e22ebb8..cce477370a539 100644
--- a/mlir/lib/Dialect/GPU/TransformOps/GPUTransformOps.cpp
+++ b/mlir/lib/Dialect/GPU/TransformOps/GPUTransformOps.cpp
@@ -426,8 +426,7 @@ static DiagnosedSilenceableFailure rewriteOneForallCommonImpl(
"requires statically sized, normalized forall op");
SmallVector<int64_t> tmpMappingSizes = numParallelIterations.value();
SetVector<Attribute> forallMappingAttrs;
- forallMappingAttrs.insert(forallOp.getMapping()->getValue().begin(),
- forallOp.getMapping()->getValue().end());
+ forallMappingAttrs.insert_range(forallOp.getMapping()->getValue());
auto comparator = [](Attribute a, Attribute b) -> bool {
return cast<DeviceMappingAttrInterface>(a).getMappingId() <
cast<DeviceMappingAttrInterface>(b).getMappingId();
diff --git a/mlir/lib/Dialect/GPU/Transforms/KernelOutlining.cpp b/mlir/lib/Dialect/GPU/Transforms/KernelOutlining.cpp
index f5b5e3709d8e9..62dc1f13412d4 100644
--- a/mlir/lib/Dialect/GPU/Transforms/KernelOutlining.cpp
+++ b/mlir/lib/Dialect/GPU/Transforms/KernelOutlining.cpp
@@ -266,7 +266,7 @@ gpu::GPUFuncOp mlir::outlineKernelFunc(gpu::LaunchOp launchOp,
StringRef kernelFnName,
llvm::SmallVectorImpl<Value> &operands) {
DenseSet<Value> inputOperandSet;
- inputOperandSet.insert(operands.begin(), operands.end());
+ inputOperandSet.insert_range(operands);
SetVector<Value> operandSet(operands.begin(), operands.end());
auto funcOp = outlineKernelFuncImpl(launchOp, kernelFnName, operandSet);
for (auto operand : operandSet) {
diff --git a/mlir/lib/Dialect/GPU/Utils/DistributionUtils.cpp b/mlir/lib/Dialect/GPU/Utils/DistributionUtils.cpp
index 9d51ac3fc4bdc..29f6f32892f72 100644
--- a/mlir/lib/Dialect/GPU/Utils/DistributionUtils.cpp
+++ b/mlir/lib/Dialect/GPU/Utils/DistributionUtils.cpp
@@ -73,7 +73,7 @@ WarpDistributionPattern::moveRegionToNewWarpOpAndAppendReturns(
}
}
}
- yieldValues.insert(newYieldedValues.begin(), newYieldedValues.end());
+ yieldValues.insert_range(newYieldedValues);
WarpExecuteOnLane0Op newWarpOp = moveRegionToNewWarpOpAndReplaceReturns(
rewriter, warpOp, yieldValues.getArrayRef(), types);
rewriter.replaceOp(warpOp,
diff --git a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
index 07b19e5cb1a89..afd90bc96f234 100644
--- a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
+++ b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
@@ -1087,7 +1087,7 @@ void GenericOp::print(OpAsmPrinter &p) {
auto genericAttrNames = linalgTraitAttrNames();
llvm::StringSet<> genericAttrNamesSet;
- genericAttrNamesSet.insert(genericAttrNames.begin(), genericAttrNames.end());
+ genericAttrNamesSet.insert_range(genericAttrNames);
SmallVector<NamedAttribute, 8> genericAttrs;
for (auto attr : (*this)->getAttrs()) {
if (attr.getName() == getIteratorTypesAttrName()) {
@@ -4884,8 +4884,7 @@ static bool inferStaticShape(PackOp packOp, SmallVectorImpl<int64_t> &srcShape,
destShape.assign(packOp.getDestType().getShape().begin(),
packOp.getDestType().getShape().end());
llvm::SmallSetVector<int64_t, 4> innerDims;
- innerDims.insert(packOp.getInnerDimsPos().begin(),
- packOp.getInnerDimsPos().end());
+ innerDims.insert_range(packOp.getInnerDimsPos());
SmallVector<int64_t> inverseOuterDimsPerm;
if (!packOp.getOuterDimsPerm().empty())
inverseOuterDimsPerm = invertPermutationVector(packOp.getOuterDimsPerm());
@@ -5197,7 +5196,7 @@ static bool inferStaticShape(UnPackOp op, SmallVectorImpl<int64_t> &srcShape,
destShape.assign(op.getDestType().getShape().begin(),
op.getDestType().getShape().end());
llvm::SmallSetVector<int64_t, 4> innerDims;
- innerDims.insert(op.getInnerDimsPos().begin(), op.getInnerDimsPos().end());
+ innerDims.insert_range(op.getInnerDimsPos());
SmallVector<int64_t> inverseOuterDimsPerm;
if (!op.getOuterDimsPerm().empty())
inverseOuterDimsPerm = invertPermutationVector(op.getOuterDimsPerm());
diff --git a/mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp b/mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp
index ef7b4757a04b4..5d65e7e4666c3 100644
--- a/mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp
+++ b/mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp
@@ -1251,8 +1251,7 @@ transform::MatchOp::apply(transform::TransformRewriter &rewriter,
transform::TransformState &state) {
llvm::StringSet<> strs;
if (getOps().has_value())
- strs.insert(getOps()->getAsValueRange<StringAttr>().begin(),
- getOps()->getAsValueRange<StringAttr>().end());
+ strs.insert_range(getOps()->getAsValueRange<StringAttr>());
auto payloadOps = state.getPayloadOps(getTarget());
if (!llvm::hasSingleElement(payloadOps)) {
diff --git a/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp b/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp
index 927ce066038d8..d6b7ab0a980eb 100644
--- a/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp
+++ b/mlir/lib/Dialect/Linalg/Transforms/ElementwiseOpFusion.cpp
@@ -1205,7 +1205,7 @@ bool mlir::linalg::isDimSequencePreserved(AffineMap indexingMap,
"expected indexing map to be projected permutation");
llvm::SmallDenseSet<unsigned, 4> sequenceElements;
- sequenceElements.insert(dimSequence.begin(), dimSequence.end());
+ sequenceElements.insert_range(dimSequence);
unsigned dimSequenceStart = dimSequence[0];
for (const auto &expr : enumerate(indexingMap.getResults())) {
@@ -1381,8 +1381,7 @@ getCollapsableIterationSpaceDims(GenericOp genericOp, OpOperand *fusableOperand,
}))
continue;
- processedIterationDims.insert(foldedIterationSpaceDims.begin(),
- foldedIterationSpaceDims.end());
+ processedIterationDims.insert_range(foldedIterationSpaceDims);
iterationSpaceReassociation.emplace_back(
std::move(foldedIterationSpaceDims));
}
diff --git a/mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp b/mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp
index adde8a66d8354..bcff5d32a3d2c 100644
--- a/mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp
+++ b/mlir/lib/Dialect/Utils/StructuredOpsUtils.cpp
@@ -219,7 +219,7 @@ Operation *mlir::cloneWithoutRegions(OpBuilder &b, Operation *op,
SmallVector<NamedAttribute>
mlir::getPrunedAttributeList(Operation *op, ArrayRef<StringRef> elidedAttrs) {
llvm::StringSet<> elidedAttrsSet;
- elidedAttrsSet.insert(elidedAttrs.begin(), elidedAttrs.end());
+ elidedAttrsSet.insert_range(elidedAttrs);
SmallVector<NamedAttribute> attrs;
for (auto attr : op->getAttrs()) {
if (elidedAttrsSet.count(attr.getName()))
diff --git a/mlir/lib/Dialect/Vector/IR/VectorOps.cpp b/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
index 8e0e723cf4ed3..d4c1da30d498d 100644
--- a/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
+++ b/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
@@ -838,7 +838,7 @@ void ContractionOp::print(OpAsmPrinter &p) {
// TODO: Unify printing code with linalg ops.
auto attrNames = getTraitAttrNames();
llvm::StringSet<> traitAttrsSet;
- traitAttrsSet.insert(attrNames.begin(), attrNames.end());
+ traitAttrsSet.insert_range(attrNames);
SmallVector<NamedAttribute, 8> attrs;
for (auto attr : (*this)->getAttrs()) {
if (attr.getName() == getIteratorTypesAttrName()) {
diff --git a/mlir/lib/IR/Location.cpp b/mlir/lib/IR/Location.cpp
index 7a4df4fbd46d9..506a6c1fc16fb 100644
--- a/mlir/lib/IR/Location.cpp
+++ b/mlir/lib/IR/Location.cpp
@@ -219,8 +219,7 @@ Location FusedLoc::get(ArrayRef<Location> locs, Attribute metadata,
if (fusedLoc.getMetadata() == metadata) {
// UnknownLoc's have already been removed from FusedLocs so we can
// simply add all of the internal locations.
- decomposedLocs.insert(fusedLoc.getLocations().begin(),
- fusedLoc.getLocations().end());
+ decomposedLocs.insert_range(fusedLoc.getLocations());
continue;
}
}
diff --git a/mlir/lib/Rewrite/FrozenRewritePatternSet.cpp b/mlir/lib/Rewrite/FrozenRewritePatternSet.cpp
index c022ff4b8da49..e706d95855435 100644
--- a/mlir/lib/Rewrite/FrozenRewritePatternSet.cpp
+++ b/mlir/lib/Rewrite/FrozenRewritePatternSet.cpp
@@ -65,10 +65,8 @@ FrozenRewritePatternSet::FrozenRewritePatternSet(
ArrayRef<std::string> enabledPatternLabels)
: impl(std::make_shared<Impl>()) {
DenseSet<StringRef> disabledPatterns, enabledPatterns;
- disabledPatterns.insert(disabledPatternLabels.begin(),
- disabledPatternLabels.end());
- enabledPatterns.insert(enabledPatternLabels.begin(),
- enabledPatternLabels.end());
+ disabledPatterns.insert_range(disabledPatternLabels);
+ enabledPatterns.insert_range(enabledPatternLabels);
// Functor used to walk all of the operations registered in the context. This
// is useful for patterns that get applied to multiple operations, such as
diff --git a/mlir/lib/Target/LLVMIR/ModuleImport.cpp b/mlir/lib/Target/LLVMIR/ModuleImport.cpp
index a07189ae1323c..76460b541fe0e 100644
--- a/mlir/lib/Target/LLVMIR/ModuleImport.cpp
+++ b/mlir/lib/Target/LLVMIR/ModuleImport.cpp
@@ -152,7 +152,7 @@ getTopologicallySortedBlocks(ArrayRef<llvm::BasicBlock *> basicBlocks) {
for (llvm::BasicBlock *basicBlock : basicBlocks) {
if (!blocks.contains(basicBlock)) {
llvm::ReversePostOrderTraversal<llvm::BasicBlock *> traversal(basicBlock);
- blocks.insert(traversal.begin(), traversal.end());
+ blocks.insert_range(traversal);
}
}
assert(blocks.size() == basicBlocks.size() && "some blocks are not sorted");
diff --git a/mlir/lib/Transforms/Mem2Reg.cpp b/mlir/lib/Transforms/Mem2Reg.cpp
index d339073771fb5..cf039c328e07f 100644
--- a/mlir/lib/Transforms/Mem2Reg.cpp
+++ b/mlir/lib/Transforms/Mem2Reg.cpp
@@ -408,7 +408,7 @@ void MemorySlotPromotionAnalyzer::computeMergePoints(
SmallVector<Block *> mergePointsVec;
idfCalculator.calculate(mergePointsVec);
- mergePoints.insert(mergePointsVec.begin(), mergePointsVec.end());
+ mergePoints.insert_range(mergePointsVec);
}
bool MemorySlotPromotionAnalyzer::areMergePointsUsable(
diff --git a/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp b/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
index fe84c61300646..8fe7d87c71767 100644
--- a/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
+++ b/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
@@ -961,11 +961,11 @@ MultiOpPatternRewriteDriver::MultiOpPatternRewriteDriver(
: GreedyPatternRewriteDriver(ctx, patterns, config),
survivingOps(survivingOps) {
if (config.strictMode != GreedyRewriteStrictness::AnyOp)
- strictModeFilteredOps.insert(ops.begin(), ops.end());
+ strictModeFilteredOps.insert_range(ops);
if (survivingOps) {
survivingOps->clear();
- survivingOps->insert(ops.begin(), ops.end());
+ survivingOps->insert_range(ops);
}
}
diff --git a/mlir/test/lib/Analysis/DataFlow/TestDenseDataFlowAnalysis.h b/mlir/test/lib/Analysis/DataFlow/TestDenseDataFlowAnalysis.h
index 86eb8651cb90c..6012c90f84539 100644
--- a/mlir/test/lib/Analysis/DataFlow/TestDenseDataFlowAnalysis.h
+++ b/mlir/test/lib/Analysis/DataFlow/TestDenseDataFlowAnalysis.h
@@ -76,7 +76,7 @@ class AdjacentAccess {
}
size_t sizeBefore = accesses.size();
- accesses.insert(other.accesses.begin(), other.accesses.end());
+ accesses.insert_range(other.accesses);
return accesses.size() == sizeBefore ? ChangeResult::NoChange
: ChangeResult::Change;
}
diff --git a/mlir/test/lib/Analysis/DataFlow/TestSparseBackwardDataFlowAnalysis.cpp b/mlir/test/lib/Analysis/DataFlow/TestSparseBackwardDataFlowAnalysis.cpp
index 1f3cab1e02adb..4b02865b6ae19 100644
--- a/mlir/test/lib/Analysis/DataFlow/TestSparseBackwardDataFlowAnalysis.cpp
+++ b/mlir/test/lib/Analysis/DataFlow/TestSparseBackwardDataFlowAnalysis.cpp
@@ -41,7 +41,7 @@ struct WrittenToLatticeValue {
ChangeResult addWrites(const SetVector<StringAttr> &writes) {
int sizeBefore = this->writes.size();
- this->writes.insert(writes.begin(), writes.end());
+ this->writes.insert_range(writes);
int sizeAfter = this->writes.size();
return sizeBefore == sizeAfter ? ChangeResult::NoChange
: ChangeResult::Change;
diff --git a/mlir/test/lib/Dialect/Linalg/TestLinalgElementwiseFusion.cpp b/mlir/test/lib/Dialect/Linalg/TestLinalgElementwiseFusion.cpp
index 34a396f18f90e..cb215197253bb 100644
--- a/mlir/test/lib/Dialect/Linalg/TestLinalgElementwiseFusion.cpp
+++ b/mlir/test/lib/Dialect/Linalg/TestLinalgElementwiseFusion.cpp
@@ -27,7 +27,7 @@ static void addOperands(Operation *op, SetVector<Value> &operandSet) {
TypeSwitch<Operation *, void>(op)
.Case<linalg::LinalgOp>([&](linalg::LinalgOp linalgOp) {
SmallVector<Value> inputOperands = linalgOp.getDpsInputs();
- operandSet.insert(inputOperands.begin(), inputOperands.end());
+ operandSet.insert_range(inputOperands);
})
.Default([&](Operation *operation) {
operandSet.insert(operation->operand_begin(), operation->operand_end());
diff --git a/mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp b/mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp
index 74838bc0ca2fb..c9bb34b692dd0 100644
--- a/mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp
+++ b/mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp
@@ -753,7 +753,7 @@ struct TestCreateVectorBroadcast
cast<DenseI64ArrayAttr>(op->getDiscardableAttr("broadcast_dims"))
.asArrayRef();
llvm::SetVector<int64_t> broadcastedDims;
- broadcastedDims.insert(arrayAttr.begin(), arrayAttr.end());
+ broadcastedDims.insert_range(arrayAttr);
OpBuilder b(op);
Value bcast = vector::BroadcastOp::createOrFoldBroadcastOp(
b, op->getOperand(0), targetShape, broadcastedDims);
diff --git a/mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp b/mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp
index 6a39424bd463f..4ab3e9250c29d 100644
--- a/mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp
+++ b/mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp
@@ -551,8 +551,7 @@ void DefGen::emitTraitMethods(const InterfaceTrait &trait) {
// Get the set of methods that should always be declared.
auto alwaysDeclaredMethods = trait.getAlwaysDeclaredMethods();
StringSet<> alwaysDeclared;
- alwaysDeclared.insert(alwaysDeclaredMethods.begin(),
- alwaysDeclaredMethods.end());
+ alwaysDeclared.insert_range(alwaysDeclaredMethods);
Interface iface = trait.getInterface(); // causes strange bugs if elided
for (auto &method : iface.getMethods()) {
diff --git a/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp b/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
index b957c8ee9f8ab..ad45376a4fa44 100644
--- a/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
+++ b/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
@@ -3570,8 +3570,7 @@ void OpEmitter::genOpInterfaceMethods(const tblgen::InterfaceTrait *opTrait) {
// Get the set of methods that should always be declared.
auto alwaysDeclaredMethodsVec = opTrait->getAlwaysDeclaredMethods();
llvm::StringSet<> alwaysDeclaredMethods;
- alwaysDeclaredMethods.insert(alwaysDeclaredMethodsVec.begin(),
- alwaysDeclaredMethodsVec.end());
+ alwaysDeclaredMethods.insert_range(alwaysDeclaredMethodsVec);
for (const InterfaceMethod &method : interface.getMethods()) {
// Don't declare if the method has a body.
More information about the Mlir-commits
mailing list