Changed pack alignment of GPU command buffer commands.

gpu/command_buffer/common/cmd_buffer_common.h

Index: gpu/command_buffer/common/cmd_buffer_common.h
diff --git a/gpu/command_buffer/common/cmd_buffer_common.h b/gpu/command_buffer/common/cmd_buffer_common.h
index c692b7553f071f3bf95d77601bfadd5adc4a54f4..b1690fd94ac531e97ee0131c234a556919f4bcc1 100644
--- a/gpu/command_buffer/common/cmd_buffer_common.h
+++ b/gpu/command_buffer/common/cmd_buffer_common.h
@@ -92,9 +92,8 @@ const size_t kCommandBufferEntrySize = 4;
 COMPILE_ASSERT(sizeof(CommandBufferEntry) == kCommandBufferEntrySize,
                Sizeof_CommandBufferEntry_is_not_4);
 
-// Make sure the compiler does not add extra padding to any of the command
-// structures.
-#pragma pack(push, 1)
+// Command buffer is 4 byte aligned.
+#pragma pack(push, 4)

[piman, 2014/02/03 22:30:49]

Do we even want anything at all, instead of just letting the compiler just use
whatever packing it likes best?

[vmiura, 2014/02/03 22:44:38]

The command buffer is 4 byte aligned, so this pack(4) is mainly here to make
sure any 8 byte aligned types in commands won't fault due to bad alignment on
any architecture.

Currently I think just QuerySync has a "uint64 result".

All other commands use just uint32/int32/float, so in theory pack(4) will be a
noop.

 
 // Gets the address of memory just after a structure in a typesafe way. This is
 // used for IMMEDIATE commands to get the address of the place to put the data.