This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] i386: Do not align small stack slots to 16 bytes
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: Segher Boessenkool <segher at kernel dot crashing dot org>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Shmuel Hanoch <shmuel at infinitylabs dot co dot il>
- Date: Tue, 7 Mar 2017 08:28:43 +0100
- Subject: Re: [PATCH] i386: Do not align small stack slots to 16 bytes
- Authentication-results: sourceware.org; auth=none
- References: <7d76131e803aad99c6252fe73eaed43e80ffed40.1488759797.git.segher@kernel.crashing.org> <CAFULd4YzJwZMuX47Du0DbiKW7=4WhcU-dd8Hdsubzb61M8V+RA@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Mar 07, 2017 at 08:23:17AM +0100, Uros Bizjak wrote:
> On Mon, Mar 6, 2017 at 6:51 PM, Segher Boessenkool
> <segher@kernel.crashing.org> wrote:
> > As Shmuel reported in <https://gcc.gnu.org/ml/gcc-help/2017-03/msg00009.html>,
> > on x86-64 small structures in automatic storage are aligned to 16 bytes.
> > This seems to be because of a mix-up between bits and bytes in the i386
> > target code.
> >
> > Is this okay for trunk?
> >
> >
> > Segher
> >
> >
> > 2017-03-06 Segher Boessenkool <segher@kernel.crashing.org>
> >
> > * gcc/config/i386/i386.c (ix86_local_alignment): Align most aggregates
> > of 16 bytes and more to 16 bytes, not those of 16 bits and more.
>
> OK for trunk and backports.
Just without gcc/ prefix in ChangeLog ;)
Jakub