Re: IRA performance regressions on PPC

2008-09-12 Thread Vladimir Makarov
Vladimir Makarov wrote: Luis Machado wrote: Upon further investigation on facerec's regression, it looks like the code generated by the IRA-enabled gcc has many more spills than the one with a disabled IRA, twice or sometimes three times more. I'm trying to reduce the testcase a bit further

Re: IRA performance regressions on PPC

2008-09-12 Thread Luis Machado
Hi Vladimir, Firstly, thanks for looking into this. Analysis of 187.facerec problem was actually easier than applu one. It has one very hot (80%) function localmove::graphRoutines.f90 and there is only one hot loop in the function. Although the loop is pretty big because of inlining

Re: IRA performance regressions on PPC

2008-09-11 Thread Vladimir Makarov
Luis Machado wrote: Upon further investigation on facerec's regression, it looks like the code generated by the IRA-enabled gcc has many more spills than the one with a disabled IRA, twice or sometimes three times more. I'm trying to reduce the testcase a bit further so it's simpler to

Re: IRA performance regressions on PPC

2008-09-09 Thread Luis Machado
On Mon, 2008-09-08 at 09:47 -0400, Vladimir Makarov wrote: Jeff Law wrote: H.J. Lu wrote: My understanding is PowerPC is quite sensitive to choice of register as shown in PR 28690. IRA merge may make fixes for PR 28690 ineffective. There are a few small testcases in PR 28690. You can

Re: IRA performance regressions on PPC

2008-09-09 Thread H.J. Lu
On Tue, Sep 9, 2008 at 11:13 AM, Luis Machado [EMAIL PROTECTED] wrote: On Mon, 2008-09-08 at 09:47 -0400, Vladimir Makarov wrote: Jeff Law wrote: H.J. Lu wrote: My understanding is PowerPC is quite sensitive to choice of register as shown in PR 28690. IRA merge may make fixes for PR 28690

Re: IRA performance regressions on PPC

2008-09-08 Thread Vladimir Makarov
Jeff Law wrote: H.J. Lu wrote: My understanding is PowerPC is quite sensitive to choice of register as shown in PR 28690. IRA merge may make fixes for PR 28690 ineffective. There are a few small testcases in PR 28690. You can check if those problems in PR 28690 come back due to IRA merge. Also,

Re: IRA performance regressions on PPC

2008-09-06 Thread Richard Sandiford
H.J. Lu [EMAIL PROTECTED] writes: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: 1-3 days usually because gcc community and RA reviewers are very responsive. So I don't see a big difference in using ira-merge and trunk. I'd only recommend to apply patch

Re: IRA performance regressions on PPC

2008-09-06 Thread Luis Machado
On Fri, 2008-09-05 at 10:34 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: H.J. Lu keeps ira-branch merge more fresh than trunk. But the lag is only I won't apply any non-IRA related patches to ira-merge branch so that you can get a fair

Re: IRA performance regressions on PPC

2008-09-06 Thread H.J. Lu
On Sat, Sep 6, 2008 at 12:47 AM, Richard Sandiford [EMAIL PROTECTED] wrote: H.J. Lu [EMAIL PROTECTED] writes: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: 1-3 days usually because gcc community and RA reviewers are very responsive. So I don't see a big difference

Re: IRA performance regressions on PPC

2008-09-06 Thread H.J. Lu
On Sat, Sep 6, 2008 at 7:05 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 10:34 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: H.J. Lu keeps ira-branch merge more fresh than trunk. But the lag is only I won't apply any

Re: IRA performance regressions on PPC

2008-09-06 Thread Luis Machado
On Sat, 2008-09-06 at 07:49 -0700, H.J. Lu wrote: On Sat, Sep 6, 2008 at 7:05 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 10:34 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: H.J. Lu keeps ira-branch merge more

Re: IRA performance regressions on PPC

2008-09-06 Thread H.J. Lu
On Sat, Sep 6, 2008 at 11:24 AM, Luis Machado [EMAIL PROTECTED] wrote: On Sat, 2008-09-06 at 07:49 -0700, H.J. Lu wrote: On Sat, Sep 6, 2008 at 7:05 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 10:34 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 10:24 AM, Vladimir

Re: IRA performance regressions on PPC

2008-09-06 Thread Jeff Law
H.J. Lu wrote: On Sat, Sep 6, 2008 at 11:24 AM, Luis Machado [EMAIL PROTECTED] wrote: On Sat, 2008-09-06 at 07:49 -0700, H.J. Lu wrote: On Sat, Sep 6, 2008 at 7:05 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 10:34 -0700, H.J. Lu wrote: On Fri, Sep

IRA performance regressions on PPC

2008-09-05 Thread Luis Machado
Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from 08/26 (http://gcc.gnu.org/ml/gcc-cvs/2008-08/msg01152.html) caused a significant regression on both facerec (~17%) and applu (~4%) for 64-bit PPC. There are other degradations that i'm still working on

Re: IRA performance regressions on PPC

2008-09-05 Thread H.J. Lu
On Fri, Sep 5, 2008 at 6:59 AM, Luis Machado [EMAIL PROTECTED] wrote: Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from 08/26 (http://gcc.gnu.org/ml/gcc-cvs/2008-08/msg01152.html) caused a significant regression on both facerec (~17%) and applu

Re: IRA performance regressions on PPC

2008-09-05 Thread Luis Machado
On Fri, 2008-09-05 at 07:16 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 6:59 AM, Luis Machado [EMAIL PROTECTED] wrote: Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from 08/26 (http://gcc.gnu.org/ml/gcc-cvs/2008-08/msg01152.html) caused a

Re: IRA performance regressions on PPC

2008-09-05 Thread Vladimir Makarov
Luis Machado wrote: Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from 08/26 (http://gcc.gnu.org/ml/gcc-cvs/2008-08/msg01152.html) caused a significant regression on both facerec (~17%) and applu (~4%) for 64-bit PPC. There are other degradations that

Re: IRA performance regressions on PPC

2008-09-05 Thread Luis Machado
On Fri, 2008-09-05 at 09:03 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 8:01 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 07:16 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 6:59 AM, Luis Machado [EMAIL PROTECTED] wrote: Hi Vladimir, I was just going through some

Re: IRA performance regressions on PPC

2008-09-05 Thread Luis Machado
On Fri, 2008-09-05 at 12:36 -0400, Vladimir Makarov wrote: Luis Machado wrote: Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from 08/26 (http://gcc.gnu.org/ml/gcc-cvs/2008-08/msg01152.html) caused a significant regression on both facerec (~17%)

Re: IRA performance regressions on PPC

2008-09-05 Thread H.J. Lu
On Fri, Sep 5, 2008 at 8:01 AM, Luis Machado [EMAIL PROTECTED] wrote: On Fri, 2008-09-05 at 07:16 -0700, H.J. Lu wrote: On Fri, Sep 5, 2008 at 6:59 AM, Luis Machado [EMAIL PROTECTED] wrote: Hi Vladimir, I was just going through some benchmarks on PPC and noticed that your patch from

Re: IRA performance regressions on PPC

2008-09-05 Thread Vladimir Makarov
Luis Machado wrote: This is a Power6 4.7Ghz (altivec supported) Great. Now I have an access to power6. So I am going to try it too. What options (especially march or mtune) you are using? IRA is very sensitive to correct times of ld/st/moves in machine description. I'm

Re: IRA performance regressions on PPC

2008-09-05 Thread H.J. Lu
On Fri, Sep 5, 2008 at 10:24 AM, Vladimir Makarov [EMAIL PROTECTED] wrote: H.J. Lu keeps ira-branch merge more fresh than trunk. But the lag is only I won't apply any non-IRA related patches to ira-merge branch so that you can get a fair comparison for IRA without regressions introduced by