From a880b66d6e0e446501fcbc27b87a1ec0e4ecde4c Mon Sep 17 00:00:00 2001 From: Ted Kremenek Date: Wed, 4 Mar 2009 22:53:46 +0000 Subject: [PATCH] Add an optional "tag" to conjured symbols that allows us to distinguish between multiple symbols conjured at the same location. All that is required of the tag is that it is a fixed void* value that points to an memory address that remains valid throughout the remainder of the lifetime of the SymbolManager. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@66092 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../Analysis/PathSensitive/SymbolManager.h | 32 ++++++++++++------- lib/Analysis/SymbolManager.cpp | 7 ++-- 2 files changed, 24 insertions(+), 15 deletions(-) diff --git a/include/clang/Analysis/PathSensitive/SymbolManager.h b/include/clang/Analysis/PathSensitive/SymbolManager.h index 91d3b348b9..62dadd338f 100644 --- a/include/clang/Analysis/PathSensitive/SymbolManager.h +++ b/include/clang/Analysis/PathSensitive/SymbolManager.h @@ -139,28 +139,33 @@ class SymbolConjured : public SymbolData { Stmt* S; QualType T; unsigned Count; + const void* SymbolTag; public: - SymbolConjured(SymbolRef Sym, Stmt* s, QualType t, unsigned count) - : SymbolData(ConjuredKind, Sym), S(s), T(t), Count(count) {} + SymbolConjured(SymbolRef Sym, Stmt* s, QualType t, unsigned count, + const void* symbolTag) + : SymbolData(ConjuredKind, Sym), S(s), T(t), Count(count), + SymbolTag(symbolTag) {} Stmt* getStmt() const { return S; } - unsigned getCount() const { return Count; } + unsigned getCount() const { return Count; } + const void* getTag() const { return SymbolTag; } + QualType getType(ASTContext&) const; - static void Profile(llvm::FoldingSetNodeID& profile, - Stmt* S, QualType T, unsigned Count) { - + static void Profile(llvm::FoldingSetNodeID& profile, Stmt* S, QualType T, + unsigned Count, const void* SymbolTag) { profile.AddInteger((unsigned) ConjuredKind); profile.AddPointer(S); profile.Add(T); profile.AddInteger(Count); + profile.AddPointer(SymbolTag); } - + virtual void Profile(llvm::FoldingSetNodeID& profile) { - Profile(profile, S, T, Count); + Profile(profile, S, T, Count, SymbolTag); } - + // Implement isa support. static inline bool classof(const SymbolData* D) { return D->getKind() == ConjuredKind; @@ -217,9 +222,12 @@ public: /// Make a unique symbol for MemRegion R according to its kind. SymbolRef getRegionRValueSymbol(const MemRegion* R); - SymbolRef getConjuredSymbol(Stmt* E, QualType T, unsigned VisitCount); - SymbolRef getConjuredSymbol(Expr* E, unsigned VisitCount) { - return getConjuredSymbol(E, E->getType(), VisitCount); + SymbolRef getConjuredSymbol(Stmt* E, QualType T, unsigned VisitCount, + const void* SymbolTag = 0); + + SymbolRef getConjuredSymbol(Expr* E, unsigned VisitCount, + const void* SymbolTag = 0) { + return getConjuredSymbol(E, E->getType(), VisitCount, SymbolTag); } const SymbolData& getSymbolData(SymbolRef ID) const; diff --git a/lib/Analysis/SymbolManager.cpp b/lib/Analysis/SymbolManager.cpp index 45e1aae23b..589178fecb 100644 --- a/lib/Analysis/SymbolManager.cpp +++ b/lib/Analysis/SymbolManager.cpp @@ -52,10 +52,11 @@ SymbolRef SymbolManager::getRegionRValueSymbol(const MemRegion* R) { return SymbolCounter++; } -SymbolRef SymbolManager::getConjuredSymbol(Stmt* E, QualType T, unsigned Count){ +SymbolRef SymbolManager::getConjuredSymbol(Stmt* E, QualType T, unsigned Count, + const void* SymbolTag) { llvm::FoldingSetNodeID profile; - SymbolConjured::Profile(profile, E, T, Count); + SymbolConjured::Profile(profile, E, T, Count, SymbolTag); void* InsertPos; SymbolData* SD = DataSet.FindNodeOrInsertPos(profile, InsertPos); @@ -64,7 +65,7 @@ SymbolRef SymbolManager::getConjuredSymbol(Stmt* E, QualType T, unsigned Count){ return SD->getSymbol(); SD = (SymbolData*) BPAlloc.Allocate(); - new (SD) SymbolConjured(SymbolCounter, E, T, Count); + new (SD) SymbolConjured(SymbolCounter, E, T, Count, SymbolTag); DataSet.InsertNode(SD, InsertPos); DataMap[SymbolCounter] = SD; -- 2.40.0