martong added a comment.

I like it, though I've found a nit.



================
Comment at: clang/lib/AST/ASTImporter.cpp:2301
+  ToD->setLexicalDeclContext(LexicalDC);
+  DC->addDeclInternal(ToD);
+  ToD->setBinding(ToType, ToBinding);
----------------
Should we use rather `addDeclToContexts` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102492

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D102492: [clang][AST... Gabor Marton via Phabricator via cfe-commits

Reply via email to