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][RFA] Many store-motion.c cleanups to make it a pass in its own right

From: Steven Bosscher <stevenb dot gcc at gmail dot com>
To: Richard Guenther <richard dot guenther at gmail dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 1 May 2009 21:38:38 +0200
Subject: Re: [PATCH][RFA] Many store-motion.c cleanups to make it a pass in its own right
References: <571f6b510905010659m785ad75clde9931603379f454@mail.gmail.com> <84fc9c000905011226j16c23390x1780195eaf39cf3d@mail.gmail.com>

On Fri, May 1, 2009 at 9:26 PM, Richard Guenther
<richard.guenther@gmail.com> wrote:

> Ok with this spurious(?) change reverted:
>
> -Common Report Var(flag_gcse_sm) Init(0) Optimization
> +Common Report Var(flag_gcse_sm) Init(1) Optimization

Ah, that's not spurious.  I just svn diff'ed but forgot to disable
store motion again.  This is the patch I bootstrapped, i.e. with store
motion enabled. Amazingly, this works, which is unusual, for a pass
that isn't enabled by default. I enabled it to make sure I didn't
break anything. There are almost no test cases for store motion in the
test suite after all: just three test cases that have -fgcse-sm.

Anyway, thanks, I'll commit the patch in a minute.

Ciao!
Steven

References:
- [PATCH][RFA] Many store-motion.c cleanups to make it a pass in its own right
  - From: Steven Bosscher
- Re: [PATCH][RFA] Many store-motion.c cleanups to make it a pass in its own right
  - From: Richard Guenther

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