[clang] Use range constructors of *Set (NFC)

[kazutakahirata]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-codegen

@llvm/pr-subscribers-clang-analysis

Author: Kazu Hirata (kazutakahirata)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/137574.diff

5 Files Affected:

• (modified) clang/lib/Analysis/FlowSensitive/DataflowAnalysisContext.cpp (+1-1)
• (modified) clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp (+3-4)
• (modified) clang/lib/Sema/SemaTemplateVariadic.cpp (+1-1)
• (modified) clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp (+1-1)
• (modified) clang/unittests/Analysis/FlowSensitive/SimplifyConstraintsTest.cpp (+1-2)

diff --git a/clang/lib/Analysis/FlowSensitive/DataflowAnalysisContext.cpp b/clang/lib/Analysis/FlowSensitive/DataflowAnalysisContext.cpp index 1c4fe5c6d5019..6409b4bc8ff4d 100644 --- a/clang/lib/Analysis/FlowSensitive/DataflowAnalysisContext.cpp +++ b/clang/lib/Analysis/FlowSensitive/DataflowAnalysisContext.cpp @@ -86,7 +86,7 @@ StorageLocation &DataflowAnalysisContext::createStorageLocation(QualType Type) { // Can't use `StringSet` as the return type as it doesn't support `operator==`. template <typename T> static llvm::DenseSet<llvm::StringRef> getKeys(const llvm::StringMap<T> &Map) { - return llvm::DenseSet<llvm::StringRef>(Map.keys().begin(), Map.keys().end()); + return llvm::DenseSet<llvm::StringRef>(llvm::from_range, Map.keys()); } RecordStorageLocation &DataflowAnalysisContext::createRecordStorageLocation( diff --git a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp index f697c13f4c522..59a5f7b914ce5 100644 --- a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp +++ b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp @@ -129,8 +129,8 @@ static RecordDecl *buildRecordForGlobalizedVars( // }; RecordDecl *GlobalizedRD = C.buildImplicitRecord("_globalized_locals_ty"); GlobalizedRD->startDefinition(); - llvm::SmallPtrSet<const ValueDecl *, 16> SingleEscaped( - EscapedDeclsForTeams.begin(), EscapedDeclsForTeams.end()); + llvm::SmallPtrSet<const ValueDecl *, 16> SingleEscaped(llvm::from_range, + EscapedDeclsForTeams); for (const auto &Pair : GlobalizedVars) { const ValueDecl *VD = Pair.second; QualType Type = VD->getType(); @@ -322,8 +322,7 @@ class CheckVarsEscapingDeclContext final public: CheckVarsEscapingDeclContext(CodeGenFunction &CGF, ArrayRef<const ValueDecl *> TeamsReductions) - : CGF(CGF), EscapedDecls(TeamsReductions.begin(), TeamsReductions.end()) { - } + : CGF(CGF), EscapedDecls(llvm::from_range, TeamsReductions) {} virtual ~CheckVarsEscapingDeclContext() = default; void VisitDeclStmt(const DeclStmt *S) { if (!S) diff --git a/clang/lib/Sema/SemaTemplateVariadic.cpp b/clang/lib/Sema/SemaTemplateVariadic.cpp index 87682233c5246..34fec069d6df6 100644 --- a/clang/lib/Sema/SemaTemplateVariadic.cpp +++ b/clang/lib/Sema/SemaTemplateVariadic.cpp @@ -498,7 +498,7 @@ bool Sema::DiagnoseUnexpandedParameterPackInRequiresExpr(RequiresExpr *RE) { // We only care about unexpanded references to the RequiresExpr's own // parameter packs. auto Parms = RE->getLocalParameters(); - llvm::SmallPtrSet<NamedDecl*, 8> ParmSet(Parms.begin(), Parms.end()); + llvm::SmallPtrSet<NamedDecl *, 8> ParmSet(llvm::from_range, Parms); SmallVector<UnexpandedParameterPack, 2> UnexpandedParms; for (auto Parm : Unexpanded) if (ParmSet.contains(Parm.first.dyn_cast<NamedDecl *>())) diff --git a/clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp b/clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp index 90c95d5cf60d8..b88e6db7cceb7 100644 --- a/clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp +++ b/clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp @@ -77,7 +77,7 @@ static int countSymbols(Lang Language) { } } #undef SYMBOL - return llvm::DenseSet<StringRef>(Symbols.begin(), Symbols.end()).size(); + return llvm::DenseSet<StringRef>(llvm::from_range, Symbols).size(); } static int initialize(Lang Language) { diff --git a/clang/unittests/Analysis/FlowSensitive/SimplifyConstraintsTest.cpp b/clang/unittests/Analysis/FlowSensitive/SimplifyConstraintsTest.cpp index 1f34ae076d5ed..c82199c15e068 100644 --- a/clang/unittests/Analysis/FlowSensitive/SimplifyConstraintsTest.cpp +++ b/clang/unittests/Analysis/FlowSensitive/SimplifyConstraintsTest.cpp @@ -24,8 +24,7 @@ class SimplifyConstraintsTest : public ::testing::Test { protected: llvm::SetVector<const Formula *> parse(StringRef Lines) { std::vector<const Formula *> formulas = test::parseFormulas(A, Lines); - llvm::SetVector<const Formula *> Constraints(formulas.begin(), - formulas.end()); + llvm::SetVector<const Formula *> Constraints(llvm::from_range, formulas); return Constraints; }