Re: PowerPC prologue and epilogue 6

2012-05-31 Thread Mike Stump
On May 31, 2012, at 6:42 AM, Dominique Dhumieres wrote: >> This is really stretching my testsuite knowledge. Maybe add >> >> /* { dg-additional-options "-mdynamic-no-pic" { target *-*-darwin* } } */ > > Using > > /* { dg-options "-fno-inline -fomit-frame-pointer" } */ > /* { dg-additional-optio

Re: PowerPC prologue and epilogue 6

2012-05-31 Thread Dominique Dhumieres
> This is really stretching my testsuite knowledge. Maybe add > >/* { dg-additional-options "-mdynamic-no-pic" { target *-*-darwin* } } */ Using /* { dg-options "-fno-inline -fomit-frame-pointer" } */ /* { dg-additional-options "-mdynamic-no-pic" { target *-*-darwin* } } */ works for me on powe

Re: PowerPC prologue and epilogue 6

2012-05-31 Thread Alan Modra
On Thu, May 31, 2012 at 02:16:32PM +0200, Dominique Dhumieres wrote: > (the final patch will require the suitable dg directives;-). This is really stretching my testsuite knowledge. Maybe add /* { dg-additional-options "-mdynamic-no-pic" { target *-*-darwin* } } */ -- Alan Modra Australia Dev

Re: PowerPC prologue and epilogue 6

2012-05-31 Thread Dominique Dhumieres
> Please try out this patch on Darwin. Bootstrapped and regression > tested powerpc-linux. I have applied the patch to r188026 and updated the build. As patched the test gcc.target/powerpc/savres.c now fails with FAIL: gcc.target/powerpc/savres.c (test for excess errors) Excess errors: ld_classi

Re: PowerPC prologue and epilogue 6

2012-05-31 Thread Alan Modra
On Thu, May 31, 2012 at 10:41:26AM +0930, Alan Modra wrote: > Looks like it is one I introduced. gcc-4.6 uses r12 to save altivec > regs, my new code tries to use r11. Will fix. Please try out this patch on Darwin. Bootstrapped and regression tested powerpc-linux. gcc/ * config/rs6000/

Re: PowerPC prologue and epilogue 6

2012-05-30 Thread Alan Modra
On Thu, May 31, 2012 at 09:43:09AM +0930, Alan Modra wrote: > real bug that the register checks have uncovered. I haven't > determined whether this is a new bug introduced with my prologue > changes, or whether it's a long-standing bug. I suspect the latter. Looks like it is one I introduced. g

Re: PowerPC prologue and epilogue 6

2012-05-30 Thread Alan Modra
On Wed, May 30, 2012 at 03:21:28PM +0200, Dominique Dhumieres wrote: > I get an ICE of the form > > /opt/gcc/work/gcc/testsuite/gcc.target/powerpc/savres.c: In function 'nb_all': > /opt/gcc/work/gcc/testsuite/gcc.target/powerpc/savres.c:473:3: internal > compiler error: in rs6000_emit_prologue, a

Re: PowerPC prologue and epilogue 6

2012-05-30 Thread Dominique Dhumieres
> Yes indeed, and it would be wise to ensure torture-options.exp is > loaded too. I'm committing the following as obvious. Thanks > Hmm, this will be because darwin is PIC by default. Does adding > -static to the dg-options line in savres.c fix the darwin fail? With the following change --- /

Re: PowerPC prologue and epilogue 6

2012-05-30 Thread Alan Modra
On Tue, May 29, 2012 at 09:26:37PM +0200, Dominique Dhumieres wrote: > gcc-dg-runtest [list $srcdir/$subdir/savres.c] $alti > > # All done. > +torture-finish > dg-finish > > is required to avoid the errors of the kind > > ERROR: tcl error sourcing > /home/gccbuild/gcc_trunk_anonsvn/gcc/gcc

Re: PowerPC prologue and epilogue 6

2012-05-29 Thread Dominique Dhumieres
Alan, I think the following patch --- ../_gcc_clean/gcc/testsuite/gcc.target/powerpc/powerpc.exp 2012-05-02 14:25:40.0 +0200 +++ ../work/gcc/testsuite/gcc.target/powerpc/powerpc.exp2012-05-29 21:14:48.0 +0200 @@ -47,4 +47,5 @@ set-torture-options $SAVRES_TEST_OPTS gcc-

Re: PowerPC prologue and epilogue 6

2012-04-25 Thread David Edelsohn
On Wed, Apr 25, 2012 at 1:20 AM, Alan Modra wrote: > This patch adds a testcase to verify register saves and restores. > I tried to write it so that it will run on all powerpc targets.  From > past experience it probably won't.  OK to apply anyway, and fix > fallout later? > >        * gcc.target

Re: PowerPC prologue and epilogue 6

2012-04-24 Thread Alan Modra
On Tue, Apr 24, 2012 at 07:19:42PM -0400, David Edelsohn wrote: > This patch is okay with the macro usage fix. Thanks, series 2 to 6 committed as 186796, 186797, 186798, 186799, 186800. I noticed after I committed the lot that 186797 has some duplicated lines (harmless), corrected in 186798, and

Re: PowerPC prologue and epilogue 6

2012-04-24 Thread David Edelsohn
On Sat, Apr 21, 2012 at 2:48 AM, Alan Modra wrote: > This patch adds out-of-line vector saves and restores.  To do this I > made some infrastructure changes to various functions like > rs6000_emit_savres_rtx that currently take boolean parameters (savep, > gpr, and lr).  Rather than add yet anothe

PowerPC prologue and epilogue 6

2012-04-20 Thread Alan Modra
This patch adds out-of-line vector saves and restores. To do this I made some infrastructure changes to various functions like rs6000_emit_savres_rtx that currently take boolean parameters (savep, gpr, and lr). Rather than add yet another boolean to specify vector regs, I chose to lump them all t