[clang] [LifetimeSafety] Use StringMap::contains (NFC) (PR #167186)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Nov 8 15:04:34 PST 2025
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
More information about the cfe-commits
mailing list