[Lldb-commits] [PATCH] D42955: Make Module::GetSectionList output consistent

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Feb 22 10:06:16 PST 2018

labath added inline comments.

Comment at: source/Core/Module.cpp:1283-1284
   if (!m_sections_ap) {
-    ObjectFile *obj_file = GetObjectFile();
-    if (obj_file != nullptr)
+    if (ObjectFile *obj_file = GetObjectFile())
+    if (SymbolVendor *vendor = GetSymbolVendor())
clayborg wrote:
> If we are going to let the symbol vendor do it's thing, then we should probably not manually call ObjectFile::CreateSections. The idea behind SymbolVendor is that _it_ knows how best to do things for one or more object files. Can we move all logic for parsing sections into the SymbolVendor::CreateSections()?
I like that idea. I'm going to take a look to see how that could be done.


More information about the lldb-commits mailing list