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

Balázs Kéri via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 17 03:21:34 PDT 2018


balazske added a comment.

In the current version all clang tests do pass and most of the LLDB tests. Some LLDB tests do not work on my machine, the problems exist on master too.
This code is still a temporary version (possible to commit now or wait until every change is made?). Later the `Import` functions should return `Expected` and the remaining functions in ASTImporter.cpp should be updated. For compatibility with old code in LLDB the `Import` functions can not be changed without LLDB code changes so a `Import_New` will be created instead. Code in clang will only use the `Import_New`. When LLDB was updated, `Import_New` should be renamed to `Import`.


Repository:
  rC Clang

https://reviews.llvm.org/D51633





More information about the cfe-commits mailing list