aprantl added inline comments.
================ Comment at: lldb/include/lldb/Symbol/ClangASTContext.h:1005 + // May be null if we are already done parsing this ASTContext or the + // ASTContext wasn't created by parsing source code. + clang::Sema * m_sema = nullptr; ---------------- `///` ================ Comment at: lldb/include/lldb/Symbol/ClangASTImporter.h:249 + /// it at the end of the scope. Supports being used multiple times on the + /// same Minion instance in nested scopes. + class CxxModuleScope { ---------------- I really wish we could rename Minion with something actually descriptive... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59537/new/ https://reviews.llvm.org/D59537 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits