commit 68c1f010d28431800451cb5bc21433f7f412f237
author Ted Kremenek <kremenek@apple.com> Wed Aug 05 05:31:02 2009 +0000
committer Ted Kremenek <kremenek@apple.com> Wed Aug 05 05:31:02 2009 +0000
tree 5caba0c3bed108b205b5f837c2700dd1fe6e30aa
parent c58e5699be58c3e68c7bdf312c381789becd6df2

Fix a bug in RegionStoreSubRegionManager::add() where multiple subregions wouldn't correctly get registered in the SubRegion map.

llvm-svn: 78162

clang/lib/Analysis/RegionStore.cpp

diff --git a/clang/lib/Analysis/RegionStore.cpp b/clang/lib/Analysis/RegionStore.cpp
index c47aaa2..7020175 100644
--- a/clang/lib/Analysis/RegionStore.cpp
+++ b/clang/lib/Analysis/RegionStore.cpp
@@ -121,8 +121,10 @@
 public:
   void add(const MemRegion* Parent, const MemRegion* SubRegion) {
     Map::iterator I = M.find(Parent);
-    M.insert(std::make_pair(Parent, 
-             F.Add(I == M.end() ? F.GetEmptySet() : I->second, SubRegion)));
+    if (I == M.end())
+      M.insert(std::make_pair(Parent, F.Add(F.GetEmptySet(), SubRegion)));
+    else
+      I->second = F.Add(I->second, SubRegion);
   }
     
   ~RegionStoreSubRegionMap() {}