[Lldb-commits] [PATCH] D49410: [PDB] Parse UDT symbols and pointers to members

Aleksandr Urakov via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 19 02:50:59 PDT 2018

aleksandr.urakov added inline comments.

Comment at: source/Plugins/SymbolFile/PDB/PDBASTParser.cpp:696
+          base_ast_type.GetOpaqueQualType(), access,
+          pdb_base_class->isVirtualBaseClass(), /*base_of_class*/ true);
+      base_classes.push_back(base_spec);
Hui wrote:
> aleksandr.urakov wrote:
> > If I understand correctly, `base_of_class` must be `false` for structs. May be check the kind of `pdb_udt` here?
> Struct/Union can't have base class. Only class udt is handled. I think it is safe to set it ture. 
I think that it must be `false` for following code:
class B {
  char _b;

struct S : B {
  int _s;


More information about the lldb-commits mailing list