Skip to content

Destroy RecursiveImportCacheRepository after PersistentSymbolTableRepo

Igor Kushnir requested to merge work/persistent-symbol-table-cache-fixes into master

I don't have an AddressSanitizer setup, so cannot verify that the alternative crash fix in the first commit actually works.

The second commit is tentative. It may be wrong. For example, if PersistentSymbolTable::visitFilteredDeclarations() iterates over an empty declaration set during the cleanup and shutdown. I couldn't check this because visitFilteredDeclarations() was never called after the removed line in DUChainPrivate::doMoreCleanup() in my few test runs.

Merge request reports