retain checker: Don't bother using a FoldingSet to unique summaries.
We never compare summaries by their pointers, and we create only a
handful of them when analyzing a given function.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@70824 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Analysis/CFRefCount.cpp b/lib/Analysis/CFRefCount.cpp
index 58f581a..505ec7d 100644
--- a/lib/Analysis/CFRefCount.cpp
+++ b/lib/Analysis/CFRefCount.cpp
@@ -530,9 +530,6 @@
// Typedefs.
//==-----------------------------------------------------------------==//
- typedef llvm::FoldingSet<RetainSummary>
- SummarySetTy;
-
typedef llvm::DenseMap<FunctionDecl*, RetainSummary*>
FuncSummariesTy;
@@ -551,10 +548,7 @@
/// GCEnabled - Records whether or not the analyzed code runs in GC mode.
const bool GCEnabled;
-
- /// SummarySet - A FoldingSet of uniqued summaries.
- SummarySetTy SummarySet;
-
+
/// FuncSummaries - A map from FunctionDecls to summaries.
FuncSummariesTy FuncSummaries;
@@ -782,25 +776,10 @@
RetainSummaryManager::getPersistentSummary(ArgEffects AE, RetEffect RetEff,
ArgEffect ReceiverEff,
ArgEffect DefaultEff,
- bool isEndPath) {
-
- // Generate a profile for the summary.
- llvm::FoldingSetNodeID profile;
- RetainSummary::Profile(profile, AE, RetEff, DefaultEff, ReceiverEff,
- isEndPath);
-
- // Look up the uniqued summary, or create one if it doesn't exist.
- void* InsertPos;
- RetainSummary* Summ = SummarySet.FindNodeOrInsertPos(profile, InsertPos);
-
- if (Summ)
- return Summ;
-
+ bool isEndPath) {
// Create the summary and return it.
- Summ = (RetainSummary*) BPAlloc.Allocate<RetainSummary>();
+ RetainSummary *Summ = (RetainSummary*) BPAlloc.Allocate<RetainSummary>();
new (Summ) RetainSummary(AE, RetEff, DefaultEff, ReceiverEff, isEndPath);
- SummarySet.InsertNode(Summ, InsertPos);
-
return Summ;
}