diff options
| author | Nathan Sidwell <nathan@codesourcery.com> | 2005-11-28 10:34:30 +0000 |
|---|---|---|
| committer | Nathan Sidwell <nathan@gcc.gnu.org> | 2005-11-28 10:34:30 +0000 |
| commit | 646c0835465a8438ce3543bfcae30becf641a63e (patch) | |
| tree | 7e925b64a93bd395a2f07912807080dfbe0ecd38 /gcc/c-common.c | |
| parent | bbee4b64682d27f5ea99df3654e3e1b139443e0e (diff) | |
| download | gcc-646c0835465a8438ce3543bfcae30becf641a63e.zip gcc-646c0835465a8438ce3543bfcae30becf641a63e.tar.gz gcc-646c0835465a8438ce3543bfcae30becf641a63e.tar.bz2 | |
re PR c++/21166 (g++ gives error on reference to packed structure elements)
.: PR c++/21166
* c-decl.c (finish_struct): Only set DECL_PACKED on a field
when its natural alignment is > BITS_PER_UNIT.
* stor-layout.c (finalize_type_size): Revert my patch of 2005-08-08.
* c-common.c (handle_packed_attribute): Ignore packing on a field
whose type is naturally char aligned.
cp:
PR c++/21166
* class.c (check_field_decls): Only set DECL_PACKED on a field
when its natural alignment is > BITS_PER_UNIT.
From-SVN: r107599
Diffstat (limited to 'gcc/c-common.c')
| -rw-r--r-- | gcc/c-common.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/gcc/c-common.c b/gcc/c-common.c index b416a5d..43c147b 100644 --- a/gcc/c-common.c +++ b/gcc/c-common.c @@ -4065,17 +4065,23 @@ handle_packed_attribute (tree *node, tree name, tree ARG_UNUSED (args), struct Foo { struct Foo const *ptr; // creates a variant w/o packed flag - } __ attribute__((packed)); // packs it now. - */ + } __ attribute__((packed)); // packs it now. + */ tree probe; for (probe = *node; probe; probe = TYPE_NEXT_VARIANT (probe)) TYPE_PACKED (probe) = 1; } - } else if (TREE_CODE (*node) == FIELD_DECL) - DECL_PACKED (*node) = 1; + { + if (TYPE_ALIGN (TREE_TYPE (*node)) <= BITS_PER_UNIT) + warning (OPT_Wattributes, + "%qE attribute ignored for field of type %qT", + name, TREE_TYPE (*node)); + else + DECL_PACKED (*node) = 1; + } /* We can't set DECL_PACKED for a VAR_DECL, because the bit is used for DECL_REGISTER. It wouldn't mean anything anyway. We can't set DECL_PACKED on the type of a TYPE_DECL, because |