This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug tree-optimization/43528] ICE: in tree_low_cst, at tree.c:6198 with -mms-bitfields at x86_64-linux
- From: "ebotcazou at gcc dot gnu dot org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 26 Mar 2010 11:50:02 -0000
- Subject: [Bug tree-optimization/43528] ICE: in tree_low_cst, at tree.c:6198 with -mms-bitfields at x86_64-linux
- References: <bug-43528-14164@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Comment #6 from ebotcazou at gcc dot gnu dot org 2010-03-26 11:50 -------
> If this is indeed required for tree_low_cst (which is defined as signed HWI!),
> then the patch that reverses the semantics of my previous patch works as well.
>
> Index: stor-layout.c
> ===================================================================
> --- stor-layout.c (revision 157742)
> +++ stor-layout.c (working copy)
> @@ -1349,9 +1349,9 @@ place_field (record_layout_info rli, tre
> && host_integerp (TYPE_SIZE (TREE_TYPE (field)), 0)
> && host_integerp (DECL_SIZE (field), 0))
> {
> - HOST_WIDE_INT bitsize = tree_low_cst (DECL_SIZE (field), 1);
> + HOST_WIDE_INT bitsize = tree_low_cst (DECL_SIZE (field), 0);
> HOST_WIDE_INT typesize
> - = tree_low_cst (TYPE_SIZE (TREE_TYPE (field)), 1);
> + = tree_low_cst (TYPE_SIZE (TREE_TYPE (field)), 0);
>
> if (typesize < bitsize)
> rli->remaining_in_alignment = 0;
Yep, that's the right version.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43528