[PATCH] D18918: [DebugInfo] Try to make class memory layout more efficient

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: include/llvm/IR/DebugInfoMetadata.h:1237
@@ -1238,1 +1236,3 @@
+  bool IsDefinition:1;
+  bool IsOptimized:1;
 
----------------
probinson wrote:
> MSVC starts a new container field every time you change the base type, so this won't pack the way you want on Windows.  Use unsigned as the base type in all of these.
Oh, sigh. I'll make sure to test with MSVC.


http://reviews.llvm.org/D18918