diff options
author | Richard Kenner <kenner@vlsi1.ultra.nyu.edu> | 2001-12-29 21:00:51 +0000 |
---|---|---|
committer | Richard Kenner <kenner@gcc.gnu.org> | 2001-12-29 16:00:51 -0500 |
commit | 6790d1bd90d6d41631034507a42a42d04f1d4003 (patch) | |
tree | 9eff696e051cd33a2392999d77a89728d2cec952 | |
parent | f11c3043051e421744ea3d5d3b32c9149fbacf4e (diff) | |
download | gcc-6790d1bd90d6d41631034507a42a42d04f1d4003.zip gcc-6790d1bd90d6d41631034507a42a42d04f1d4003.tar.gz gcc-6790d1bd90d6d41631034507a42a42d04f1d4003.tar.bz2 |
stor-layout.c (layout_decl): Don't misalign field of variable size for packed record.
* stor-layout.c (layout_decl): Don't misalign field of variable size
for packed record.
From-SVN: r48384
-rw-r--r-- | gcc/ChangeLog | 3 | ||||
-rw-r--r-- | gcc/stor-layout.c | 10 |
2 files changed, 12 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e4f8b81..5ba989d 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,8 @@ Sat Dec 29 15:48:54 2001 Richard Kenner <kenner@vlsi1.ultra.nyu.edu> + * stor-layout.c (layout_decl): Don't misalign field of variable size + for packed record. + * dwarf2out.c (compute_section_prefix): Avoid cast warning. (gen_decl_die): Only check DECL_IGNORED_P on decls. (dwarf2out_decl): Check for DECL_IGNORED_P only when needed. diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c index f657463..fe9ed2c 100644 --- a/gcc/stor-layout.c +++ b/gcc/stor-layout.c @@ -389,7 +389,15 @@ layout_decl (decl, known_align) DECL_BIT_FIELD_TYPE (decl) = DECL_BIT_FIELD (decl) ? type : 0; if (maximum_field_alignment != 0) DECL_ALIGN (decl) = MIN (DECL_ALIGN (decl), maximum_field_alignment); - else if (DECL_PACKED (decl)) + + /* If the field is of variable size, we can't misalign it since we + have no way to make a temporary to align the result. But this + isn't an issue if the decl is not addressable. Likewise if it + is of unknown size. */ + else if (DECL_PACKED (decl) + && (DECL_NONADDRESSABLE_P (decl) + || DECL_SIZE_UNIT (decl) == 0 + || TREE_CODE (DECL_SIZE_UNIT (decl)) == INTEGER_CST)) { DECL_ALIGN (decl) = MIN (DECL_ALIGN (decl), BITS_PER_UNIT); DECL_USER_ALIGN (decl) = 0; |