[Lldb-commits] [PATCH] D75488: Preserve the owning module information from DWARF in the synthesized AST

Adrian Prantl via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 31 16:00:04 PDT 2020

aprantl marked an inline comment as done.
aprantl added inline comments.

Comment at: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:1260
+  module->Name = name.str();
+  return ast_source->RegisterModule(module);
teemperor wrote:
> Why is that done? The module should already have the correct name when it is returned from findOrCreateModule (where we pass the same name and then create the new module with that name).
> If this line is removed, do we still need D75561?
Good catch! This is a leftover from my first version of the patch where I created the Module object manually without instantiating a clang::ModuleMap.

Comment at: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:1256
+      name,
+      parent_desc ? const_cast<clang::Module *>(parent_desc->getModuleOrNull())
+                  : nullptr,
here ^^



More information about the lldb-commits mailing list