[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;
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).


More information about the lldb-commits mailing list