zequanwu created this revision.
Herald added a project: All.
zequanwu requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D136006
Files:
lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.cpp
lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.h
Index: lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.h
===================================================================
--- lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.h
+++ lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.h
@@ -11,6 +11,7 @@
#include "llvm/ADT/DenseMap.h"
#include "llvm/ADT/StringRef.h"
+#include "llvm/Support/Threading.h"
#include "Plugins/ExpressionParser/Clang/ClangASTImporter.h"
@@ -122,7 +123,9 @@
TypeIndex func_ti, CompilerType func_ct,
uint32_t param_count, clang::StorageClass func_storage,
bool is_inline, clang::DeclContext *parent);
- void ParseAllNamespacesPlusChildrenOf(llvm::Optional<llvm::StringRef> parent);
+ void ParseAllNamespacesPlusChildrenOf(llvm::StringRef parent);
+ void ParseAllTypes();
+ void ParseAllFunctionsAndNonLocalVars();
void ParseDeclsForSimpleContext(clang::DeclContext &context);
void ParseBlockChildren(PdbCompilandSymId block_id);
@@ -135,7 +138,8 @@
TypeSystemClang &m_clang;
ClangASTImporter m_importer;
-
+ llvm::once_flag m_parse_functions_and_non_local_vars;
+ llvm::once_flag m_parse_all_types;
llvm::DenseMap<clang::Decl *, DeclStatus> m_decl_to_status;
llvm::DenseMap<lldb::user_id_t, clang::Decl *> m_uid_to_decl;
llvm::DenseMap<lldb::user_id_t, clang::QualType> m_uid_to_type;
Index: lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.cpp
===================================================================
--- lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.cpp
+++ lldb/source/Plugins/SymbolFile/NativePDB/PdbAstBuilder.cpp
@@ -14,6 +14,7 @@
#include "llvm/DebugInfo/PDB/Native/TpiStream.h"
#include "llvm/Demangle/MicrosoftDemangle.h"
+
#include "Plugins/ExpressionParser/Clang/ClangASTMetadata.h"
#include "Plugins/ExpressionParser/Clang/ClangUtil.h"
#include "Plugins/Language/CPlusPlus/MSVCUndecoratedNameParser.h"
@@ -21,7 +22,7 @@
#include "lldb/Core/Module.h"
#include "lldb/Symbol/ObjectFile.h"
#include "lldb/Utility/LLDBAssert.h"
-
+#include "lldb/Utility/Timer.h"
#include "PdbUtil.h"
#include "UdtRecordCompleter.h"
#include "SymbolFileNativePDB.h"
@@ -1233,7 +1234,7 @@
}
void PdbAstBuilder::ParseAllNamespacesPlusChildrenOf(
- llvm::Optional<llvm::StringRef> parent) {
+ llvm::StringRef parent) {
SymbolFileNativePDB *pdb = static_cast<SymbolFileNativePDB *>(
m_clang.GetSymbolFile()->GetBackingSymbolFile());
PdbIndex &index = pdb->GetIndex();
@@ -1247,12 +1248,6 @@
CVTagRecord tag = CVTagRecord::create(cvt);
- if (!parent) {
- clang::QualType qt = GetOrCreateType(tid);
- CompleteType(qt);
- continue;
- }
-
// Call CreateDeclInfoForType unconditionally so that the namespace info
// gets created. But only call CreateRecordType if the namespace name
// matches.
@@ -1263,41 +1258,69 @@
continue;
clang::NamespaceDecl *ns = llvm::cast<clang::NamespaceDecl>(context);
- std::string actual_ns = ns->getQualifiedNameAsString();
- if (llvm::StringRef(actual_ns).startswith(*parent)) {
- clang::QualType qt = GetOrCreateType(tid);
- CompleteType(qt);
- continue;
+ llvm::StringRef ns_name = ns->getName();
+ if (ns_name.startswith(parent)) {
+ ns_name = ns_name.drop_front(parent.size());
+ if (ns_name.startswith("::")) {
+ clang::QualType qt = GetOrCreateType(tid);
+ CompleteType(qt);
+ }
}
}
+}
- uint32_t module_count = index.dbi().modules().getModuleCount();
- for (uint16_t modi = 0; modi < module_count; ++modi) {
- CompilandIndexItem &cii = index.compilands().GetOrCreateCompiland(modi);
- const CVSymbolArray &symbols = cii.m_debug_stream.getSymbolArray();
- auto iter = symbols.begin();
- while (iter != symbols.end()) {
- PdbCompilandSymId sym_id{modi, iter.offset()};
-
- switch (iter->kind()) {
- case S_GPROC32:
- case S_LPROC32:
- GetOrCreateFunctionDecl(sym_id);
- iter = symbols.at(getScopeEndOffset(*iter));
- break;
- case S_GDATA32:
- case S_GTHREAD32:
- case S_LDATA32:
- case S_LTHREAD32:
- GetOrCreateVariableDecl(PdbCompilandSymId(modi, 0), sym_id);
- ++iter;
- break;
- default:
- ++iter;
+void PdbAstBuilder::ParseAllTypes() {
+ llvm::call_once(m_parse_all_types, [this]() {
+ SymbolFileNativePDB *pdb = static_cast<SymbolFileNativePDB *>(
+ m_clang.GetSymbolFile()->GetBackingSymbolFile());
+ PdbIndex &index = pdb->GetIndex();
+ TypeIndex ti{index.tpi().TypeIndexBegin()};
+ for (const CVType &cvt : index.tpi().typeArray()) {
+ PdbTypeSymId tid{ti};
+ ++ti;
+
+ if (!IsTagRecord(cvt))
continue;
+
+ clang::QualType qt = GetOrCreateType(tid);
+ CompleteType(qt);
+ }
+ });
+}
+
+void PdbAstBuilder::ParseAllFunctionsAndNonLocalVars() {
+ llvm::call_once(m_parse_functions_and_non_local_vars, [this]() {
+ SymbolFileNativePDB *pdb = static_cast<SymbolFileNativePDB *>(
+ m_clang.GetSymbolFile()->GetBackingSymbolFile());
+ PdbIndex &index = pdb->GetIndex();
+ uint32_t module_count = index.dbi().modules().getModuleCount();
+ for (uint16_t modi = 0; modi < module_count; ++modi) {
+ CompilandIndexItem &cii = index.compilands().GetOrCreateCompiland(modi);
+ const CVSymbolArray &symbols = cii.m_debug_stream.getSymbolArray();
+ auto iter = symbols.begin();
+ while (iter != symbols.end()) {
+ PdbCompilandSymId sym_id{modi, iter.offset()};
+
+ switch (iter->kind()) {
+ case S_GPROC32:
+ case S_LPROC32:
+ GetOrCreateFunctionDecl(sym_id);
+ iter = symbols.at(getScopeEndOffset(*iter));
+ break;
+ case S_GDATA32:
+ case S_GTHREAD32:
+ case S_LDATA32:
+ case S_LTHREAD32:
+ GetOrCreateVariableDecl(PdbCompilandSymId(modi, 0), sym_id);
+ ++iter;
+ break;
+ default:
+ ++iter;
+ continue;
+ }
}
}
- }
+ });
}
static CVSymbolArray skipFunctionParameters(clang::Decl &decl,
@@ -1384,7 +1407,8 @@
// work (such as parsing the items that appear within the namespaces) at the
// same time.
if (context.isTranslationUnit()) {
- ParseAllNamespacesPlusChildrenOf(llvm::None);
+ ParseAllTypes();
+ ParseAllFunctionsAndNonLocalVars();
return;
}
@@ -1392,6 +1416,7 @@
clang::NamespaceDecl &ns = *llvm::dyn_cast<clang::NamespaceDecl>(&context);
std::string qname = ns.getQualifiedNameAsString();
ParseAllNamespacesPlusChildrenOf(llvm::StringRef{qname});
+ ParseAllFunctionsAndNonLocalVars();
return;
}
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
