llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-temporal-safety Author: Kazu Hirata (kazutakahirata) <details> <summary>Changes</summary> Identified with readability-container-contains. --- Full diff: https://github.com/llvm/llvm-project/pull/167186.diff 1 Files Affected: - (modified) clang/lib/Analysis/LifetimeSafety/Facts.cpp (+1-2) ``````````diff diff --git a/clang/lib/Analysis/LifetimeSafety/Facts.cpp b/clang/lib/Analysis/LifetimeSafety/Facts.cpp index 4a4172fe55bf3..190c038f46401 100644 --- a/clang/lib/Analysis/LifetimeSafety/Facts.cpp +++ b/clang/lib/Analysis/LifetimeSafety/Facts.cpp @@ -68,8 +68,7 @@ llvm::StringMap<ProgramPoint> FactManager::getTestPoints() const { for (const Fact *F : BlockFacts) { if (const auto *TPF = F->getAs<TestPointFact>()) { StringRef PointName = TPF->getAnnotation(); - assert(AnnotationToPointMap.find(PointName) == - AnnotationToPointMap.end() && + assert(!AnnotationToPointMap.contains(PointName) && "more than one test points with the same name"); AnnotationToPointMap[PointName] = F; } `````````` </details> https://github.com/llvm/llvm-project/pull/167186 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
