[PATCH] D105564: Fix for DWARF parsing to better handle auto return type for member functions

Greg Clayton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 12 13:33:26 PDT 2021


clayborg requested changes to this revision.
clayborg added inline comments.


================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:1305
+
+  // If a function has an auto return type we need to find the defintion since
+  // that will have the deduced return type and adjust the FunctionDecl to
----------------
s/defintion/definition/


================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:2677
+
+  Symbol *defintion_class_symbol = nullptr;
+  if (m_objfile_sp) {
----------------
type


================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:2689
+  DWARFUnit *unit = die.GetCU();
+  DWARFCompileUnit *dcu = llvm::cast_or_null<DWARFCompileUnit>(unit);
+  DWARFDIE other_die =
----------------
check dcu to ensure it isn't NULL


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105564/new/

https://reviews.llvm.org/D105564



More information about the cfe-commits mailing list