maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

When we're deserializing declarations lexically stored in a RecordDecl 

after having already deserialized the fields, clear out the fields
first. This makes sure that we keep all of the declarations in the
lexical context (including those implicitly added by later
type-checking) within the same list. A test case for this behavior is
coming as part of another commit; testing for this problem in
isolation is a nightmare.

llvm-svn: 138661
This commit is contained in:
Douglas Gregor 2011-08-26 21:23:06 +00:00
parent 5e5704277f
commit f337ae9477
1 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
clang/lib/AST/DeclBase.cpp
View File

@ -841,6 +841,22 @@ DeclContext::LoadLexicalDeclsFromExternalStorage() const {
// Notify that we have a DeclContext that is initializing.
ExternalASTSource::Deserializing ADeclContext(Source);
// We may have already loaded just the fields of this record, in which case
// we remove all of the fields from the list. The fields will be reloaded
// from the external source as part of re-establishing the context.
if (const RecordDecl *RD = dyn_cast<RecordDecl>(this)) {
if (RD->LoadedFieldsFromExternalStorage) {
while (FirstDecl && isa<FieldDecl>(FirstDecl)) {
Decl *Next = FirstDecl->NextDeclInContext;
FirstDecl->NextDeclInContext = 0;
FirstDecl = Next;
}
if (!FirstDecl)
LastDecl = 0;
}
}
// Load the external declarations, if any.
SmallVector<Decl*, 64> Decls;
ExternalLexicalStorage = false;
@ -856,14 +872,6 @@ DeclContext::LoadLexicalDeclsFromExternalStorage() const {
if (Decls.empty())
return;
// We may have already loaded just the fields of this record, in which case
// don't add the decls, just replace the FirstDecl/LastDecl chain.
if (const RecordDecl *RD = dyn_cast<RecordDecl>(this))
if (RD->LoadedFieldsFromExternalStorage) {
llvm::tie(FirstDecl, LastDecl) = BuildDeclChain(Decls);
return;
}
// Splice the newly-read declarations into the beginning of the list
// of declarations.
Decl *ExternalFirst, *ExternalLast;