aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Stump <mikestump@comcast.net>2010-03-22 23:13:10 +0000
committerMike Stump <mrs@gcc.gnu.org>2010-03-22 23:13:10 +0000
commit212066e660034c73cdeffc6120b03a0938afcb53 (patch)
treeb7179fef7508abaffda8ff48ecb0c485d36365a7
parentcc72bbaac742dd66ce5aaa9791a20903cdbac3ec (diff)
downloadgcc-212066e660034c73cdeffc6120b03a0938afcb53.zip
gcc-212066e660034c73cdeffc6120b03a0938afcb53.tar.gz
gcc-212066e660034c73cdeffc6120b03a0938afcb53.tar.bz2
re PR target/23071 (Darwin alignment ignores "attribute packed" for first 'double' element of a struct)
PR target/23071 * config/rs6000/rs6000.c (darwin_rs6000_special_round_type_align): Don't overly align based upon packed packed fields. From-SVN: r157654
-rw-r--r--gcc/ChangeLog6
-rw-r--r--gcc/config/rs6000/rs6000.c3
-rw-r--r--gcc/testsuite/gcc.target/powerpc/darwin-abi-12.c14
3 files changed, 23 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index f8515cd..e1bd54a 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2010-03-22 Mike Stump <mikestump@comcast.net>
+
+ PR target/23071
+ * config/rs6000/rs6000.c (darwin_rs6000_special_round_type_align):
+ Don't overly align based upon packed packed fields.
+
2010-03-22 Jason Merrill <jason@redhat.com>
* c-pretty-print.c (pp_c_specifier_qualifier_list) [VECTOR_TYPE]:
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index dea0271..ab48e82 100644
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -4653,6 +4653,9 @@ darwin_rs6000_special_round_type_align (tree type, unsigned int computed,
field = TREE_CHAIN (field);
if (! field)
break;
+ /* A packed field does not contribute any extra alignment. */
+ if (DECL_PACKED (field))
+ return align;
type = TREE_TYPE (field);
while (TREE_CODE (type) == ARRAY_TYPE)
type = TREE_TYPE (type);
diff --git a/gcc/testsuite/gcc.target/powerpc/darwin-abi-12.c b/gcc/testsuite/gcc.target/powerpc/darwin-abi-12.c
new file mode 100644
index 0000000..d02c486
--- /dev/null
+++ b/gcc/testsuite/gcc.target/powerpc/darwin-abi-12.c
@@ -0,0 +1,14 @@
+/* { dg-do compile { target powerpc*-*-darwin* } } */
+/* { dg-final { scan-assembler ".comm _x,12,2" } } */
+/* { dg-final { scan-assembler-not ".space 7" } } */
+/* PR 23071 */
+
+struct Test {
+ double D __attribute__((packed,aligned(4)));
+ short X;
+} x;
+
+struct {
+ char x;
+ struct Test t;
+} b = { 1, { 2, 3 } };