[cfe-commits] r71723 - /cfe/trunk/test/Sema/pragma-pack-3.c

Daniel Dunbar daniel at zuster.org
Wed May 13 14:34:11 PDT 2009


Author: ddunbar
Date: Wed May 13 16:34:08 2009
New Revision: 71723

URL: http://llvm.org/viewvc/llvm-project?rev=71723&view=rev
Log:
Add a test case to showcase a difference in #pragma pack handling
compared to gcc. This is worrisome, but I believe we are doing the
"correct" thing, and if I recall correctly I previously verified this
versus MSVC.

Added:
    cfe/trunk/test/Sema/pragma-pack-3.c

Added: cfe/trunk/test/Sema/pragma-pack-3.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/pragma-pack-3.c?rev=71723&view=auto

==============================================================================
--- cfe/trunk/test/Sema/pragma-pack-3.c (added)
+++ cfe/trunk/test/Sema/pragma-pack-3.c Wed May 13 16:34:08 2009
@@ -0,0 +1,34 @@
+// RUN: clang-cc -triple i686-apple-darwin9 %s -fsyntax-only -verify
+
+// Stack: [], Alignment: 8
+
+#pragma pack(push, 1)
+// Stack: [8], Alignment: 1
+
+#pragma pack(push, 4)
+// Stack: [8, 1], Alignment: 4
+
+// Note that this differs from gcc; pack() in gcc appears to pop the
+// top stack entry and resets the current alignment. This is both
+// inconsistent with MSVC, and the gcc documentation. In other cases,
+// for example changing this to pack(8), I don't even understand what gcc
+// is doing.
+
+#pragma pack()
+// Stack: [8, 1], Alignment: 8
+
+#pragma pack(pop)
+// Stack: [8], Alignment: 1
+struct s0 {
+  char f0;
+  short f1;
+};
+int a[sizeof(struct s0) == 3 ? 1 : -1];
+
+#pragma pack(pop)
+// Stack: [], Alignment: 8
+struct s1 {
+  char f0;
+  short f1;
+};
+int b[sizeof(struct s1) == 4 ? 1 : -1];





More information about the cfe-commits mailing list