[PATCH] D61438: [ASTImporter] Use llvm::Expected and Error in the importer API

Adrian Prantl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 9 09:29:58 PDT 2019


aprantl added inline comments.


================
Comment at: clang/include/clang/AST/ASTImporter.h:203
     /// context, or the import error.
-    llvm::Expected<TypeSourceInfo *> Import_New(TypeSourceInfo *FromTSI);
-    // FIXME: Remove this version.
-    TypeSourceInfo *Import(TypeSourceInfo *FromTSI);
+    llvm::Expected<TypeSourceInfo *> Import(TypeSourceInfo *FromTSI);
 
----------------
Wouldn't it make more sense to return `Expected<TypeSourceInfo &>`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61438





More information about the cfe-commits mailing list