Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)
http://clang.llvm.org/
Other
15 stars 10 forks source link

[ASTImporter] Using most recent previous decl at importRecordDecl. #520

Closed balazske closed 5 years ago

balazske commented 5 years ago

The object found in ASTImporterLookupTable is probably not the most recent one. Passing not the most recent as PreviousDecl when create the new record can cause problems. One failure (#519) seems to be fixed by this change, but not totally sure (the error appears "sporadically").

balazske commented 5 years ago

In commit message: should be "VisitRecordDecl"

martong commented 5 years ago

This was a good catch. :+1:

martong commented 5 years ago

https://reviews.llvm.org/D53655 brings this one too.