[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