[Lldb-commits] [PATCH] D59198: Correctly look up declarations in inline namespaces

2019-03-12 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rLLDB355897: Correctly look up declarations in inline namespaces (authored by teemperor, committed by ). Changed prior to commit: https://reviews.llvm.org/D59198?vs=190111&id=190215#toc Repository: rLL

[Lldb-commits] [PATCH] D59198: Correctly look up declarations in inline namespaces

2019-03-11 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor updated this revision to Diff 190111. teemperor added a comment. - Addressed Adrian's comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59198/new/ https://reviews.llvm.org/D59198 Files: lldb/include/lldb/Symbol/ClangASTContext.h lldb/include/lldb/Symbol/CompilerDecl

[Lldb-commits] [PATCH] D59198: Correctly look up declarations in inline namespaces

2019-03-11 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl accepted this revision. aprantl added inline comments. This revision is now accepted and ready to land. Comment at: lldb/source/Symbol/ClangASTContext.cpp:10269 +void *opaque_decl_ctx, void *other_opaque_decl_ctx) { + auto decl_ctx = (clang::DeclContext *)opaque_decl

[Lldb-commits] [PATCH] D59198: Correctly look up declarations in inline namespaces

2019-03-10 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. teemperor added reviewers: aprantl, shafik. teemperor added a project: C++ modules in LLDB. Herald added subscribers: lldb-commits, jdoerfert, eraman. Herald added a reviewer: serge-sans-paille. Herald added a project: LLDB. This patch marks the inline namespaces f