This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch i386]: Fix PR 56807
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Jan Hubicka <hubicka at ucw dot cz>, Kai Tietz <ktietz70 at googlemail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 10 Dec 2013 13:01:34 +0100
- Subject: Re: [patch i386]: Fix PR 56807
- Authentication-results: sourceware.org; auth=none
- References: <CAEwic4YRnyDhML9GYvM1+OyrTdX+-F4HagKJWyfjDnNhLkM4wQ at mail dot gmail dot com> <20131210114820 dot GF8808 at atrey dot karlin dot mff dot cuni dot cz> <20131210115601 dot GR892 at tucnak dot redhat dot com>
> On Tue, Dec 10, 2013 at 12:48:20PM +0100, Jan Hubicka wrote:
> > > Hi,
> > >
> > >
> > > ChangeLog
> > >
> > > 2013-12-06 Kai Tietz <ktietz@redhat.com>
> > >
> > > PR target/56807
> > > * config/i386/i386.c (ix86_expand_prologue):
> > >
> > > Tested for i686-w64-mingw32, x86_64-unknown-linux-gnu. Ok for apply?
> >
> > So the code in question does spilling relative to stack pointer before function
> > call and relative to base pointer after the function call and this leads to mismatch
> > when stack alignment is in the way?
> >
> > The chagne seems OK if that is the case.
>
> The ChangeLog entry is not ok though.
Kai sent updated one in reply to Jeff's email I beleive.
honza
>
> Jakub