[Lldb-commits] [PATCH] D66175: Improve anonymous class heuristic in ClangASTContext::CreateRecordType
Adrian Prantl via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Aug 13 16:40:30 PDT 2019
aprantl added inline comments.
Comment at: source/Symbol/ClangASTContext.cpp:1521
+ // This fix will unfortunately still mislabel unnamed classes within a class
+ // but this improves the situation greatly since getting this wrong in the
// FIXME: An unnamed class within a class is also wrongly recognized as an anonymous struct.
Comment at: source/Symbol/ClangASTContext.cpp:1524
+ // other cases can lead to an assert in clang CodeCompletion since
+ // SemaAccess assumes the DeclContext of an anonymous class is a
+ // CXXRecordDecl.
and move the excuse in the commit message.
It would be really helpful though to include tiny source code examples here to iiustrate the difference between unnames struct and anonymous structs vs. unnamed structs in structs.
CHANGES SINCE LAST ACTION
More information about the lldb-commits