vsapsai added a comment.

This is a clean-up before https://reviews.llvm.org/D110453



================
Comment at: clang/lib/AST/DeclObjC.cpp:608
-  // Make the type point at the definition, now that we have one.
-  if (TypeForDecl)
-    cast<ObjCInterfaceType>(TypeForDecl)->Decl = this;
----------------
Tracking a type per decl is complementary to tracking decl per type and it 
looks potentially bug-prone. But I haven't noticed any problems with it and 
haven't tried to remove `TypeForDecl`. If anybody has any extra information on 
it, we can schedule subsequent fixes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110452/new/

https://reviews.llvm.org/D110452

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to