This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
build_duplicate_type and transparent_union
- From: Richard Guenther <rguenther at suse dot de>
- To: gcc-patches at gcc dot gnu dot org
- Cc: rth at redhat dot com
- Date: Wed, 28 Mar 2007 11:39:38 +0200 (CEST)
- Subject: build_duplicate_type and transparent_union
Whilst looking over tree-inline.c I noticed that the only user of
build_duplicate_type (transparent_union handling) does now a no-op
transform of its type. With r105338 remap_type got splitted up into
remap_type_1 and remap_type so the new build_duplicate_type would always
build a new type, but that change was made a no-op by putting back
the body if remap_type into remap_type_1 (making the former useless now)
by r109797 (both from rth).
It also looks like we don't have a testcase for
/* A type variant isn't good enough, since we don't want a cast
to such a type removed as a no-op. */
as stated in handle_transparent_union_attribute, but I believe that
it would be broken now (as clearly unions cannot be of variable modified
type).
Now, can you please clarify and/or fix it up somehow? ;)
Thanks,
Richard.