From 21b719dfcf35a5d139d7eafaab0b3c2c7527ffa7 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Wed, 30 Apr 2014 23:46:27 +0000 Subject: LexicalScopes: Use unique_ptr to manage ownership of abstract LexicalScopes. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207726 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/LexicalScopes.cpp | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) (limited to 'lib/CodeGen/LexicalScopes.cpp') diff --git a/lib/CodeGen/LexicalScopes.cpp b/lib/CodeGen/LexicalScopes.cpp index cdc2cb2cc5..d4f3b70995 100644 --- a/lib/CodeGen/LexicalScopes.cpp +++ b/lib/CodeGen/LexicalScopes.cpp @@ -33,7 +33,6 @@ LexicalScopes::~LexicalScopes() { reset(); } void LexicalScopes::reset() { MF = nullptr; CurrentFnLexicalScope = nullptr; - DeleteContainerSeconds(AbstractScopeMap); InlinedLexicalScopeMap.clear(); AbstractScopesList.clear(); } @@ -194,9 +193,11 @@ LexicalScope *LexicalScopes::getOrCreateAbstractScope(const MDNode *N) { DIDescriptor Scope(N); if (Scope.isLexicalBlockFile()) Scope = DILexicalBlockFile(Scope).getScope(); - LexicalScope *AScope = AbstractScopeMap.lookup(N); - if (AScope) - return AScope; + auto IterBool = AbstractScopeMap.insert( + std::make_pair(N, std::unique_ptr())); + auto &MapEntry = *IterBool.first; + if (!IterBool.second) + return MapEntry.second.get(); LexicalScope *Parent = nullptr; if (Scope.isLexicalBlock()) { @@ -204,11 +205,11 @@ LexicalScope *LexicalScopes::getOrCreateAbstractScope(const MDNode *N) { DIDescriptor ParentDesc = DB.getContext(); Parent = getOrCreateAbstractScope(ParentDesc); } - AScope = new LexicalScope(Parent, DIDescriptor(N), nullptr, true); - AbstractScopeMap[N] = AScope; + MapEntry.second = + make_unique(Parent, DIDescriptor(N), nullptr, true); if (DIDescriptor(N).isSubprogram()) - AbstractScopesList.push_back(AScope); - return AScope; + AbstractScopesList.push_back(MapEntry.second.get()); + return MapEntry.second.get(); } /// constructScopeNest -- cgit v1.2.3