[clang] f093ccb - [StaticAnalyzer] Remove FIXME from InnerPointerBRVisitor (#186303)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 13 01:30:57 PDT 2026
Author: Benedek Kaibas
Date: 2026-03-13T08:30:51Z
New Revision: f093ccb2ebba424c57de9c482786d1a2cf43125c
URL: https://github.com/llvm/llvm-project/commit/f093ccb2ebba424c57de9c482786d1a2cf43125c
DIFF: https://github.com/llvm/llvm-project/commit/f093ccb2ebba424c57de9c482786d1a2cf43125c.diff
LOG: [StaticAnalyzer] Remove FIXME from InnerPointerBRVisitor (#186303)
Removed the FIXME based on the discussion in #185796 . The code path
only happens during bug reporting therefore it is not performance
sensitive.
Added:
Modified:
clang/lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp
Removed:
################################################################################
diff --git a/clang/lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp
index 42d58159bc3e0..d40c2ed346cf4 100644
--- a/clang/lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp
+++ b/clang/lib/StaticAnalyzer/Checkers/InnerPointerChecker.cpp
@@ -76,8 +76,6 @@ class InnerPointerChecker
BugReporterContext &BRC,
PathSensitiveBugReport &BR) override;
- // FIXME: Scan the map once in the visitor's constructor and do a direct
- // lookup by region.
bool isSymbolTracked(ProgramStateRef State, SymbolRef Sym) {
RawPtrMapTy Map = State->get<RawPtrMap>();
for (const auto &Entry : Map) {
More information about the cfe-commits
mailing list