[Lldb-commits] [PATCH] D40615: Fix assertion in ClangASTContext

[Pavel Labath via Phabricator via lldb-commits]

labath marked an inline comment as done.
labath added inline comments.


================
Comment at: include/lldb/Symbol/CompilerType.h:294
 
+  struct IntegralTemplateArgument;
+
----------------
clayborg wrote:
> 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).


https://reviews.llvm.org/D40615