gamzinav/cling
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Implement sanity check, that ensures no lookup entries were left over.

Browse Source
This commit is contained in:
Vassil Vassilev 2013-10-29 22:33:56 -05:00 committed by sftnight
parent 688d82d64f
commit 8ea52103a0
1 changed files with 19 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
lib/Interpreter/ASTNodeEraser.cpp
View File

@ -310,13 +310,26 @@ namespace cling {
m_Sema->IdResolver.RemoveDecl(ND);
}
// Find other decls that the old one has replaced
#ifndef NDEBUG
StoredDeclsMap *Map = DC->getPrimaryContext()->getLookupPtr();
if (!Map)
return false;
StoredDeclsMap::iterator Pos = Map->find(ND->getDeclName());
assert((ND->isHidden() || Pos != Map->end())
&& "no lookup entry for decl");
if (Map) { // DeclContexts like EnumDecls don't have lookup maps.
// Make sure we the decl doesn't exist in the lookup tables.
StoredDeclsMap::iterator Pos = Map->find(ND->getDeclName());
// Most decls only have one entry in their list, special case it.
if (NamedDecl *OldD = Pos->second.getAsDecl())
assert(OldD != ND && "Lookup entry still exists.");
else if (StoredDeclsList::DeclsTy* Vec = Pos->second.getAsVector()) {
// Otherwise iterate over the list with entries with the same name.
// TODO: Walk the redeclaration chain if the entry was a redeclaration.
for (StoredDeclsList::DeclsTy::const_iterator I = Vec->begin(),
E = Vec->end(); I != E; ++I)
assert(*I != ND && "Lookup entry still exists.");
}
else
assert(Pos->second.isNull() && "!?");
}
#endif
return Successful;
}