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

Reply via email to