From 9ca75f15bc838ca74422a37533324d464e1c72f7 Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Mon, 19 Apr 2004 18:02:40 +0000 Subject: [PATCH] stor-layout.c (layout_decl): Check DECL_PACKED before calling ADJUST_FIELD_ALIGN. * stor-layout.c (layout_decl): Check DECL_PACKED before calling ADJUST_FIELD_ALIGN. Check maximum_field_alignment after. From-SVN: r80860 --- gcc/ChangeLog | 5 +++++ gcc/stor-layout.c | 9 +++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 3bd73fee5e5..198314a3f7a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2004-04-19 Daniel Jacobowitz + + * stor-layout.c (layout_decl): Check DECL_PACKED before calling + ADJUST_FIELD_ALIGN. Check maximum_field_alignment after. + 2004-04-19 Andrew PInski * builtins.c (fold_builtin_cabs): Remove fndecl parameter. diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c index d19fc9fdf3f..33d0a86e90d 100644 --- a/gcc/stor-layout.c +++ b/gcc/stor-layout.c @@ -512,10 +512,7 @@ layout_decl (tree decl, unsigned int known_align) || TREE_CODE (DECL_SIZE_UNIT (decl)) == INTEGER_CST)) DECL_ALIGN (decl) = MIN (DECL_ALIGN (decl), BITS_PER_UNIT); - /* Should this be controlled by DECL_USER_ALIGN, too? */ - if (maximum_field_alignment != 0) - DECL_ALIGN (decl) = MIN (DECL_ALIGN (decl), maximum_field_alignment); - if (! DECL_USER_ALIGN (decl)) + if (! DECL_USER_ALIGN (decl) && ! DECL_PACKED (decl)) { /* Some targets (i.e. i386, VMS) limit struct field alignment to a lower boundary than alignment of variables unless @@ -528,6 +525,10 @@ layout_decl (tree decl, unsigned int known_align) DECL_ALIGN (decl) = ADJUST_FIELD_ALIGN (decl, DECL_ALIGN (decl)); #endif } + + /* Should this be controlled by DECL_USER_ALIGN, too? */ + if (maximum_field_alignment != 0) + DECL_ALIGN (decl) = MIN (DECL_ALIGN (decl), maximum_field_alignment); } /* Evaluate nonconstant size only once, either now or as soon as safe. */