[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