[PATCH] D51633: [ASTImporter] Added error handling for AST import.

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 18 14:50:18 PDT 2018


a_sidorin accepted this revision.
a_sidorin added a comment.
This revision is now accepted and ready to land.

Hi Balasz,
I cannot find any problems with the latest changes. I suggest you to commit the patch to make further reviews easier.



================
Comment at: lib/AST/ASTImporter.cpp:8197
 
-void ASTImporter::ImportDefinition(Decl *From) {
+Error ASTImporter::ImportDefinition_New(Decl *From) {
   Decl *To = Import(From);
----------------
ImportDefinitionOrError?


Repository:
  rC Clang

https://reviews.llvm.org/D51633





More information about the cfe-commits mailing list