shafik added inline comments.
================ Comment at: source/Symbol/ClangASTContext.cpp:9195 + GetAsEnumDecl(type->GetFullCompilerType())) + enum_decl->dump(s.AsRawOstream()); + else { ---------------- teemperor wrote: > shafik wrote: > > aprantl wrote: > > > is an EnumDecl not a TagDecl? > > I don't understand the comment, `TagDecl` is handled above. > I think Adrian's point is that EnumDecl is a subclass of TagDecl. This is a good point, this also goes for `CXXRecordDecl` so removed those cases since they are both covered by `TagDecl`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67994/new/ https://reviews.llvm.org/D67994 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits