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

Reply via email to