[PATCH] D21526: [codeview] Improved array type support (multi dimension array)

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 12:29:32 PDT 2016


majnemer added inline comments.

================
Comment at: lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:859-862
@@ +858,6 @@
+  // FIXME:
+  // There is a bug in the front-end where an array of an incomplete structure
+  // declaration ends up not getting a size assigned to it.  This needs to
+  // be fixed in the front-end, but in the meantime we don't want to trigger an
+  // assertion because of this.
+  if (Ty->getSizeInBits() == 0) {
----------------
If a type is incomplete, how can we emit a size for it?


http://reviews.llvm.org/D21526





More information about the llvm-commits mailing list