================
Comment at: lib/CodeGen/MicrosoftCXXABI.cpp:1014
@@ +1013,3 @@
+    int32_t IntSize =
+        getContext().getTypeSizeInChars(getContext().IntTy).getQuantity();
+    llvm::Value *VBPtr;
----------------
Reid Kleckner wrote:
> I'm frustrated that I can't think of a shorter way to write this.
Could as well be just "4" :)

================
Comment at: include/clang/Basic/ABI.h:51
@@ +50,3 @@
+      int64_t VBaseOffsetOffset;
+    };
+
----------------
unnamed structs in a union turned out to be MSVC+gcc extensions, so I've named 
them


http://llvm-reviews.chandlerc.com/D2026
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to