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

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 4 04:13:38 PDT 2019


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

👍



================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp:277
   merger.RemoveSources(importer_source);
-  return ret;
+  if (ret_or_error)
+    return *ret_or_error;
----------------
'true' body needs to be surrounded by braces as well. Same below.


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