balazske wrote: > Does this have any affect on other redeclarable `DeclContext`s such as class > forward declarations?
In the code of `DeclContext::getPrimaryContext` it is visible that it has only effect on declarations and it depends on whether it has a definition or not (except namespace and translation unit and some ObjC things). So I think the only case when there is change compared to the previous (except namespace) is when a definition of a decl is imported, because `getPrimaryContext` will return a different object (after a definition is imported). For namespace the new lookup is added to `ASTImporter`. At translation unit there should be no problem. https://github.com/llvm/llvm-project/pull/118466 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits