[clang] [libclang] Avoid repeated hash lookups (NFC) (PR #111087)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 3 19:58:02 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/111087.diff
1 Files Affected:
- (modified) clang/tools/libclang/CXIndexDataConsumer.cpp (+2-8)
``````````diff
diff --git a/clang/tools/libclang/CXIndexDataConsumer.cpp b/clang/tools/libclang/CXIndexDataConsumer.cpp
index 8d364ed8876a12..ced94e13baf12e 100644
--- a/clang/tools/libclang/CXIndexDataConsumer.cpp
+++ b/clang/tools/libclang/CXIndexDataConsumer.cpp
@@ -952,18 +952,12 @@ void CXIndexDataConsumer::addContainerInMap(const DeclContext *DC,
if (!DC)
return;
- ContainerMapTy::iterator I = ContainerMap.find(DC);
- if (I == ContainerMap.end()) {
- if (container)
- ContainerMap[DC] = container;
- return;
- }
// Allow changing the container of a previously seen DeclContext so we
// can handle invalid user code, like a function re-definition.
if (container)
- I->second = container;
+ ContainerMap[DC] = container;
else
- ContainerMap.erase(I);
+ ContainerMap.erase(DC);
}
CXIdxClientEntity CXIndexDataConsumer::getClientEntity(const Decl *D) const {
``````````
</details>
https://github.com/llvm/llvm-project/pull/111087
More information about the cfe-commits
mailing list