[PATCH] D53755: [ASTImporter] Remove import of definition from GetAlreadyImportedOrNull

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 25 11:10:34 PST 2018


a_sidorin added a comment.

More constants are always welcome.



================
Comment at: lib/AST/ExternalASTMerger.cpp:154
       ToContainer->setHasExternalLexicalStorage();
-      ToContainer->setMustBuildLookupTable();
+      ToContainer->getPrimaryContext()->setMustBuildLookupTable();
       assert(Parent.CanComplete(ToContainer));
----------------
Do we have to do the same for NamespaceDecls?


Repository:
  rC Clang

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

https://reviews.llvm.org/D53755





More information about the cfe-commits mailing list