[PATCH] D83972: Modify ImportDefiniton for ObjCInterfaceDecl so that we always the ImportDeclContext one we start the definition
    Gabor Marton via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Jul 24 00:45:46 PDT 2020
    
    
  
martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.
> Unlike RecordDecl case which uses DefinitionCompleter to force completeDefinition we don't seem to have a similar mechanism for ObjCInterfaceDecl.
It is unfortunate that the AST does not have something similar for ObjC interfaces. 
So, this seems to be the right way unless we want to modify DeclObjC.h.
LGTM!
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83972/new/
https://reviews.llvm.org/D83972
    
    
More information about the cfe-commits
mailing list