sgraenitz added inline comments.
================ Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp:1980 + } + return *type_or_error; } ---------------- >>! In D61438#1490102, @jingham wrote: > [...] include the contents of that error n the log message? e.g: ``` if (auto type_owner = merger.ImporterForOrigin(from_context).Import(type)) { return *type_owner; } else { Log *log = lldb_private::GetLogIfAllCategoriesSet(LIBLLDB_LOG_EXPRESSIONS); LLDB_LOG_ERROR(log, type_owner.takeError(), "Couldn't import type: {0}") return QualType(); } ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61438/new/ https://reviews.llvm.org/D61438 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits