This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Enable fmove-loop-invariants by default

From: Steven Bosscher <stevenb dot gcc at gmail dot com>
To: gcc-patches at gcc dot gnu dot org
Cc: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>, Steven Bosscher <stevenb dot gcc at gmail dot com>, Mark Mitchell <mark at codesourcery dot com>, Roger Sayle <roger at eyesopen dot com>
Date: Sun, 26 Feb 2006 19:34:51 +0100
Subject: Re: [patch] Enable fmove-loop-invariants by default
References: <Pine.LNX.4.44.0602100721040.28186-100000@www.eyesopen.com> <200602261329.47753.steven@gcc.gnu.org> <20060226183129.GA716@atrey.karlin.mff.cuni.cz>

On Sunday 26 February 2006 19:31, Zdenek Dvorak wrote:
> > This already passed bootstrap&testing on x86-64-unknown-linux-gnu.  I'm
> > running mips-sim now as extra precaution.

...that has passed now too.

> I have already tested a similar patch; I find it somewhat suspicious
> that this one passed regtesting, as it leaves the following testcases
> that use -frerun-loop-opt untouched:
>
> g++.old-deja/g++.robertl/eb132.C
> gcc.dg/20031201-1.c

I changed those after diffing.

> Also, this one misses removal of -fstrength-reduce, whose only use also
> disappears with pass_loop_optimize 

This one, I really missed, sorry.

Are you going to apply your patch soon?

Gr.
Steven

Follow-Ups:
- Re: [patch] Enable fmove-loop-invariants by default
  - From: Zdenek Dvorak

References:
- Re: [patch] Enable fmove-loop-invariants by default
  - From: Roger Sayle
- Re: [patch] Enable fmove-loop-invariants by default
  - From: Steven Bosscher
- Re: [patch] Enable fmove-loop-invariants by default
  - From: Zdenek Dvorak

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]