This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Bug tree-optimization/21829] [4.1/4.2 Regression] missed jump threading after unroller
- From: Jeffrey A Law <law at redhat dot com>
- To: Richard Guenther <richard dot guenther at gmail dot com>
- Cc: gcc-bugzilla at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Wed, 22 Mar 2006 07:01:20 -0700
- Subject: Re: [Bug tree-optimization/21829] [4.1/4.2 Regression] missed jump threading after unroller
- References: <bug-21829-6528@http.gcc.gnu.org/bugzilla/> <20060211005940.30518.qmail@sourceware.org> <1142917666.23430.71.camel@to-dhcp19.toronto.redhat.com> <84fc9c000603220314j17ad999kd8e3fd3d58e96e88@mail.gmail.com>
- Reply-to: law at redhat dot com
On Wed, 2006-03-22 at 12:14 +0100, Richard Guenther wrote:
> On 3/21/06, Jeffrey A Law <law@redhat.com> wrote:
> > It turns out this specialized PHI optimization pass is as effective
> > as running copy-prop and CCP on PHI nodes after DOM. Better yet, it
> > is a teeny tiny slowdown compared to just running the stripped down
> > copyprop. ie, for an almost unmeasurable slowdown we can do both
> > constant and copy propagation instead of just copy propagation.
>
> This patch caused a compile-time regression from 139s to 143s, resp.
> 192s to 197s (leafify) accounted by increases of operand scan / SSA incremental
> and tree CCP times for compiling tramp3d. Also memory usage during compiling
> went up from 655494 kB to 660626kB (this may be due to the VRP patch, though).
>
> Runtime of tramp3d did not improve but regress slightly (but that
> might be in the
> noise - we'll see).
>
> For this simple cleanup pass can you try updating SSA form manually please?
I'm more than happy to look at it; however, be aware that if you're
seeing increased time in CCP then either you're seeing some truly
bizzarre secondary effect or your testing methodology is suspect.
The patch did not affect CCP. In fact, the changes only affect
passes which run *after* CCP in the optimization pipeline.
Jeff