balazske marked an inline comment as done. balazske added inline comments.
================ Comment at: clang/lib/CrossTU/CrossTranslationUnit.cpp:723 + // Parent map is invalidated after changing the AST. + ToDecl->getASTContext().getParentMapContext().clear(); + ---------------- balazske wrote: > martong wrote: > > > ... the TU is modified by getCrossTUDefinition the parent map does not > > contain the newly imported objects and has to be re-created. > > > > I see we clear it here, but when/where will be the parent map re-created? > It should be created at any access to it. (Actually I did not check how this > works after the test showed that the fix works.) `ASTContext::getParentMapContext` does the job. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82568/new/ https://reviews.llvm.org/D82568 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
