Re: [PATCH] S/390: Fix r6 vararg handling.

2016-02-05 Thread Andreas Krebbel
On 02/05/2016 12:50 PM, Jakub Jelinek wrote: > On Fri, Feb 05, 2016 at 12:43:38PM +0100, Andreas Krebbel wrote: >> Dominik just made me aware of this stupid copy and paste bug which made me >> ending up with the very >> same loops twice :( >> I've committed the attached patch to fix this: >> >> di

Re: [PATCH] S/390: Fix r6 vararg handling.

2016-02-05 Thread Jakub Jelinek
On Fri, Feb 05, 2016 at 12:43:38PM +0100, Andreas Krebbel wrote: > Dominik just made me aware of this stupid copy and paste bug which made me > ending up with the very > same loops twice :( > I've committed the attached patch to fix this: > > diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/

Re: [PATCH] S/390: Fix r6 vararg handling.

2016-02-05 Thread Andreas Krebbel
On 02/04/2016 05:58 PM, Andreas Krebbel wrote: > +static void > +s390_register_info_set_ranges () > +{ > + int i, j; > + > + /* Find the first and the last save slot supposed to use the stack > + to set the restore range. > + Vararg regs might be marked as save to stack but only the > +

[PATCH] S/390: Fix r6 vararg handling.

2016-02-04 Thread Andreas Krebbel
This patch fixes a problem introduced with the GPR into FPR slot save feature for leaf functions. r6 is argument register as well as call-saved. Currently we might decide that it will be a candidate for being saved into an FPR. If it turns out later that r6 also needs to be saved due to being re