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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to