Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Andreas Krebbel
On 02/09/2015 01:17 PM, Jakub Jelinek wrote: > On Mon, Feb 09, 2015 at 01:05:23PM +0100, Jakub Jelinek wrote: >> As you can see, the updated testcase fails even on x86_64-linux. > > Here is an updated patch that succeeds even on i686-linux. Your patch fixes my testcase on s390x. Thanks! -Andreas

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Andreas Krebbel
On 02/09/2015 01:05 PM, Jakub Jelinek wrote: > On Mon, Feb 09, 2015 at 12:40:05PM +0100, Andreas Krebbel wrote: >> On 02/09/2015 12:29 PM, Jakub Jelinek wrote: >>> On Mon, Feb 09, 2015 at 10:50:34AM +0100, Andreas Krebbel wrote: Hi, the attached patch fixes a critical problem in the

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Richard Biener
On Mon, Feb 9, 2015 at 1:17 PM, Jakub Jelinek wrote: > On Mon, Feb 09, 2015 at 01:05:23PM +0100, Jakub Jelinek wrote: >> As you can see, the updated testcase fails even on x86_64-linux. > > Here is an updated patch that succeeds even on i686-linux. Ok if it fixes s390. Thanks, Richard. > 2015-0

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Jakub Jelinek
On Mon, Feb 09, 2015 at 01:05:23PM +0100, Jakub Jelinek wrote: > As you can see, the updated testcase fails even on x86_64-linux. Here is an updated patch that succeeds even on i686-linux. 2015-02-09 Jakub Jelinek PR target/64979 * tree-stdarg.c (pass_stdarg::execute): Scan ph

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Jakub Jelinek
On Mon, Feb 09, 2015 at 12:40:05PM +0100, Andreas Krebbel wrote: > On 02/09/2015 12:29 PM, Jakub Jelinek wrote: > > On Mon, Feb 09, 2015 at 10:50:34AM +0100, Andreas Krebbel wrote: > >> Hi, > >> > >> the attached patch fixes a critical problem in the va_start expansion > >> code in the S/390 backen

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Andreas Krebbel
On 02/09/2015 12:29 PM, Jakub Jelinek wrote: > On Mon, Feb 09, 2015 at 10:50:34AM +0100, Andreas Krebbel wrote: >> Hi, >> >> the attached patch fixes a critical problem in the va_start expansion >> code in the S/390 backend. The problem exists since GCC 4.0. >> >> Ok to commit to 4.9 branch and mai

Re: [PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Jakub Jelinek
On Mon, Feb 09, 2015 at 10:50:34AM +0100, Andreas Krebbel wrote: > Hi, > > the attached patch fixes a critical problem in the va_start expansion > code in the S/390 backend. The problem exists since GCC 4.0. > > Ok to commit to 4.9 branch and mainline? No. This isn't a backend problem, but a bu

[PATCH] PR64979: S/390: Fix setup of overflow area pointer in va_start

2015-02-09 Thread Andreas Krebbel
Hi, the attached patch fixes a critical problem in the va_start expansion code in the S/390 backend. The problem exists since GCC 4.0. Ok to commit to 4.9 branch and mainline? Bye, -Andreas- 2015-02-09 Andreas Krebbel PR target/64979 * config/s390/s390.c (s390_va_start): M