This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: GCSE store motion
- From: "David S. Miller" <davem at redhat dot com>
- To: mark at codesourcery dot com
- Cc: gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Wed, 15 May 2002 00:33:48 -0700 (PDT)
- Subject: Re: GCSE store motion
- References: <20020514.220313.18285009.davem@redhat.com><4690000.1021447609@gandalf.codesourcery.com>
From: Mark Mitchell <mark@codesourcery.com>
Date: Wed, 15 May 2002 00:26:49 -0700
--On Tuesday, May 14, 2002 10:03:13 PM -0700 "David S. Miller"
<davem@redhat.com> wrote:
>
> While walking over diffs between 3.1 and the mainline I came across
> PR/5200. Basically the fix on the branch just disabled store-motion
> in GCSE.
>
> This is one of those "fix it properly on the mainline" cases.
>
> So who is going to step up and work on fixing store motion?
> Aparently Dan Berlin and Jakub have some idea of the problems.
> Refer to the GNATS PR for more information.
For the time being, please disable it on the mainline too.
Done, as follows:
2002-03-09 Jakub Jelinek <jakub@redhat.com>
PR optimization/5172, optimization/5200
* gcse.c (gcse_main): Disable store_motion.
--- gcse.c.~1~ Sun Apr 28 21:54:59 2002
+++ gcse.c Tue May 14 21:00:24 2002
@@ -905,7 +905,8 @@
end_alias_analysis ();
allocate_reg_info (max_reg_num (), FALSE, FALSE);
- if (!optimize_size && flag_gcse_sm)
+ /* Store motion disabled until it is fixed. */
+ if (0 && !optimize_size && flag_gcse_sm)
store_motion ();
/* Record where pseudo-registers are set. */
return run_jump_opt_after_gcse;