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(); + ---------------- 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.) 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
