This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH] check_dbra_loop thinko
- To: Mark Mitchell <mark at codesourcery dot com>
- Subject: [PATCH] check_dbra_loop thinko
- From: Dan Nicolaescu <dann at godzilla dot ICS dot UCI dot EDU>
- Date: Tue, 04 Sep 2001 14:14:51 -0700
- Cc: gcc-patches at gcc dot gnu dot org
This patch
2001-05-11 Mark Mitchell <mark@codesourcery.com>
[snip]
(check_dbra_loop): Always clear reversible_mem_store if there
are any memory sets.
is a thinko, it makes the for loop following that line dead code....
It is also present in 3.0
Is the following the correct fix?
2001-09-04 Dan Nicolaescu <dann@ics.uci.edu>
* loop.c (check_dbra_loop): Set reversible_mem_store before the
loop.
*** loop.c.~1.353.~ Tue Sep 4 09:31:25 2001
--- loop.c Tue Sep 4 13:24:02 2001
***************
*** 7380,7386 ****
written to was different, then we could reverse the
store -- but we don't presently have any way of
knowing that. */
! reversible_mem_store = 0;
/* If the store depends on a register that is set after the
store, it depends on the initial value, and is thus not
--- 7380,7386 ----
written to was different, then we could reverse the
store -- but we don't presently have any way of
knowing that. */
! reversible_mem_store = 1;
/* If the store depends on a register that is set after the
store, it depends on the initial value, and is thus not