[Lldb-commits] [PATCH] D53590: Refactor SetBaseClassesForType and DeleteBaseClasses to be more C++'y

Shafik Yaghmour via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 23 12:39:04 PDT 2018

shafik added a comment.

+1 for modernizing code!

Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:2262
+          // assertion in the call to clang_type.TransferBaseClasses()
+          for (auto &base_class : bases) {
             clang::TypeSourceInfo *type_source_info =
`const auto &`

Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:3274
+          if (!result)
+            break;
Is this correct? the if/else below does not seem dependent on this result?


More information about the lldb-commits mailing list