[Lldb-commits] [PATCH] D146286: [lldb][PDB] Rename GetDeclarationForSymbol() -> AddSourceInfoToDecl()
Stefan Gränitz via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Mar 17 04:35:05 PDT 2023
sgraenitz created this revision.
sgraenitz added reviewers: aleksandr.urakov, labath, rnk, asmith, zturner.
Herald added a project: All.
sgraenitz requested review of this revision.
Herald added a project: LLDB.
The old name of this function was confusing for me, when I started working on the PDB parser. The new name clearifies that the function adds file, line and column (typically referred as source info) and indicates that the information is stored in the provided decl parameter.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D146286
Files:
lldb/source/Plugins/SymbolFile/PDB/PDBASTParser.cpp
Index: lldb/source/Plugins/SymbolFile/PDB/PDBASTParser.cpp
===================================================================
--- lldb/source/Plugins/SymbolFile/PDB/PDBASTParser.cpp
+++ lldb/source/Plugins/SymbolFile/PDB/PDBASTParser.cpp
@@ -190,8 +190,7 @@
return compiler_type.GetTypeName();
}
-static bool GetDeclarationForSymbol(const PDBSymbol &symbol,
- Declaration &decl) {
+static bool AddSourceInfoToDecl(const PDBSymbol &symbol, Declaration &decl) {
auto &raw_sym = symbol.getRawSymbol();
auto first_line_up = raw_sym.getSrcLineOnTypeDefn();
@@ -464,7 +463,7 @@
if (udt->isVolatileType())
clang_type = clang_type.AddVolatileModifier();
- GetDeclarationForSymbol(type, decl);
+ AddSourceInfoToDecl(type, decl);
return m_ast.GetSymbolFile()->MakeType(
type.getSymIndexId(), ConstString(name), udt->getLength(), nullptr,
LLDB_INVALID_UID, lldb_private::Type::eEncodingIsUID, decl, clang_type,
@@ -533,7 +532,7 @@
if (enum_type->isVolatileType())
ast_enum = ast_enum.AddVolatileModifier();
- GetDeclarationForSymbol(type, decl);
+ AddSourceInfoToDecl(type, decl);
return m_ast.GetSymbolFile()->MakeType(
type.getSymIndexId(), ConstString(name), bytes, nullptr,
LLDB_INVALID_UID, lldb_private::Type::eEncodingIsUID, decl, ast_enum,
@@ -579,7 +578,7 @@
if (type_def->isVolatileType())
ast_typedef = ast_typedef.AddVolatileModifier();
- GetDeclarationForSymbol(type, decl);
+ AddSourceInfoToDecl(type, decl);
std::optional<uint64_t> size;
if (type_def->getLength())
size = type_def->getLength();
@@ -659,7 +658,7 @@
m_ast.CreateFunctionType(return_ast_type, arg_list.data(),
arg_list.size(), is_variadic, type_quals, cc);
- GetDeclarationForSymbol(type, decl);
+ AddSourceInfoToDecl(type, decl);
return m_ast.GetSymbolFile()->MakeType(
type.getSymIndexId(), ConstString(name), std::nullopt, nullptr,
LLDB_INVALID_UID, lldb_private::Type::eEncodingIsUID, decl,
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D146286.506038.patch
Type: text/x-patch
Size: 2110 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230317/9847cf7f/attachment.bin>
More information about the lldb-commits
mailing list