[Lldb-commits] [PATCH] D70663: [lldb] Remove lldb's own ASTDumper

[Pavel Labath via Phabricator via lldb-commits]

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

Yay



================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp:1675
+           "  CEDM::FEVD[{0}] Found variable {1}, returned\n{2} (original {3})",
+           current_id, decl_name.c_str(), ClangUtil::DumpDecl(var_decl),
+           ClangUtil::ToString(ut));
----------------
drop .c_str()


================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp:1709
+  LLDB_LOG(log, "  CEDM::FEVD[{0}] Added pvar {1}, returned\n{2}", current_id,
+           pvar_sp->GetName().GetCString(), ClangUtil::DumpDecl(var_decl));
 }
----------------
drop .GetCString()


================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp:1762
+  LLDB_LOG(log, "  CEDM::FEVD[{0}] Found variable {1}, returned\n{2}",
+           current_id, decl_name.c_str(), ClangUtil::DumpDecl(var_decl));
 }
----------------
drop .c_str()


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D70663