[Lldb-commits] [PATCH] D40615: Fix assertion in ClangASTContext
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Nov 30 01:59:47 PST 2017
labath marked an inline comment as done.
labath added inline comments.
Comment at: include/lldb/Symbol/CompilerType.h:294
+ struct IntegralTemplateArgument;
> Can't you just define the type right here?
Unfortunately, I can't because it contains a CompilerType member, which means CompilerType has to be a complete type, and this only happens after we reach the end of the class definition.
However, I may have gone a bit overboard with the nesting of this type inside CompilerType. I could make this just a standalone class if you think that will make things more understandable (a forward declaration will still be needed though).
More information about the lldb-commits