https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53037
Rainer Orth <ro at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ebotcazou at gcc dot gnu.org
--- Comment #31 from Rainer Orth <ro at gcc dot gnu.org> ---
(In reply to H.J. Lu from comment #29)
> This patch
>
> diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c
> index 3028d55773a..6dd605810ac 100644
> --- a/gcc/stor-layout.c
> +++ b/gcc/stor-layout.c
> @@ -1784,7 +1784,7 @@ finalize_type_size (tree type)
>
> /* Don't override a larger alignment requirement coming from a user
> alignment of one of the fields. */
> - if (mode_align >= TYPE_ALIGN (type))
> + if (mode_align > TYPE_ALIGN (type))
> {
> SET_TYPE_ALIGN (type, mode_align);
> TYPE_USER_ALIGN (type) = 0;
>
> works with cross compiler. But I have no idea if it is correct.
I've included it in a sparc bootstrap and the only effect on testresults was
to fix the set of failures I've reported.
Let's ask Eric for his opinion.
Rainer
